Monday, 2024-03-25

opendevreviewYoungjun Yoo proposed openstack/neutron master: Refactor some docstrings  https://review.opendev.org/c/openstack/neutron/+/91348600:39
*** mklejn__ is now known as mklejn05:49
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: ensure that cleaning DHCP process with one segment happens first  https://review.opendev.org/c/openstack/neutron/+/90561709:27
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix dhcp cleaning stale devices process when enable action  https://review.opendev.org/c/openstack/neutron/+/90725009:27
ralonsohslaweq, hello! if you have a couple of mins, no rush09:32
ralonsohhttps://review.opendev.org/c/openstack/neutron-tempest-plugin/+/91110509:32
slaweqralonsoh sure, just let me finish one thing and I will check it09:38
ralonsohfor sure09:38
ralonsohah, and https://review.opendev.org/c/openstack/neutron-lib/+/91227309:38
ralonsoh(related)09:38
ralonsohthanks a lot!09:39
slaweqdone09:49
sahido/10:09
sahidqucik question regarding dhcp agent,10:09
sahidif an agent is down, the networks on that agent get rescheduled on the other automatically?10:10
sahidI heard something like that but I did not find anything related in the code10:10
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix auto_scheduler_network understanding dhcp_agents_per_network  https://review.opendev.org/c/openstack/neutron/+/91070810:22
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP - Add ``OVNGatewayHAChassisGroup`` scheduler class  https://review.opendev.org/c/openstack/neutron/+/87203310:24
ralonsohsahid, if an DHCP agent is down, the DhcpAgentSchedulerDbMixin has a periodic checker 10:26
ralonsohthat will remove the NetworkDhcpAgentBinding associations between the network and the agent10:27
ralonsohand once that happens, the scheduler will re-schedule the network to another alive agent10:27
sahidralonsoh: oh that is really interesting, thanks a lot I will look more on DhcpAgentSchedulerDbMixin10:45
opendevreviewMerged openstack/neutron-specs master: Spec folder for 2024.2 cycle  https://review.opendev.org/c/openstack/neutron-specs/+/91404311:05
opendevreviewRodolfo Alonso proposed openstack/neutron-vpnaas-dashboard stable/2024.1: Explicitly convert ``assertQuerysetEqual`` input parameters  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91401211:14
opendevreviewOpenStack Release Bot proposed openstack/neutron-vpnaas-dashboard stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91289011:15
opendevreviewOpenStack Release Bot proposed openstack/neutron-vpnaas-dashboard stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91289211:15
ralonsohlajoskatona, ^^ if you have 1 minute11:15
ralonsohthanks!11:15
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add "subnet-external-network" extension to "subnet" resource  https://review.opendev.org/c/openstack/neutron/+/90731311:23
opendevreviewMerged openstack/neutron-lib master: Enable filtering flag in subnet 'router:external' field  https://review.opendev.org/c/openstack/neutron-lib/+/91227311:26
lajoskatonaralonsoh: thanks11:44
lajoskatonaslaweq, ykarel: Hi, if you have a sec please check this from ralonsoh: https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91401211:45
slaweqralonsoh done11:49
opendevreviewMerged openstack/neutron-vpnaas-dashboard stable/2024.1: Explicitly convert ``assertQuerysetEqual`` input parameters  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91401212:06
opendevreviewMerged openstack/neutron-vpnaas-dashboard stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91289012:06
opendevreviewMerged openstack/neutron-vpnaas-dashboard stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/91289212:06
opendevreviewMerged openstack/neutron-tempest-plugin master: Check device_owner with actual availability zone  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/89971412:39
fungiralonsoh: can you have a look at the comments on https://review.opendev.org/c/openstack/neutron/+/913708 from thursday? you've blocked the corresponding stable/2024.1 backport and we're dangerously close to releasing 2024.1/caracal with that security vulnerability if we don't get it merged and a new rc tagged asap13:19
opendevreviewLajos Katona proposed openstack/neutron-lib master: api-ref: add api-ref for remote_address_group in sg rules  https://review.opendev.org/c/openstack/neutron-lib/+/91388813:34
ralonsohfungi, this is a security related patch14:12
ralonsohand the testing provided is almost null14:12
fungiralonsoh: yes, there is a follow-up change linked in the bug which adds testing14:12
ralonsohI saw it this morning14:13
ralonsohand Linux Bridge is failing14:13
fungiralonsoh: regardless, you asked questions, two others tried to answer you and asked for additional input. just trying to make sure that set of changes is a priority since we switched it to public specifically in order to try and make faster progress (in hopes of having it fixed before the release)14:14
fungiso help with that is definitely appreciated14:16
ralonsohfor sure, I'll check it today14:16
fungithanks!14:16
haleybralonsoh: linux bridge is failing in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/913850 because the fix for remote address groups is only in master14:17
ralonsohyes14:17
ralonsohI'm checking the logs now14:17
ralonsohand I'm happy with the n-t-p patch14:17
ralonsohthis is what I was expecting first14:17
ralonsoha feature/bug like this cannot be merged without the proper testing14:18
haleybralonsoh: you're good with a 3-second sleep?14:18
ralonsohbut now checking the n-t-p (in master only), it seems to work14:18
ralonsohhmmmmm I'm actually checking that right now14:18
ralonsohgive me some mins14:18
haleybsee my comments, no hurry14:18
ralonsohI agree with you: we have the active waits in the "self.check_remote_connectivity" and "ping_ip_address" calls14:20
ralonsohin any case, I'll unblock the backport, as seen in this patch, this functionality works fine (in master and not in stable branches, as expected)14:20
fungithanks again for the quick reviews!14:22
lajoskatonahaleyb, ralonsoh, slaweq: Hi, if you have a min could you check this patch from Liu please: https://review.opendev.org/c/openstack/neutron/+/907382 ? 14:30
ralonsohsure14:30
lajoskatonahaleyb, ralonsoh, slaweq: the patch from far related to https://review.opendev.org/c/openstack/neutron/+/905125 and https://launchpad.net/bugs/204878514:30
ralonsohlajoskatona, how are both patches related?14:32
ralonsohsorry, I don't see the link14:32
lajoskatonaralonsoh: sorry, in the launchpad bug there is really no mention of the link, in our enviroment originally we found these with trunks, and one part of the issue was with https://review.opendev.org/c/openstack/neutron/+/905125 and in egress direction https://review.opendev.org/c/openstack/neutron/+/907382 fixes the 2nd part of the experienced issue14:37
ralonsohahhh understood14:40
opendevreviewMerged openstack/neutron master: Clean up state VRRP PID file  https://review.opendev.org/c/openstack/neutron/+/91204716:12
kevkoHi, i have this in our CI tempest system 16:22
kevkohttps://a6ab593594613a0a633c-3e445c1a5b8a8c84f71b5a61cff5a09b.ssl.cf2.rackcdn.com/913728/61/check/kolla-ansible-rocky9-multinode-ipv6/5c981d2/primary/logs/kolla/all-ERROR.txt16:22
kevkohttps://a6ab593594613a0a633c-3e445c1a5b8a8c84f71b5a61cff5a09b.ssl.cf2.rackcdn.com/913728/61/check/kolla-ansible-rocky9-multinode-ipv6/5c981d2/primary/logs/tempest/reports/tempest-smoke.html << tempest report html16:23
kevkois it know bug ? or ? 16:23
lajoskatonakevko: is an OVS and l3 agent based deployment am I right?16:44
opendevreviewMerged openstack/neutron-tempest-plugin master: Add extension "subnet-external-network"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/91110516:49
opendevreviewBrian Haley proposed openstack/neutron master: Update Neutron Liuetenants  https://review.opendev.org/c/openstack/neutron/+/91414618:28
*** jamesdenton_ is now known as jamesdenton18:58
opendevreviewAnton Kurbatov proposed openstack/neutron master: Fixing the 500 HTTP code in the metadata service if Nova is down  https://review.opendev.org/c/openstack/neutron/+/91415419:06
opendevreviewAnton Kurbatov proposed openstack/neutron master: Fixing the 500 HTTP code in the metadata service if Nova is down  https://review.opendev.org/c/openstack/neutron/+/91415419:13
*** dasm is now known as Guest391520:22
*** Guest3915 is now known as dasm20:30
mlavalle420:59
haleyb621:02
opendevreviewMiguel Lavalle proposed openstack/neutron master: Check unspecified flavor in user defined driver  https://review.opendev.org/c/openstack/neutron/+/91416222:39
opendevreviewMiguel Lavalle proposed openstack/neutron master: Enable HA for OVN router flavors  https://review.opendev.org/c/openstack/neutron/+/90151323:53

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