Wednesday, 2024-07-10

opendevreviewMerged openstack/neutron master: [DHCP] Lock the execution of ``_dhcp_ready_ports_loop``  https://review.opendev.org/c/openstack/neutron/+/92362500:24
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Iterate through table rows using ``row.items()``  https://review.opendev.org/c/openstack/neutron/+/92381205:54
slaweqfrickler: hi, I heard yesterday on the ci-meeting that you may know something about issue with the neutron-tempest-plugin-designate-scenario job. Is that correct? I reported bug in https://bugs.launchpad.net/designate/+bug/207262706:59
fricklerslaweq: thx for the bug, johnsom started to look at this already. added the oslo.log reference that likely caused this latent issue to surface07:18
slaweqfrickler thx for confirmation :)07:24
ralonsohslaweq, should we disable this job temporarily? it is preventing any neutron job to be merged08:08
ralonsohlajoskatona, ^08:08
lajoskatonaralonsoh: +1 for making it non-voting temporarily, when the issue is fixed in oslo.log or in designate we can make it voting again08:17
ralonsohperfect, I'll push a patch08:17
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Mark ``neutron-tempest-plugin-designate-scenario`` as non-voting  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92382608:20
lajoskatonaralonsoh: thanks, I don't see patch for it in designate repo at the moment.08:21
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-designate-scenario non-voting temporary  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92382708:26
ralonsohslaweq, I pushed a patch 5 mins before08:26
ralonsohok, I'll abandon mine08:27
slaweqralonsoh ah, sorry, I missed Your patch, I was in the meeting, had some spare time so I though I will do it :)08:36
lajoskatonaralonsoh: I quickly checked taas vs wsgi problem, and what I see is that the service_provider isn't loaded due to that the taas_plugin.ini file is not loaded when neutron-services-server.service starts10:01
ralonsohwhen is the service_provider loaded? (when using eventlet)10:02
lajoskatonaralonsoh: if I add service_provider = TAAS:TA..... blah blah to neutron.conf the server at lease start (I have to continue this to see what happens next)10:02
ralonsohso this is just a configuration problem, right?10:03
lajoskatonawhen the service plugin is started: https://opendev.org/openstack/tap-as-a-service/src/branch/master/neutron_taas/services/taas/taas_plugin.py#L5810:03
ralonsohok so we are not passing taas_plugin.ini to the wsgi server10:03
lajoskatonaralonsoh: looks like, after lunch I push devstack forward to see if really everything works as expected10:03
ralonsohlajoskatona, the config files are defined when the service is created by devstack10:05
ralonsohwe have other jobs, like sfc, using it 10:05
ralonsohlet me check to what file are we adding the config10:05
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Remove tenant_id from the security group default rules definition  https://review.opendev.org/c/openstack/neutron/+/91966510:31
opendevreviewMerged openstack/neutron stable/2023.2: Checking pci_slot to avoid changing staus to BUILD forever  https://review.opendev.org/c/openstack/neutron/+/92377310:47
opendevreviewMerged openstack/neutron stable/2023.1: Checking pci_slot to avoid changing staus to BUILD forever  https://review.opendev.org/c/openstack/neutron/+/92377410:47
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron  https://review.opendev.org/c/openstack/tap-as-a-service/+/92383711:57
opendevreviewLajos Katona proposed openstack/neutron unmaintained/zed: Checking pci_slot to avoid changing staus to BUILD forever  https://review.opendev.org/c/openstack/neutron/+/92383812:06
lajoskatonaralonsoh: https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 , locally it seems to fix the issue, but there's still things which I don't understand12:25
ralonsohlajoskatona, let me check12:25
lajoskatonaralonsoh: in my local env devstack imeouts waiting for neutron-api service, but when I check the service is running, so there are questions what is really happenning there12:26
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Adding manager role support  https://review.opendev.org/c/openstack/neutron/+/92357812:32
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Change to new syntax of calling super() in policies unit tests modules  https://review.opendev.org/c/openstack/neutron/+/92384012:32
opendevreviewMerged openstack/tap-as-a-service unmaintained/zed: Update .gitreview for unmaintained/zed  https://review.opendev.org/c/openstack/tap-as-a-service/+/91757713:09
mnaserhi neutron team, https://review.opendev.org/c/openstack/neutron/+/916123 does this seem reasonable to backport? I think it is breaking network logging for us in split scenario13:51
mnaserI dont see the harm, ill push it up and get reviews I ugess13:52
opendevreviewMohammed Naser proposed openstack/neutron stable/2024.1: Start logging plugin RPC via service framework  https://review.opendev.org/c/openstack/neutron/+/92384813:52
opendevreviewMohammed Naser proposed openstack/neutron stable/2023.2: Start logging plugin RPC via service framework  https://review.opendev.org/c/openstack/neutron/+/92384913:53
opendevreviewMohammed Naser proposed openstack/neutron stable/2023.1: Start logging plugin RPC via service framework  https://review.opendev.org/c/openstack/neutron/+/92385013:53
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Add port trusted vif extension  https://review.opendev.org/c/openstack/neutron-lib/+/92386014:46
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron  https://review.opendev.org/c/openstack/tap-as-a-service/+/92383716:32
opendevreviewMerged openstack/neutron master: [OVN] Iterate through table rows using ``row.items()``  https://review.opendev.org/c/openstack/neutron/+/92381217:23
opendevreviewMerged openstack/neutron master: Change the "get_metadata_port" name and input variable  https://review.opendev.org/c/openstack/neutron/+/92340619:50
opendevreviewMerged openstack/neutron master: Check subnet service_type for subnet_create  https://review.opendev.org/c/openstack/neutron/+/90122519:51
opendevreviewMerged openstack/neutron master: Refactor external-network update to router gateway cascading effect  https://review.opendev.org/c/openstack/neutron/+/90839319:51
opendevreviewmelanie witt proposed openstack/neutron master: Add dynamic lookup for tcpdump binary  https://review.opendev.org/c/openstack/neutron/+/92307320:21
opendevreviewmelanie witt proposed openstack/neutron master: Add dynamic lookup for tcpdump binary  https://review.opendev.org/c/openstack/neutron/+/92307320:24
*** haleyb is now known as haleyb|out22:50

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!