opendevreview | Youngjun Yoo proposed openstack/neutron master: Refactor some docstrings https://review.opendev.org/c/openstack/neutron/+/913486 | 00:39 |
---|---|---|
*** mklejn__ is now known as mklejn | 05:49 | |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: ensure that cleaning DHCP process with one segment happens first https://review.opendev.org/c/openstack/neutron/+/905617 | 09:27 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix dhcp cleaning stale devices process when enable action https://review.opendev.org/c/openstack/neutron/+/907250 | 09:27 |
ralonsoh | slaweq, hello! if you have a couple of mins, no rush | 09:32 |
ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/911105 | 09:32 |
slaweq | ralonsoh sure, just let me finish one thing and I will check it | 09:38 |
ralonsoh | for sure | 09:38 |
ralonsoh | ah, and https://review.opendev.org/c/openstack/neutron-lib/+/912273 | 09:38 |
ralonsoh | (related) | 09:38 |
ralonsoh | thanks a lot! | 09:39 |
slaweq | done | 09:49 |
sahid | o/ | 10:09 |
sahid | qucik question regarding dhcp agent, | 10:09 |
sahid | if an agent is down, the networks on that agent get rescheduled on the other automatically? | 10:10 |
sahid | I heard something like that but I did not find anything related in the code | 10:10 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: fix auto_scheduler_network understanding dhcp_agents_per_network https://review.opendev.org/c/openstack/neutron/+/910708 | 10:22 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP - Add ``OVNGatewayHAChassisGroup`` scheduler class https://review.opendev.org/c/openstack/neutron/+/872033 | 10:24 |
ralonsoh | sahid, if an DHCP agent is down, the DhcpAgentSchedulerDbMixin has a periodic checker | 10:26 |
ralonsoh | that will remove the NetworkDhcpAgentBinding associations between the network and the agent | 10:27 |
ralonsoh | and once that happens, the scheduler will re-schedule the network to another alive agent | 10:27 |
sahid | ralonsoh: oh that is really interesting, thanks a lot I will look more on DhcpAgentSchedulerDbMixin | 10:45 |
opendevreview | Merged openstack/neutron-specs master: Spec folder for 2024.2 cycle https://review.opendev.org/c/openstack/neutron-specs/+/914043 | 11:05 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas-dashboard stable/2024.1: Explicitly convert ``assertQuerysetEqual`` input parameters https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/914012 | 11:14 |
opendevreview | OpenStack 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/+/912890 | 11:15 |
opendevreview | OpenStack 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/+/912892 | 11:15 |
ralonsoh | lajoskatona, ^^ if you have 1 minute | 11:15 |
ralonsoh | thanks! | 11:15 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "subnet-external-network" extension to "subnet" resource https://review.opendev.org/c/openstack/neutron/+/907313 | 11:23 |
opendevreview | Merged openstack/neutron-lib master: Enable filtering flag in subnet 'router:external' field https://review.opendev.org/c/openstack/neutron-lib/+/912273 | 11:26 |
lajoskatona | ralonsoh: thanks | 11:44 |
lajoskatona | slaweq, ykarel: Hi, if you have a sec please check this from ralonsoh: https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/914012 | 11:45 |
slaweq | ralonsoh done | 11:49 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2024.1: Explicitly convert ``assertQuerysetEqual`` input parameters https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/914012 | 12:06 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/912890 | 12:06 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/912892 | 12:06 |
opendevreview | Merged openstack/neutron-tempest-plugin master: Check device_owner with actual availability zone https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/899714 | 12:39 |
fungi | ralonsoh: 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 asap | 13:19 |
opendevreview | Lajos 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/+/913888 | 13:34 |
ralonsoh | fungi, this is a security related patch | 14:12 |
ralonsoh | and the testing provided is almost null | 14:12 |
fungi | ralonsoh: yes, there is a follow-up change linked in the bug which adds testing | 14:12 |
ralonsoh | I saw it this morning | 14:13 |
ralonsoh | and Linux Bridge is failing | 14:13 |
fungi | ralonsoh: 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 |
fungi | so help with that is definitely appreciated | 14:16 |
ralonsoh | for sure, I'll check it today | 14:16 |
fungi | thanks! | 14:16 |
haleyb | ralonsoh: 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 master | 14:17 |
ralonsoh | yes | 14:17 |
ralonsoh | I'm checking the logs now | 14:17 |
ralonsoh | and I'm happy with the n-t-p patch | 14:17 |
ralonsoh | this is what I was expecting first | 14:17 |
ralonsoh | a feature/bug like this cannot be merged without the proper testing | 14:18 |
haleyb | ralonsoh: you're good with a 3-second sleep? | 14:18 |
ralonsoh | but now checking the n-t-p (in master only), it seems to work | 14:18 |
ralonsoh | hmmmmm I'm actually checking that right now | 14:18 |
ralonsoh | give me some mins | 14:18 |
haleyb | see my comments, no hurry | 14:18 |
ralonsoh | I agree with you: we have the active waits in the "self.check_remote_connectivity" and "ping_ip_address" calls | 14:20 |
ralonsoh | in 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 |
fungi | thanks again for the quick reviews! | 14:22 |
lajoskatona | haleyb, 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 |
ralonsoh | sure | 14:30 |
lajoskatona | haleyb, ralonsoh, slaweq: the patch from far related to https://review.opendev.org/c/openstack/neutron/+/905125 and https://launchpad.net/bugs/2048785 | 14:30 |
ralonsoh | lajoskatona, how are both patches related? | 14:32 |
ralonsoh | sorry, I don't see the link | 14:32 |
lajoskatona | ralonsoh: 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 issue | 14:37 |
ralonsoh | ahhh understood | 14:40 |
opendevreview | Merged openstack/neutron master: Clean up state VRRP PID file https://review.opendev.org/c/openstack/neutron/+/912047 | 16:12 |
kevko | Hi, i have this in our CI tempest system | 16:22 |
kevko | https://a6ab593594613a0a633c-3e445c1a5b8a8c84f71b5a61cff5a09b.ssl.cf2.rackcdn.com/913728/61/check/kolla-ansible-rocky9-multinode-ipv6/5c981d2/primary/logs/kolla/all-ERROR.txt | 16:22 |
kevko | https://a6ab593594613a0a633c-3e445c1a5b8a8c84f71b5a61cff5a09b.ssl.cf2.rackcdn.com/913728/61/check/kolla-ansible-rocky9-multinode-ipv6/5c981d2/primary/logs/tempest/reports/tempest-smoke.html << tempest report html | 16:23 |
kevko | is it know bug ? or ? | 16:23 |
lajoskatona | kevko: is an OVS and l3 agent based deployment am I right? | 16:44 |
opendevreview | Merged openstack/neutron-tempest-plugin master: Add extension "subnet-external-network" https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/911105 | 16:49 |
opendevreview | Brian Haley proposed openstack/neutron master: Update Neutron Liuetenants https://review.opendev.org/c/openstack/neutron/+/914146 | 18:28 |
*** jamesdenton_ is now known as jamesdenton | 18:58 | |
opendevreview | Anton 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/+/914154 | 19:06 |
opendevreview | Anton 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/+/914154 | 19:13 |
*** dasm is now known as Guest3915 | 20:22 | |
*** Guest3915 is now known as dasm | 20:30 | |
mlavalle | 4 | 20:59 |
haleyb | 6 | 21:02 |
opendevreview | Miguel Lavalle proposed openstack/neutron master: Check unspecified flavor in user defined driver https://review.opendev.org/c/openstack/neutron/+/914162 | 22:39 |
opendevreview | Miguel Lavalle proposed openstack/neutron master: Enable HA for OVN router flavors https://review.opendev.org/c/openstack/neutron/+/901513 | 23:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!