opendevreview | yatin proposed openstack/neutron stable/2024.1: [2024.1 only] Switch to 2024.1 neutron-tempest-plugin jobs https://review.opendev.org/c/openstack/neutron/+/925159 | 04:53 |
---|---|---|
ralonsoh | slaweq, ^^^ this patch is fixing 2024.1 (the tempest tests) | 05:35 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Initialize the policy enforcer for the "tagging" service plugin https://review.opendev.org/c/openstack/neutron/+/925077 | 05:36 |
opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 06:18 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WSGI] Move all OVN jobs to use WSGI API module https://review.opendev.org/c/openstack/neutron/+/924317 | 07:13 |
ykarel | slaweq, when around please check https://review.opendev.org/c/openstack/neutron/+/925159 | 07:57 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Do not release the executor inside ``_check_child_processes`` https://review.opendev.org/c/openstack/neutron/+/925174 | 08:45 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Do not release the executor inside ``_check_child_processes`` https://review.opendev.org/c/openstack/neutron/+/925175 | 08:45 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Do not release the executor inside ``_check_child_processes`` https://review.opendev.org/c/openstack/neutron/+/925176 | 08:46 |
opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 08:54 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the RBAC actions field for "network" and "subnet" https://review.opendev.org/c/openstack/neutron/+/915370 | 09:23 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.2: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/925178 | 09:35 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.1: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/925179 | 09:39 |
opendevreview | yatin proposed openstack/neutron master: [ovs-master jobs] Move OVS_BRANCH to branch-3.4 https://review.opendev.org/c/openstack/neutron/+/925184 | 10:53 |
opendevreview | yatin proposed openstack/neutron master: [DNM] check ovn/ovs source job ovs_branch=3.4 https://review.opendev.org/c/openstack/neutron/+/925187 | 11:06 |
ralonsoh | ykarel, https://review.opendev.org/c/openstack/neutron/+/925091 | 11:30 |
ralonsoh | that patch was proposed yesterday | 11:30 |
ykarel | ralonsoh, thx i missed that, let's take that in , will abandon mine | 11:31 |
ralonsoh | but the dnm is not passing | 11:31 |
ralonsoh | https://review.opendev.org/c/openstack/neutron/+/925187?tab=change-view-tab-header-zuul-results-summary | 11:31 |
ralonsoh | so there is something else that is needed | 11:32 |
ykarel | yes looking into that now | 11:33 |
ykarel | seems change is not picked | 11:34 |
ykarel | as seeing 2024-07-30 11:11:11.902321 | controller | +++ OVS_BRANCH=branch-3.3 | 11:34 |
opendevreview | yatin proposed openstack/neutron master: [DNM] check ovn/ovs source job ovs_branch=3.4 https://review.opendev.org/c/openstack/neutron/+/925187 | 11:38 |
ralonsoh | ykarel, qq, if you know | 11:43 |
ralonsoh | have you seen this error before: https://paste.opendev.org/show/bwsUPO7SEpDxoUXGpjAB/ | 11:43 |
ralonsoh | I'm having this error in the ML2/OVN jobs with WSGI (actually there is one only, in the experimental queue) | 11:43 |
ralonsoh | and started to fail 7 days ago | 11:43 |
ralonsoh | I thought it was because of the RPC stuff in devstack, but it was working fine (with the RPC worker) before that | 11:44 |
ykarel | ralonsoh, nope i don't recall seeing those errors recently apart from the bug you recently reported | 11:45 |
ralonsoh | I think this is because the IPv6 address but I need to justify it | 11:45 |
ykarel | yes likely | 11:47 |
ykarel | wrt change not picked for ovs-master job, it's because stable/2024.1 variant being used in the job, but with above dnm it should work. and we could drop those variants from stable branches as relevant for master only | 11:48 |
ralonsoh | ykarel, are you talking about https://review.opendev.org/c/openstack/neutron/+/925184? | 11:49 |
ralonsoh | I don't understand why the DNM patch PS1 was not working | 11:49 |
ralonsoh | with PS2 is using the correct ovs branch | 11:49 |
ykarel | yes talking about dnm patch | 11:49 |
ykarel | above is the reason for not working in PS1 | 11:49 |
ralonsoh | sorry, I still don't get it | 11:50 |
ralonsoh | you are testing a master job, neutron-ovn-tempest-ipv6-only-ovs-master | 11:50 |
ykarel | yes let me share reference with that it will be more clear | 11:53 |
ralonsoh | thanks! | 11:53 |
ykarel | see https://4cdf01f2d535f547e363-277fe61cde1e0fafc7c227e50deabdc0.ssl.cf1.rackcdn.com/925187/1/check/neutron-ovn-tempest-ipv6-only-ovs-master/0a82387/zuul-info/inventory.yaml | 11:54 |
ykarel | specifically _inheritance_path: | 11:54 |
ralonsoh | no way... | 11:54 |
ykarel | there it matches source: openstack/neutron/zuul.d/tempest-singlenode.yaml@stable/2024.1#773>' | 11:54 |
ralonsoh | right | 11:54 |
ralonsoh | what I don't understand is how a inherited job can affect the parent too | 11:55 |
ralonsoh | well, the stable branch one | 11:55 |
ykarel | zuul pick jobs across branches, and we have branches: master set on these jobs | 11:56 |
ykarel | i don't know how it prioritizes one over the other though | 11:56 |
ykarel | may be frickler or fungi can tell that | 11:57 |
ralonsoh | pffff "the ways of zuul are unsearchable" | 11:57 |
ralonsoh | I didn't know that but that's very good to know it | 11:57 |
opendevreview | Merged openstack/neutron stable/2024.1: [2024.1 only] Switch to 2024.1 neutron-tempest-plugin jobs https://review.opendev.org/c/openstack/neutron/+/925159 | 12:00 |
frickler | ykarel: ralonsoh: iiuc the "branches: ^master$" will only affect where the job will be run, not where the job definition is getting taken from. so I think you'll want to really delete all the *-master job definitions from all stable branches and repeat that process when new branches are being made | 12:04 |
ralonsoh | ahhh but -master in this case refers to ovs-master | 12:05 |
opendevreview | yatin proposed openstack/neutron stable/2024.1: [stable/2024.1 only] Drop -master jobs https://review.opendev.org/c/openstack/neutron/+/925193 | 12:05 |
ralonsoh | no, I said nothing | 12:05 |
ykarel | frickler, yeap just pushed ^, the part that is not clear is the same job appears in both master and stable/2024.1, when the job is triggered it's picking the job from stable/2024.1 not the the one from master, how this prioritizes one over the other? | 12:07 |
ykarel | so with the above patch we can avoid this to happen atleast | 12:07 |
ralonsoh | we were using the suffix -master to refer to other libraries, like sqlalchemy or n-lib | 12:08 |
ralonsoh | but I think these tests don't make sense in stable branches | 12:08 |
frickler | ykarel: looking at that inheritance_path, the job definitions seem to get stacked, I'm not sure if zuul makes any guarantee on the ordering of that stack, might be random or alphabetic | 12:12 |
ralonsoh | bcafarel, please check https://review.opendev.org/c/openstack/neutron/+/925193 | 12:15 |
bcafarel | ralonsoh: sure, I thought I had checked this one earlier, probably missed the button | 12:23 |
bcafarel | ah no that was the Switch to 2024.1 neutron-tempest-plugin jobs (same kind of new stable fixes) | 12:24 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once https://review.opendev.org/c/openstack/neutron/+/925194 | 12:30 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use has_lock_periodic decorator for the tasks which require ovn db lock https://review.opendev.org/c/openstack/neutron/+/925197 | 12:53 |
ykarel | frickler, thx for checking | 13:20 |
ykarel | ralonsoh, bcafarel when you get chance check https://review.opendev.org/c/openstack/neutron/+/925184 too | 13:20 |
ralonsoh | ykarel, done but then I'll ask the other folk to abandon his patch | 13:21 |
ykarel | ah sorry | 13:21 |
ykarel | let's take other one then i missed that again | 13:21 |
ykarel | as that was proposed first | 13:21 |
ralonsoh | cool | 13:21 |
bcafarel | which one should we review then? | 13:29 |
ralonsoh | bcafarel, https://review.opendev.org/c/openstack/neutron/+/925091 | 13:32 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-api-ovn-wsgi https://review.opendev.org/c/openstack/neutron/+/925205 | 13:34 |
haleyb | #startmeeting networking | 14:00 |
opendevmeet | Meeting started Tue Jul 30 14:00:17 2024 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'networking' | 14:00 |
haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh | 14:00 |
mlavalle | \o | 14:00 |
elvira | o/ | 14:00 |
ykarel | o/ | 14:00 |
ralonsoh | hi | 14:00 |
bcafarel | o/ | 14:00 |
slaweq | o/ | 14:01 |
haleyb | hi everyone | 14:01 |
haleyb | #topic announcements | 14:01 |
haleyb | We are now in Dalmatian release week (R - 9) | 14:01 |
rubasov | o/ | 14:01 |
haleyb | Non-client library freeze: August 22nd, 2024 (R-6 week) | 14:01 |
haleyb | Client library freeze: August 29th, 2024 (R-5 week) | 14:01 |
haleyb | Dalmatian-3 milestone: August 29th, 2024 (R-5 week) | 14:02 |
haleyb | so we are pretty close to the end of D | 14:02 |
haleyb | hopefully we have a happy gate so we can merge code :-) | 14:03 |
haleyb | Reminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:03 |
haleyb | anything crazy happen last week? | 14:04 |
ralonsoh | I think most of the CI problems are addressed | 14:04 |
mlavalle | we didn't wreck the place \o/ | 14:04 |
haleyb | good to know :) | 14:05 |
haleyb | i am still digging out from a backlog, so feel free to ping me on irc for anything important | 14:05 |
obondarev | o/ | 14:05 |
haleyb | #topic bugs | 14:06 |
haleyb | bcafarel was the deputy last week, here is a link to his report | 14:06 |
haleyb | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/NZKXVL2DHNFLKEEXETJ2JZYMQM5DALCZ/ | 14:06 |
haleyb | there are a few without assignee | 14:07 |
bcafarel | on the 3 unassigned bugs, I see you checked the low-haning fruit one, also thanks ralonsoh for checking first one | 14:07 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2074207 | 14:07 |
haleyb | DHCP agent makes endless attempts to configure a network with MTU < 1280 | 14:07 |
haleyb | i thought this was related to a previous bug, but it's slightly different | 14:07 |
haleyb | ralonsoh: just noticing you added a comment | 14:08 |
ralonsoh | I think it makes sense not to add the ipv6 metadata address if there are no ipv6 subnets | 14:08 |
ralonsoh | in any case, if you have ipv6 enabled in your env, you should not used smaller mtus | 14:08 |
ralonsoh | but don't fail because of us | 14:08 |
haleyb | ralonsoh: ack, i couldn't tell whether you thought it was invalid, but i agree we could fix this with a check | 14:09 |
haleyb | i can take it if noone else wants it as i worked on the last one | 14:09 |
haleyb | i don't know why people use small mtus like that, but... | 14:10 |
haleyb | next bug is | 14:11 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2073872 | 14:11 |
haleyb | Neutron L3 agent not create ECMP in HA router but in single mode is ok | 14:11 |
haleyb | looks like l3-ha differs from "regular" l3 in adding/deleting routes | 14:12 |
slaweq | I can try to maybe reproduce it and look quickly if I will have some time, but not promissing anything for now | 14:13 |
haleyb | this is reminding me of a previous bug where we had issues with overlapping routes, in this case it's just the gateway | 14:13 |
haleyb | slaweq: thanks, i'll watch it | 14:14 |
haleyb | last unassigned is | 14:14 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2074056 | 14:14 |
haleyb | Invalid documented security group rule protocol "any" | 14:14 |
haleyb | I added a comment here based on what i knew | 14:14 |
haleyb | the "any" keyword is only actually valid when using OSC, not the API | 14:15 |
haleyb | but the API ref says "any" is valid | 14:15 |
slaweq | IMO if Openstackclient accepts 'any' as valid value, we may accept it on the server side as well to be more consistent with the cli tool | 14:16 |
slaweq | it should be easy to do | 14:16 |
haleyb | We can clean-up the doc to say "use None or 0", but i was wondering if anyone thought supporting "any" as an alias in the API was a good idea | 14:16 |
haleyb | you type faster than me | 14:17 |
slaweq | :) | 14:17 |
ralonsoh | actually the value in the server is "ip" | 14:17 |
ralonsoh | for protocol 0 | 14:17 |
haleyb | ralonsoh: yes, i pointed that out in the bug, so there's almost too many choices | 14:18 |
ralonsoh | right, I'm now reading your comment | 14:18 |
ralonsoh | we should not change the CLI (or deprecate "any" with time) | 14:18 |
ralonsoh | or accept both "any" or "ip" (and the deprecate "any") | 14:19 |
haleyb | right, i think i've used 'any' in scripts myself | 14:19 |
haleyb | i think it should be easy enough to support 'any' as an alias on the server side | 14:20 |
slaweq | +1 | 14:20 |
ykarel | +1 | 14:21 |
haleyb | the only thing that might be an issue is what does the POST return for protocol? | 14:21 |
slaweq | what it returns now for None and 0 values? | 14:21 |
slaweq | I guess it should be the same and we can clearly document that 'any' is just alias for the value which is returned from API | 14:22 |
haleyb | yes, i just didn't know if someone would be confused if it didn't reutrn "any" | 14:22 |
haleyb | slaweq: yes, that sounds good | 14:22 |
haleyb | i will add that info to the bug, and should be able to work on it eventually | 14:23 |
haleyb | i had filed a bug over the weekend, one of those "this is an annoying traceback" seen in the server logs | 14:24 |
haleyb | #link https://bugs.launchpad.net/neutron/+bug/2074312 | 14:25 |
haleyb | of course my quick fix proposal didn't work | 14:25 |
haleyb | has anyone else noticed those tracebacks? | 14:25 |
haleyb | title was " DB semantic violation seen in neutron-tempest-plugin-openvswitch logs deleting router " | 14:25 |
haleyb | This handler is supposed to handle AFTER events, as in 'AFTER it's committed', not BEFORE. Offending resource event: port, after_delete | 14:26 |
ralonsoh | yes when implementing the new db facade | 14:26 |
ralonsoh | I think this is because we are creating a port inside a transaction | 14:26 |
ralonsoh | sorry, deleting the port | 14:26 |
haleyb | i found the code that should have disabled the warning but it's not working in this case | 14:26 |
ralonsoh | is that somethign recurrent? | 14:26 |
haleyb | there is also one in create_security_group() that doesn't have the "disable the warning" code | 14:26 |
haleyb | every neutron-tempest-openvswitch run has occurrences | 14:27 |
haleyb | neutron-tempest-plugin-openvswitch | 14:27 |
haleyb | i didn't check other logs | 14:27 |
haleyb | one question i had was is it possible to move the code? i figured it would have been done by now if simple | 14:28 |
haleyb | i.e. save the ports and do in a second transaction | 14:28 |
haleyb | ralonsoh: feel free to add a comment to the bug if you have a suggestion, it's not critical just an annoying traceback | 14:30 |
ralonsoh | we can try that, we are deleting the GW port before that txn | 14:30 |
ralonsoh | I'll push a patch but we need to be cautious leaving leftovers | 14:31 |
haleyb | there are at least two TODOs for that same issue, luckily finding GUARD_TRANSACTION is easy | 14:31 |
haleyb | ok, i had no other bugs | 14:32 |
haleyb | any others to discuss? | 14:32 |
haleyb | This week elvira is the bug deputy, next week will be slaweq | 14:32 |
haleyb | is that ok for both? | 14:32 |
elvira | sure! | 14:32 |
haleyb | Current bug count this week: 721, up 14 from last run | 14:33 |
haleyb | #topic community-goals | 14:34 |
slaweq | good for me | 14:34 |
haleyb | We merged the MANAGER role and added documentation, so good there, thanks slaweq ! | 14:35 |
slaweq | yeah, this can be IMO consider as done finally | 14:35 |
haleyb | lajos continues to work on neutronclient deprecation | 14:36 |
haleyb | #link https://review.opendev.org/q/topic:%22bug/1999774%22 | 14:36 |
haleyb | fixes all for horizon | 14:37 |
haleyb | ralonsoh: and it looks like you just have one change left for evenlet deprecation? | 14:38 |
haleyb | #link https://review.opendev.org/c/openstack/neutron/+/924317 | 14:38 |
ralonsoh | I'm currently working in an issue related to wsgi and OVN | 14:38 |
haleyb | #link https://review.opendev.org/q/topic:%22bug/2069581%22 | 14:38 |
ralonsoh | that is part of the eventlet server deprecation | 14:38 |
ralonsoh | yeah, ml2/ovn won't work right now with wsgi (it was 1 week before) | 14:39 |
haleyb | :( | 14:39 |
ralonsoh | so just investigating https://bugs.launchpad.net/neutron/+bug/2075147 | 14:39 |
ralonsoh | and that's all | 14:39 |
haleyb | and i do see the governance change merged | 14:39 |
haleyb | ack, thanks | 14:39 |
haleyb | #link https://review.opendev.org/c/openstack/governance/+/902585 (merged) | 14:40 |
haleyb | #topic on-demand | 14:40 |
haleyb | we already talked about the dhcp-agent looping bug | 14:41 |
haleyb | any other topics? | 14:41 |
ralonsoh | not from me | 14:41 |
haleyb | ykarel: CI meeting in-person today? | 14:42 |
ykarel | haleyb, sent a cancellation mail for today | 14:42 |
haleyb | ykarel: ack | 14:42 |
haleyb | ok, if nothing else we are done, have a good week everyone! | 14:43 |
haleyb | #endmeeting | 14:43 |
opendevmeet | Meeting ended Tue Jul 30 14:43:09 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:43 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.html | 14:43 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.txt | 14:43 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-30-14.00.log.html | 14:43 |
mlavalle | \o | 14:43 |
ralonsoh | bye | 14:43 |
ykarel | ralonsoh, can you re +W https://review.opendev.org/c/openstack/neutron/+/924471 | 14:43 |
ykarel | nova patch merged | 14:43 |
ralonsoh | sure | 14:43 |
ykarel | thx | 14:44 |
opendevreview | Merged openstack/ovn-octavia-provider master: [OVN][CI] Update OVS_BRANCH to be compatible with ovn main https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925092 | 15:00 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once https://review.opendev.org/c/openstack/neutron/+/925194 | 15:03 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Lower spacing time of the OVN maintenance tasks which should be run once https://review.opendev.org/c/openstack/neutron/+/925194 | 15:04 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-api-ovn-wsgi https://review.opendev.org/c/openstack/neutron/+/925205 | 15:06 |
opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider master: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/923196 | 15:09 |
opendevreview | Merged openstack/neutron master: [OVN][CI] Update OVS_BRANCH to be compatible with ovn main https://review.opendev.org/c/openstack/neutron/+/925091 | 15:39 |
opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Remove dead code from get_port_chassis function https://review.opendev.org/c/openstack/ovn-bgp-agent/+/924673 | 16:18 |
opendevreview | Merged openstack/neutron stable/2024.1: [stable/2024.1 only] Drop -master jobs https://review.opendev.org/c/openstack/neutron/+/925193 | 16:34 |
opendevreview | Merged openstack/neutron stable/2023.1: Don't print traceback if standard attr is missing on update https://review.opendev.org/c/openstack/neutron/+/924471 | 16:37 |
opendevreview | Adam Oswick proposed openstack/neutron master: Conditionally run address scope affinity checks https://review.opendev.org/c/openstack/neutron/+/924984 | 17:24 |
opendevreview | Miro Tomaska proposed openstack/neutron master: Only query for port do not get directly https://review.opendev.org/c/openstack/neutron/+/925039 | 19:30 |
opendevreview | Merged openstack/neutron master: Use has_lock_periodic decorator for the tasks which require ovn db lock https://review.opendev.org/c/openstack/neutron/+/925197 | 19:56 |
opendevreview | Merged openstack/neutron stable/2023.2: Add has_lock_periodic decorator for OVN Maintenance https://review.opendev.org/c/openstack/neutron/+/925178 | 19:56 |
opendevreview | Terry Wilson proposed openstack/neutron stable/2024.1: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/923187 | 20:51 |
opendevreview | Terry Wilson proposed openstack/neutron stable/2024.1: Actualy set global "removal limit" options https://review.opendev.org/c/openstack/neutron/+/925301 | 20:57 |
opendevreview | Terry Wilson proposed openstack/neutron stable/2024.1: Actualy set global "removal limit" options https://review.opendev.org/c/openstack/neutron/+/925301 | 20:59 |
opendevreview | Merged openstack/neutron master: Do not release the executor inside ``_check_child_processes`` https://review.opendev.org/c/openstack/neutron/+/924581 | 21:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!