| opendevreview | yatin proposed openstack/neutron stable/xena: Add Chassis creation wait event in "TestAgentMonitor" https://review.opendev.org/c/openstack/neutron/+/823300 | 04:28 |
|---|---|---|
| opendevreview | yatin proposed openstack/neutron stable/wallaby: Add Chassis creation wait event in "TestAgentMonitor" https://review.opendev.org/c/openstack/neutron/+/823301 | 04:31 |
| opendevreview | yatin proposed openstack/neutron stable/xena: Recheck irrelevant files https://review.opendev.org/c/openstack/neutron/+/823302 | 04:41 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Accept OVS system-id as non UUID formatted string https://review.opendev.org/c/openstack/neutron/+/822667 | 06:36 |
| opendevreview | Federico Ressi proposed openstack/neutron stable/wallaby: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823333 | 07:33 |
| opendevreview | Merged openstack/neutron stable/xena: ovn: update ACL actions on stateful field change https://review.opendev.org/c/openstack/neutron/+/821440 | 07:33 |
| opendevreview | Federico Ressi proposed openstack/neutron stable/xena: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823334 | 07:40 |
| *** ykarel_ is now known as ykarel | 08:23 | |
| opendevreview | Merged openstack/neutron stable/victoria: [OVN] Accept OVS system-id as non UUID formatted string https://review.opendev.org/c/openstack/neutron/+/822669 | 09:07 |
| ralonsoh | slaweq, hi! | 09:25 |
| ralonsoh | can you check https://review.opendev.org/c/openstack/ovsdbapp/+/822916? thanks! (NOTE: this is stolen from the Neutron patch, solving the same issue) | 09:25 |
| opendevreview | Merged openstack/neutron stable/ussuri: [OVN] Accept OVS system-id as non UUID formatted string https://review.opendev.org/c/openstack/neutron/+/822691 | 09:38 |
| opendevreview | Merged openstack/neutron master: remove unicode from code https://review.opendev.org/c/openstack/neutron/+/822868 | 09:38 |
| slaweq | ralonsoh: hi. Done | 09:46 |
| ralonsoh | thanks | 09:46 |
| opendevreview | Merged openstack/ovsdbapp master: Move linters dependencies to tox.ini https://review.opendev.org/c/openstack/ovsdbapp/+/822916 | 09:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/neutron/+/822297 | 10:07 |
| opendevreview | Andre Aranha proposed openstack/neutron-tempest-plugin master: WIP - Update usage of tempest ssh client https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/823371 | 10:14 |
| *** arnau__ is now known as averdaguer | 10:18 | |
| opendevreview | Federico Ressi proposed openstack/neutron stable/wallaby: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823333 | 10:25 |
| opendevreview | Federico Ressi proposed openstack/neutron stable/xena: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823372 | 10:29 |
| opendevreview | Federico Ressi proposed openstack/neutron stable/wallaby: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823374 | 10:33 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Use "api.lookup" to search for a "Logical_Switch" https://review.opendev.org/c/openstack/ovsdbapp/+/822909 | 10:38 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Update QoS register in "QoSAddCommand" if exists https://review.opendev.org/c/openstack/ovsdbapp/+/822138 | 10:39 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: DNM Revert "Use Port_Binding up column to set Neutron port status" https://review.opendev.org/c/openstack/neutron/+/823355 | 10:40 |
| opendevreview | Andre Aranha proposed openstack/neutron-tempest-plugin master: WIP - Update usage of tempest ssh client https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/823371 | 11:10 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/xena: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823377 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Improve DHCP RPC handler https://review.opendev.org/c/openstack/neutron/+/820190 | 11:34 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: DNM: test master https://review.opendev.org/c/openstack/networking-sfc/+/812491 | 12:28 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: DNM: test master https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/806177 | 12:28 |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: DNM: Check master status https://review.opendev.org/c/openstack/networking-odl/+/806095 | 12:29 |
| frickler | lajoskatona: stable releases for n-d-r could use PTR review https://review.opendev.org/c/openstack/releases/+/822257 | 12:32 |
| lajoskatona | frickler: Hi, thanks for headsup | 12:50 |
| opendevreview | Merged openstack/neutron stable/wallaby: Add Chassis creation wait event in "TestAgentMonitor" https://review.opendev.org/c/openstack/neutron/+/823301 | 13:05 |
| *** beagles is now known as beagles-afk | 13:06 | |
| Guest9120 | Is it possible to route a floating ip to an instance rather than using NAT? | 13:31 |
| *** Guest9120 is now known as marlinc | 13:32 | |
| lajoskatona | slaweq, ralonsoh, bcafarel, rubasov, haleyb, amotoki: Hi, I will be a little late today from the meeting, I hope only a few minutes (I have to fetch my son from the school) | 13:41 |
| ralonsoh | sure, no problem | 13:41 |
| slaweq | sure, thx for the heads up | 13:41 |
| ralonsoh | averdaguer, about https://review.opendev.org/q/I206c31201470f178efdde8839622be7900c6ae3e | 13:43 |
| ralonsoh | push patches up to Train | 13:43 |
| ralonsoh | (n-ovn in Train, of course) | 13:43 |
| averdaguer | Yes, on it, I had a conflict from Wallaby to train that I'm fixing :) ralonsoh | 13:43 |
| ralonsoh | ah ok | 13:44 |
| ralonsoh | cool | 13:44 |
| averdaguer | Thanks! | 13:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Implement floating IP network QoS inheritance https://review.opendev.org/c/openstack/neutron/+/817790 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: "L3AgentExtensionsManager" check loaded extensions https://review.opendev.org/c/openstack/neutron/+/818540 | 13:50 |
| *** ykarel__ is now known as ykarel | 13:57 | |
| ykarel | ralonsoh, hi | 13:57 |
| ykarel | when you get chance please have a look at https://review.opendev.org/c/openstack/neutron/+/822843 | 13:58 |
| ralonsoh | sure | 13:58 |
| mlavalle | lajoskatona: are we meeting today? | 14:03 |
| lajoskatona | mlavalle: Hi, just arrived back, sorry | 14:04 |
| lajoskatona | #startmeeting networking | 14:04 |
| opendevmeet | Meeting started Tue Jan 4 14:04:09 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:04 |
| opendevmeet | The meeting name has been set to 'networking' | 14:04 |
| mlavalle | o/ | 14:04 |
| lajoskatona | o/ | 14:04 |
| rubasov | o/ | 14:04 |
| lajoskatona | Welcome back everybody, and happy 2022 :-) | 14:04 |
| mlavalle | Likewise :-) | 14:04 |
| slaweq | o/ | 14:04 |
| bcafarel | o/, happy new year! | 14:05 |
| lajoskatona | #topic Announcements | 14:06 |
| lajoskatona | The usual Yoga cycle calendar https://releases.openstack.org/yoga/schedule.html | 14:06 |
| lajoskatona | and the release countdown mail: http://lists.openstack.org/pipermail/openstack-discuss/2021-December/026479.html | 14:06 |
| ralonsoh | hi (sorry for the delay) | 14:07 |
| lajoskatona | as this is Yoga-2 milestone, we have to cut new release of our libraries | 14:07 |
| lajoskatona | so be aware (I have to check them too) | 14:08 |
| slaweq | ahh, ok | 14:08 |
| slaweq | sorry, wrong window :) | 14:08 |
| lajoskatona | :-) | 14:08 |
| mlavalle | lol | 14:08 |
| lajoskatona | if no questions/comments for the milestone, we can go on | 14:09 |
| lajoskatona | #topic Bugs | 14:10 |
| lajoskatona | The last deputyies were: | 14:10 |
| lajoskatona | mlavalle: http://lists.openstack.org/pipermail/openstack-discuss/2021-December/026396.html | 14:10 |
| lajoskatona | rubasov: http://lists.openstack.org/pipermail/openstack-discuss/2021-December/026456.html | 14:10 |
| lajoskatona | and ralonsoh for last week | 14:11 |
| ralonsoh | this week you mean? | 14:11 |
| lajoskatona | ralonsoh: In the table (https://wiki.openstack.org/wiki/Network/Meetings#Bug_deputy) last week | 14:12 |
| lajoskatona | from 27. December | 14:12 |
| ralonsoh | ok I was on PTO | 14:13 |
| ralonsoh | and I wasn't aware | 14:13 |
| ralonsoh | I'll make a mail today | 14:13 |
| lajoskatona | sorry, I missed my duty to send out mail, as I was also on PTO :-) | 14:13 |
| slaweq | LOL, raise TooManyPtoException() | 14:14 |
| lajoskatona | :-) | 14:15 |
| mlavalle | that's why I always demand a personalized bugs deputy email the week befor | 14:15 |
| mlavalle | my duty | 14:15 |
| mlavalle | and it has to be from the PTL | 14:15 |
| lajoskatona | I checked and I see these unassigned bugs: | 14:16 |
| lajoskatona | #link https://bugs.launchpad.net/neutron/+bug/1954942 | 14:16 |
| lajoskatona | #link https://bugs.launchpad.net/neutron/+bug/1955008 | 14:17 |
| lajoskatona | these are fullstack and functional failures | 14:17 |
| lajoskatona | #link https://bugs.launchpad.net/neutron/+bug/1955478 | 14:18 |
| lajoskatona | this one I saw in CI meeting log as well (metadata connection issue) | 14:18 |
| lajoskatona | #link https://bugs.launchpad.net/neutron/+bug/1955639 | 14:18 |
| lajoskatona | #link https://bugs.launchpad.net/neutron/+bug/1955765 | 14:19 |
| slaweq | That last one I was investigating a bit | 14:19 |
| slaweq | but I don't really know what the problem can be there | 14:19 |
| lajoskatona | thanks for checking it (notes in the CI meeting etherpad: https://etherpad.opendev.org/p/neutron-ci-meetings#L18 ) | 14:20 |
| lajoskatona | and a last unassigned: #link https://bugs.launchpad.net/neutron/+bug/1955765 | 14:20 |
| rubasov | about 1955765 we may also ask if we are supporting the newer devstack plugin or not (not that the legacy one is un-deprecated) | 14:21 |
| lajoskatona | This one is related to the devstack changes which moved code from neutron devstack plugin to core devstack | 14:21 |
| rubasov | s/not that/now that/ | 14:22 |
| lajoskatona | rubasov: thanks, that can be really straightforward to be decided | 14:22 |
| lajoskatona | as the legacy is the used and a lot of things are not working in the new one (like OVN) | 14:22 |
| slaweq | I think we discussed that back in Shanghai and we decided to "undeprecate" neutron-legacy | 14:23 |
| slaweq | as new one wasn't never really finished | 14:23 |
| slaweq | but since then still nobody wanted/had time to do it | 14:23 |
| rubasov | slaweq: yes, but do we want to now support both the legacy and the new? | 14:24 |
| slaweq | rubasov: I don't think we should keep both | 14:24 |
| mlavalle | +1 | 14:24 |
| slaweq | if the "legacy" one is ok and works for us, I would personally stay with that one | 14:24 |
| rubasov | afaiu this report is about some breakage in the new | 14:24 |
| ralonsoh | right, we should keep one option | 14:24 |
| rubasov | I recommended the reporter that he switches back to the legacy | 14:25 |
| ralonsoh | right | 14:25 |
| rubasov | but that was kind of unofficial, just my opinion, not the neutron team's :-) | 14:25 |
| lajoskatona | the "new" plugin code is in devstack, am I right? | 14:26 |
| rubasov | I think, yes | 14:27 |
| rubasov | devstack/lib/neutron | 14:27 |
| lajoskatona | I don't know if there's any deprecation process for devstack, or how to handle if we get rid of the devstack/lib/neutron totally | 14:28 |
| lajoskatona | but I can ask to be sure | 14:29 |
| rubasov | deprecation is probably user-friendlier | 14:29 |
| lajoskatona | sure | 14:29 |
| lajoskatona | Ok, I will check it with QA team, how to deprecate it, or what should be the best way forward | 14:30 |
| lajoskatona | If there's no more thoughts for the bugs we can move forward | 14:31 |
| lajoskatona | One more sentence for the bugs: | 14:32 |
| lajoskatona | This week lucasgomes is the deputy, and next week jlibosva will be. | 14:32 |
| lajoskatona | I sent a mail to lucasgomes to be sure :-) | 14:32 |
| ralonsoh | they are aware | 14:32 |
| lajoskatona | ralonsoh: thanks | 14:33 |
| lajoskatona | I checked the meeting agenda, and no more topics from as I see, as liuyulong didn't add topics for L3, and there is no change in ryu | 14:34 |
| lajoskatona | If you have anything to discuss, please do it now | 14:34 |
| lajoskatona | #topic On Demand Agenda | 14:34 |
| mlavalle | I just want to mention that I scheduled a topic for the drivers meeting on friday, about bringinh back FWaaS. You probably saw the email exchange | 14:35 |
| slaweq | ++ | 14:36 |
| mlavalle | a team in China is interested in maintaining it and devlop the it for ovn | 14:36 |
| lajoskatona | mlavalle: thanks, good that you mention it | 14:37 |
| mlavalle | they plan to attend the meeting this coming Friday | 14:37 |
| lajoskatona | It's a little risky if I can participate, as I bring my sons for vaccination, and not sure if the gods of timing and queues are on my side.... | 14:38 |
| mlavalle | should we re-schedule for next week's Friday? | 14:39 |
| mlavalle | I think you should attend | 14:39 |
| slaweq | this friday I will be off generally but I was going to attend the meeting | 14:40 |
| lajoskatona | yeah perhaps that would be better | 14:40 |
| slaweq | if it would be moved to next week, better for me :) | 14:40 |
| mlavalle | ok, I'll send them an email in abit | 14:40 |
| lajoskatona | mlavalle: thanks | 14:40 |
| mlavalle | :-) | 14:41 |
| lajoskatona | if no more topics we can close the meeting | 14:42 |
| *** beagles-afk is now known as beagles | 14:43 | |
| slaweq | thx, and see You in 18 minutes in the CI meeting :) | 14:43 |
| bcafarel | :) | 14:43 |
| ralonsoh | bye | 14:43 |
| mlavalle | see you in a bit | 14:43 |
| lajoskatona | #endmeeting | 14:43 |
| opendevmeet | Meeting ended Tue Jan 4 14:43:49 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:43 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2022/networking.2022-01-04-14.04.html | 14:43 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2022/networking.2022-01-04-14.04.txt | 14:43 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2022/networking.2022-01-04-14.04.log.html | 14:43 |
| lajoskatona | bye | 14:43 |
| slaweq | o/ | 14:43 |
| fungi | slaweq: i replied on the ml too, but you may want to switch to meet.jit.si this week, meetpad is still offline because we're uncomfortable with the state of the current release as relates to the recent log4j vulnerabilities | 14:44 |
| fungi | their next release will drop log4j entirely, but they haven't tagged it just yet | 14:44 |
| slaweq | fungi: ok, thx for info | 14:50 |
| fungi | and sorry for the inconvenience, we're trying to be a bit conservative there in order to better manage our workload | 14:52 |
| fungi | we already brought the service up once and took it back down again as subsequent log4j vulnerabilities came to light | 14:53 |
| bcafarel | fungi++ that makes total sense with that log4j mess | 14:53 |
| slaweq | fungi: sure, thx for the heads up | 14:54 |
| fungi | yeah, they updated to log4j 2.15 in the last release, and while they seem confident the fixes in 2.16 and 2.17 aren't relevant for jitsi-meet, we're not so confident in their analysis (especially given they opted to remove/replace log4j in development rather than update it again) | 14:55 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/victoria: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823401 | 14:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: DVR: Populate ARP entries of the allowed_address_pairs to the routers https://review.opendev.org/c/openstack/neutron/+/601336 | 14:57 |
| fungi | on a related note, i started a thread on openstack-discuss about log4j vulnerabilities relative to openstack, one of the points of concern is for folks using the networking-odl driver as they need to make sure they upgrade opendaylight | 14:58 |
| fungi | if anyone knows of similar situations in openstack deployments, please reply on that thread so i can build up a list | 14:58 |
| slaweq | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Jan 4 15:00:12 2022 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| slaweq | welcome everyone in the 2022 on the Neutron CI meeting | 15:00 |
| slaweq | this week we have meeting on the video also | 15:00 |
| mlavalle | slaweq: having trouble connecting to the video | 15:00 |
| slaweq | but as fungi mentioned above, we changed server, new url is https://meet.jit.si/neutron-ci-meetings | 15:00 |
| mlavalle | ah, ok | 15:01 |
| slaweq | that's temporary only for this meeting (I hope) | 15:01 |
| fungi | yeah, i hope we have meetpad back up later this week | 15:01 |
| lajoskatona | Hi | 15:01 |
| slaweq | in the meantime | 15:01 |
| ralonsoh | hi | 15:01 |
| slaweq | Grafana dashboard: http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:01 |
| bcafarel | hi o/ | 15:03 |
| slaweq | ralonsoh: video call on https://meet.jit.si/neutron-ci-meetings | 15:03 |
| slaweq | #topic Actions from previous meetings | 15:04 |
| slaweq | slaweq to check https://bugs.launchpad.net/neutron/+bug/1955478 and broken metadata connectivity issues | 15:04 |
| ykarel | o/ | 15:04 |
| slaweq | ykarel: we have video call https://meet.jit.si/neutron-ci-meetings | 15:05 |
| slaweq | if You want to join | 15:05 |
| ykarel | slaweq, yes trying to connect there | 15:05 |
| ykarel | but seems some issue | 15:05 |
| ykarel | will try with different browser | 15:06 |
| ykarel | chrome worked | 15:06 |
| lajoskatona | log example for metadata: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_141/818844/2/check/neutron-tempest-plugin-scenario-openvswitch/1415d28/testr_results.html | 15:07 |
| mlavalle | https://bugs.launchpad.net/neutron/+bug/1956116 | 15:11 |
| mlavalle | https://review.opendev.org/c/openstack/devstack/+/823218 | 15:11 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/wallaby: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823402 | 15:12 |
| slaweq | frickler: gmann hi, can You check https://review.opendev.org/c/openstack/devstack/+/823218 when You will have few minutes? | 15:12 |
| slaweq | thx in advance | 15:12 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/victoria: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823401 | 15:14 |
| slaweq | #topic Stable branches | 15:14 |
| gmann | slaweq: ack | 15:15 |
| slaweq | #topic Grafana | 15:19 |
| slaweq | http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:19 |
| slaweq | #topic fullstack/functional | 15:20 |
| slaweq | https://bugs.launchpad.net/neutron/+bug/1956344 | 15:21 |
| slaweq | #action ralonsoh to check bug https://bugs.launchpad.net/neutron/+bug/1956344 | 15:23 |
| slaweq | https://71d2302875cffcacbcb7-bd54a9781d6bc663ca8af93b25749dfd.ssl.cf5.rackcdn.com/823300/1/gate/neutron-functional-with-uwsgi/1938908/testr_results.html | 15:24 |
| slaweq | #action slaweq to check failure https://71d2302875cffcacbcb7-bd54a9781d6bc663ca8af93b25749dfd.ssl.cf5.rackcdn.com/823300/1/gate/neutron-functional-with-uwsgi/1938908/testr_results.html | 15:25 |
| slaweq | #topic Tempest/Scenario | 15:26 |
| slaweq | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_918/820190/5/check/neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid/91804c0/testr_results.html | 15:26 |
| slaweq | https://zuul.opendev.org/t/openstack/build/3a378c37b15c4dfabc78e1f1c7016443 | 15:29 |
| slaweq | https://zuul.opendev.org/t/openstack/build/9909342e605e420da5a9cc949cedfb01 | 15:29 |
| lajoskatona | https://pypi.org/project/paramiko/#history | 15:31 |
| slaweq | https://1c04ba818e60ceed6270-76fe6537709af1be557ea31f3d630d58.ssl.cf2.rackcdn.com/815962/6/check/neutron-tempest-plugin-scenario-openvswitch/024a910/testr_results.html | 15:31 |
| ykarel | https://review.opendev.org/c/openstack/requirements/+/823128 was the fix for paramiko issue | 15:32 |
| ykarel | and https://review.opendev.org/c/openstack/devstack/+/823127 to avoid such breakages | 15:32 |
| slaweq | #topic On Demand | 15:32 |
| slaweq | #link https://etherpad.opendev.org/p/neutron-ci-improvements | 15:33 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/neutron master: Extend database to support portforwardings with port range https://review.opendev.org/c/openstack/neutron/+/798961 | 15:36 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/victoria: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823401 | 15:37 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/wallaby: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823402 | 15:38 |
| slaweq | #endmeeting | 15:43 |
| opendevmeet | Meeting ended Tue Jan 4 15:43:42 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:43 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-01-04-15.00.html | 15:43 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-01-04-15.00.txt | 15:43 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2022/neutron_ci.2022-01-04-15.00.log.html | 15:43 |
| ykarel | ralonsoh, Thanks for https://review.opendev.org/c/openstack/neutron/+/822843/comment/5441b589_834cb8b0/ | 15:44 |
| ykarel | not sure i get that | 15:44 |
| ykarel | you mean can't use ovn/dns_servers in case of ipv6? | 15:44 |
| ralonsoh | ykarel, the logic implemented there depends on the ip version | 15:44 |
| ralonsoh | dns_servers = (subnet.get('dns_nameservers') or | 15:44 |
| ralonsoh | ovn_conf.get_dns_servers() or | 15:44 |
| ralonsoh | get_system_dns_resolvers()) | 15:44 |
| ralonsoh | for ipv4 | 15:44 |
| ralonsoh | and only this for ipv6 | 15:44 |
| ralonsoh | dns_servers = subnet['dns_nameservers'] | 15:44 |
| ralonsoh | so retrieve the dns servers from the subnet | 15:44 |
| ykarel | ralonsoh, yes i changed ^ to be common | 15:44 |
| ralonsoh | that's what is wrong | 15:44 |
| ykarel | same for both ipv4/ipv6 | 15:45 |
| ralonsoh | why? | 15:45 |
| ralonsoh | you are changing the current logic | 15:45 |
| ykarel | i couldn't find any reason why ovn conf and system nameservers can't be used for ipv6 subnets | 15:45 |
| ykarel | i think i added that in commit message | 15:45 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/ussuri: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/neutron/+/823407 | 15:46 |
| ykarel | ralonsoh, so is it totally incorrect to go that way or just it needs to be done in seperate patch | 15:48 |
| ykarel | i tried to trace through networking-ovn code base when it was added, and thought of fixing it together | 15:49 |
| ralonsoh | ykarel, if you are going this way | 15:49 |
| ralonsoh | 1) you should add a release note | 15:49 |
| ralonsoh | 2) the filtering should be done before | 15:49 |
| ralonsoh | that means first you read the dns nameservers from the subnet and filter by IP version | 15:50 |
| ykarel | ralonsoh, subnet nameservers can't be of different versions | 15:50 |
| ykarel | as api block that | 15:50 |
| ralonsoh | and why your patch then? | 15:50 |
| ykarel | only conf or system nameservers need to be filtered | 15:50 |
| ykarel | for conf and system resolv conf | 15:51 |
| ykarel | actually the bug was filed for the case when conf is set ipv6 address | 15:51 |
| ykarel | dhcpv4 was getting updated | 15:51 |
| ralonsoh | but then you need to check first if the configured dns server has the correct version and apply it | 15:52 |
| ykarel | and leading to wrong nameservers in vms | 15:52 |
| ralonsoh | if not, use the system dns resolver, if the version is correct | 15:52 |
| ralonsoh | how was that possible? | 15:52 |
| ralonsoh | in a ipv6 subnet? | 15:52 |
| ykarel | in ipv4 subnet | 15:52 |
| ralonsoh | so what you need is to go step by step | 15:53 |
| ralonsoh | read the subnet nameservers | 15:53 |
| ralonsoh | if not, the configured nameserver | 15:53 |
| ralonsoh | if not, the system resolver | 15:54 |
| ykarel | ok i think i got what you mean now | 15:54 |
| ykarel | as ovn_conf.get_dns_servers() can return wrong nameserver based on version and in that case system resolvers would not be checked | 15:55 |
| ralonsoh | yes | 15:56 |
| ykarel | ok Thanks ralonsoh will update accordingly. | 15:56 |
| ykarel | but tomorrow | 15:56 |
| ykarel | ralonsoh, release NOTE needed for this case? | 15:56 |
| ralonsoh | this is changing the ipv6 behaviour | 15:57 |
| ykarel | ok for that case, yes okk will add | 15:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add new API tests for active field in the allowed_address_pairs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/795358 | 16:00 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add new scenario test for VIP address added as allowed addr pair https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/794841 | 16:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Adopt rehomed QoS FIP extension from neutron-lib 2.18.0 https://review.opendev.org/c/openstack/neutron/+/821838 | 16:20 |
| opendevreview | Arnau Verdaguer proposed openstack/networking-ovn stable/train: [OVN] Prevent deleting the only IP of a router port https://review.opendev.org/c/openstack/networking-ovn/+/823409 | 16:38 |
| opendevreview | David Hill proposed openstack/neutron stable/train: Reduce log level in "ensure_device_is_ready" https://review.opendev.org/c/openstack/neutron/+/823360 | 16:58 |
| opendevreview | David Hill proposed openstack/neutron stable/victoria: Reduce log level in "ensure_device_is_ready" https://review.opendev.org/c/openstack/neutron/+/823361 | 16:59 |
| opendevreview | David Hill proposed openstack/neutron stable/ussuri: Reduce log level in "ensure_device_is_ready" https://review.opendev.org/c/openstack/neutron/+/823362 | 16:59 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: [WIP] Ensure subports transition to DOWN https://review.opendev.org/c/openstack/neutron/+/823412 | 17:23 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: [WIP] Ensure subports transition to DOWN https://review.opendev.org/c/openstack/neutron/+/823412 | 17:24 |
| opendevreview | Merged openstack/ovsdbapp master: Remove ovsdb_connection singleton for tests https://review.opendev.org/c/openstack/ovsdbapp/+/820398 | 18:27 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/neutron master: Extend database to support portforwardings with port range https://review.opendev.org/c/openstack/neutron/+/798961 | 18:34 |
| opendevreview | Merged openstack/neutron stable/xena: Change tobiko CI job in the periodic queue https://review.opendev.org/c/openstack/neutron/+/823372 | 18:49 |
| opendevreview | Merged openstack/neutron master: Add wait event for metadataagent sb_idl https://review.opendev.org/c/openstack/neutron/+/821927 | 18:50 |
| opendevreview | Merged openstack/neutron stable/xena: Add Chassis creation wait event in "TestAgentMonitor" https://review.opendev.org/c/openstack/neutron/+/823300 | 18:50 |
| *** averdaguer is now known as averdaguer|away | 18:58 | |
| opendevreview | Lajos Katona proposed openstack/networking-odl master: Fix some requirements https://review.opendev.org/c/openstack/networking-odl/+/823428 | 19:16 |
| opendevreview | Terry Wilson proposed openstack/neutron master: WIP Use neutron db for ovn agents https://review.opendev.org/c/openstack/neutron/+/818850 | 20:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: DVR: Populate ARP entries of the allowed_address_pairs to the routers https://review.opendev.org/c/openstack/neutron/+/601336 | 20:17 |
| opendevreview | Merged openstack/ovsdbapp master: Allow functional tests to pass on older OVN w/o IC https://review.opendev.org/c/openstack/ovsdbapp/+/820275 | 20:34 |
| rm_work | I'm super confused today about how the API routing works... I'm running tests and expecting to hit this delete method: https://github.com/openstack/neutron/blob/master/neutron/api/v2/base.py#L565-L573 | 22:16 |
| rm_work | but instead it only hits this: https://github.com/openstack/neutron/blob/master/neutron/pecan_wsgi/controllers/resource.py#L76-L85 | 22:16 |
| rm_work | there's ... two distinct paths somehow? are the base.py methods just not used anymore? is that code dead/unreachable? | 22:17 |
| rm_work | or is this a quirk of the way the test-runner starts pecan_wsgi? | 22:17 |
| rm_work | I don't understand what's going on at all | 22:18 |
| opendevreview | Terry Wilson proposed openstack/neutron master: WIP Use neutron db for ovn agents https://review.opendev.org/c/openstack/neutron/+/818850 | 22:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!