| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: tooz service discovery https://review.opendev.org/c/openstack/oslo.messaging/+/1007620 | 02:25 |
|---|---|---|
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: wire protocol and client https://review.opendev.org/c/openstack/oslo.messaging/+/1007621 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: server side and entry point https://review.opendev.org/c/openstack/oslo.messaging/+/1007622 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: fanout broadcaster https://review.opendev.org/c/openstack/oslo.messaging/+/1007623 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: DevStack plugin, CI and docs https://review.opendev.org/c/openstack/oslo.messaging/+/1007624 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: oslo.metrics counters https://review.opendev.org/c/openstack/oslo.messaging/+/1011115 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: interop with the original driver https://review.opendev.org/c/openstack/oslo.messaging/+/1007625 | 02:25 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: Update Kafka for functional tests https://review.opendev.org/c/openstack/oslo.messaging/+/1011976 | 04:23 |
| rm_work | Have an agent looking into that scenario02 job failure — am I wasting time/tokens? I don’t know the history of that job tkajinam | 04:29 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: kafka: Preserve notifications at listener startup https://review.opendev.org/c/openstack/oslo.messaging/+/1011980 | 04:48 |
| rm_work | ^^ with those two it SHOULD pass | 05:07 |
| opendevreview | Michael Still proposed openstack/oslo.privsep master: Only inspect eventlet if it is already imported https://review.opendev.org/c/openstack/oslo.privsep/+/1011795 | 08:50 |
| opendevreview | Takashi Kajinami proposed openstack/osprofiler master: Do not install code to build release notes https://review.opendev.org/c/openstack/osprofiler/+/1012056 | 15:18 |
| opendevreview | Merged openstack/oslo.service master: fixture: Support threading backend for SleepFixture https://review.opendev.org/c/openstack/oslo.service/+/1011909 | 15:24 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.rootwrap master: Use real echo binary https://review.opendev.org/c/openstack/oslo.rootwrap/+/1012060 | 15:28 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.rootwrap master: DNM: Run tests with Ubuntu 26.04 https://review.opendev.org/c/openstack/oslo.rootwrap/+/1007437 | 15:28 |
| tkajinam | rm_work, thanks. I doubt that it's very popular these days ... | 15:40 |
| rm_work | sfinucan: thanks for the reviews :) I’m not sure if I’ll be able to respond fully today but I’m looking into a couple of those | 15:41 |
| tkajinam | I vaguely remember JayF mentioned they are using the kafka driver some time ago (but I might be wrong) | 15:41 |
| rm_work | tkajinam: yeah I’m not even sure what that job is for really but it as annoying me it kept breaking and figured I could throw some tokens at it while I was here lol | 15:42 |
| rm_work | Ah, ok | 15:42 |
| rm_work | I use Kafka for notifications but this isn’t that I think | 15:42 |
| tkajinam | ok so it's listener | 15:45 |
| tkajinam | the job is meant to test kakfa notification driver but it also uses that listener implementation to verify actual notification messages | 15:48 |
| tkajinam | technically you can use the listener in some services (aodh) to let them consume notifications over kafka. | 15:49 |
| tkajinam | I assume you are not using listener but only notifier | 15:50 |
| rm_work | We have Kafka listeners but not using an OpenStack implementation 😅 | 15:51 |
| tkajinam | yeah that's probably what people often do | 15:51 |
| tkajinam | because scenario002 is the only place kafka notifier is tested, and kafka listener is required to test kafka notifier let's fix the job and let these both survive | 15:55 |
| tkajinam | unless you agree we can get rid of the whole kafka driver :-P | 15:55 |
| tkajinam | updating pifpap so that it supports recent kafka with kraft would be the next thing we likely have to tackle. | 15:56 |
| tkajinam | after we get the job fixed. | 15:56 |
| rm_work | Ok, so you think those two CRs that fix it seem reasonable to do? | 16:03 |
| opendevreview | Merged openstack/tooz master: Discover etcd api version in tests https://review.opendev.org/c/openstack/tooz/+/1011896 | 16:03 |
| tkajinam | rm_work, yeah | 16:37 |
| tkajinam | I saw stephen also agreed so we can review/merge these if you remove wip | 16:37 |
| * tkajinam is looking at https://zuul.opendev.org/t/openstack/builds?job_name=glance-s3-multistore | 16:38 | |
| tkajinam | Oct 23 12:57:08.907562 np96d7cafe86b84 devstack@g-api.service[85791]: WARNING glance_store.driver [None req-171e1872-58a4-49af-8e7c-9f0073b284bb admin admin] Failed to configure store correctly: Store s3 could not be configured correctly. Reason: boto3 or botocore is not available. Disabling add method.: glance_store.exceptions.BadStoreConfiguration: Store s3 could not be configured correctly. Reason: boto3 or botocore is not available. | 16:38 |
| tkajinam | oops. wrong channel | 16:39 |
| JayF | tkajinam: rm_work: For notifications, I've heard of more operators than just my downstream utilizing kafka | 16:41 |
| JayF | tkajinam: rm_work: I do not know of any use cases of folks using them to run services; but I'm not sure that proves anytime :) | 16:42 |
| JayF | **anything | 16:42 |
| tkajinam | ok. that's good to know | 16:42 |
| tkajinam | we should even attempt to keep scenario002 voting to make sure the notifier works then | 16:43 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to restart onto updated container images, but should return within a few minutes | 17:07 | |
| rm_work | tkajinam: should I actually move it to voting in the second patch where I enable it? or maybe even just combine my CRs, honestly I originally was thinking to keep them separate since it's technically easier to review but IDK, seems odd to change and leave it broken | 18:59 |
| rm_work | in the first one | 18:59 |
| rm_work | and I meant where I "fix it" in the second patch, technically "enabling it" might mean to make it voting | 19:00 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul driver with distributed locking https://review.opendev.org/c/openstack/tooz/+/1007613 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul group membership and capabilities https://review.opendev.org/c/openstack/tooz/+/1007614 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul service catalog reads https://review.opendev.org/c/openstack/tooz/+/1007615 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: tooz service discovery https://review.opendev.org/c/openstack/oslo.messaging/+/1007620 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: wire protocol and client https://review.opendev.org/c/openstack/oslo.messaging/+/1007621 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: server side and entry point https://review.opendev.org/c/openstack/oslo.messaging/+/1007622 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: fanout broadcaster https://review.opendev.org/c/openstack/oslo.messaging/+/1007623 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: DevStack plugin, CI and docs https://review.opendev.org/c/openstack/oslo.messaging/+/1007624 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: oslo.metrics counters https://review.opendev.org/c/openstack/oslo.messaging/+/1011115 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: interop with the original driver https://review.opendev.org/c/openstack/oslo.messaging/+/1007625 | 19:13 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: kafka: Preserve notifications at listener startup https://review.opendev.org/c/openstack/oslo.messaging/+/1011980 | 19:13 |
| rm_work | stephenfin: lol sorry you JUST did +2 but I applied some changes for you | 19:14 |
| rm_work | well, not JUST i guess but | 19:14 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: kafka: Preserve notifications at listener startup https://review.opendev.org/c/openstack/oslo.messaging/+/1011980 | 19:18 |
| rm_work | and now I have posted comments/replies on everything I think | 19:31 |
| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: Add a reproducer for options missing from a private config https://review.opendev.org/c/openstack/oslo.messaging/+/1012104 | 20:04 |
| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: Register the metrics and tracing options in the transport config https://review.opendev.org/c/openstack/oslo.messaging/+/1012105 | 20:04 |
| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: doc: Add a guide to write a transport driver https://review.opendev.org/c/openstack/oslo.messaging/+/1012106 | 20:13 |
| opendevreview | Arnaud Morin proposed openstack/oslo-specs master: Fix the docs build with urllib3 2 https://review.opendev.org/c/openstack/oslo-specs/+/1012114 | 20:47 |
| opendevreview | Arnaud Morin proposed openstack/oslo-specs master: Move the http driver images next to the spec https://review.opendev.org/c/openstack/oslo-specs/+/1012115 | 20:47 |
| opendevreview | Arnaud Morin proposed openstack/oslo-specs master: oslo.messaging: Propose a BlazingMQ driver https://review.opendev.org/c/openstack/oslo-specs/+/1012116 | 20:47 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: tests: Support native functional test fixtures https://review.opendev.org/c/openstack/oslo.messaging/+/1012135 | 21:24 |
| rm_work | ^^ New one to help with non-eventlet transition | 21:31 |
| tkajinam | rm_work, you can keep the job non-voting. once we merge your two patches we monitor the job for a while and make it voting | 21:55 |
| rm_work | Ok | 22:17 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul group membership and capabilities https://review.opendev.org/c/openstack/tooz/+/1007614 | 22:23 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul service catalog reads https://review.opendev.org/c/openstack/tooz/+/1007615 | 22:23 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: DevStack plugin, CI and docs https://review.opendev.org/c/openstack/oslo.messaging/+/1007624 | 22:23 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: oslo.metrics counters https://review.opendev.org/c/openstack/oslo.messaging/+/1011115 | 22:23 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: interop with the original driver https://review.opendev.org/c/openstack/oslo.messaging/+/1007625 | 22:23 |
| rm_work | I think the two Kafka ones are good, assuming we are ok to upgrade the version that far. I addressed all of Stephen’s stuff on the consul side too I believe | 22:26 |
| rm_work | And I saw Arnaud’s CR about a driver guide and adjusted some stuff to fit that because it seemed like a good idea | 22:27 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!