opendevreview | Lewis Denny proposed openstack/neutron master: Add max limit to agent_down_time https://review.opendev.org/c/openstack/neutron/+/889373 | 00:06 |
---|---|---|
opendevreview | Merged openstack/neutron master: hash-ring: Retry all DB operations if inactive https://review.opendev.org/c/openstack/neutron/+/890245 | 01:33 |
opendevreview | Takashi Kajinami proposed openstack/neutron master: Fix missing oslo.versionedobjects library option https://review.opendev.org/c/openstack/neutron/+/890716 | 06:06 |
amorin | hey team, would you mind pinging me and aprats when neutron meeting starts? We'd like to talk about https://bugs.launchpad.net/neutron/+bug/2029722 | 07:06 |
slaweq | lajoskatona hi, can You maybe check https://review.opendev.org/c/openstack/neutron/+/889153 when You will have few minutes? Thx in advance | 07:19 |
opendevreview | wangjiaqi proposed openstack/neutron master: Cleanup py27 support https://review.opendev.org/c/openstack/neutron/+/890728 | 07:25 |
opendevreview | wangjiaqi proposed openstack/neutron-lib master: Cleanup py27 support https://review.opendev.org/c/openstack/neutron-lib/+/890729 | 07:27 |
lajoskatona | slaweq: checking | 07:28 |
opendevreview | wangjiaqi proposed openstack/neutron-tempest-plugin master: Cleanup py27 support https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/890730 | 07:28 |
opendevreview | wangjiaqi proposed openstack/os-ken master: Cleanup py27 support https://review.opendev.org/c/openstack/os-ken/+/890735 | 08:49 |
opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM Just testing some sql queries https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/890419 | 09:14 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: [OVN] Add baremetal support without Neutron DHCP agent for IPv6 https://review.opendev.org/c/openstack/neutron/+/890683 | 10:08 |
opendevreview | Merged openstack/neutron master: Add NET_OWNER_MEMBER and NET_OWNER_READER policy rules https://review.opendev.org/c/openstack/neutron/+/889153 | 10:11 |
opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Hash Ring: Better handle Neutron worker failures https://review.opendev.org/c/openstack/neutron/+/890392 | 10:41 |
opendevreview | Merged openstack/neutron stable/zed: [OVN] Hash Ring: Better handle Neutron worker failures https://review.opendev.org/c/openstack/neutron/+/890393 | 10:41 |
opendevreview | Merged openstack/neutron stable/yoga: [OVN] Hash Ring: Better handle Neutron worker failures https://review.opendev.org/c/openstack/neutron/+/890394 | 10:41 |
opendevreview | Merged openstack/neutron stable/xena: [OVN] Hash Ring: Better handle Neutron worker failures https://review.opendev.org/c/openstack/neutron/+/890395 | 10:41 |
opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and flow examples for OVS https://review.opendev.org/c/openstack/tap-as-a-service/+/828382 | 11:44 |
racosta | Hey folks, if you have some time, please check: https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/888787 | 12:35 |
*** elvira2 is now known as elvira | 12:50 | |
opendevreview | Brian Haley proposed openstack/neutron master: Remove some invalid text from the network component doc https://review.opendev.org/c/openstack/neutron/+/890790 | 13:08 |
opendevreview | Merged openstack/neutron stable/zed: dvr: Avoid installing non-dvr openflow rule on startup https://review.opendev.org/c/openstack/neutron/+/890229 | 13:11 |
opendevreview | Merged openstack/neutron stable/2023.1: dvr: Avoid installing non-dvr openflow rule on startup https://review.opendev.org/c/openstack/neutron/+/890228 | 13:11 |
opendevreview | Merged openstack/neutron stable/wallaby: dvr: Avoid installing non-dvr openflow rule on startup https://review.opendev.org/c/openstack/neutron/+/890231 | 13:11 |
*** elvira1 is now known as elvira | 13:11 | |
opendevreview | Tobias Fischer proposed openstack/neutron master: Add more logging for lrp scheduling on gtws https://review.opendev.org/c/openstack/neutron/+/890795 | 13:40 |
*** obondarev_ is now known as obondarev | 13:56 | |
slaweq | #startmeeting networking | 14:00 |
opendevmeet | Meeting started Tue Aug 8 14:00:17 2023 UTC and is due to finish in 60 minutes. The chair is slaweq. 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 |
mlavalle | o/ | 14:00 |
lajoskatona | o/ | 14:00 |
slaweq | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:00 |
slaweq | o/ | 14:00 |
frickler | \o | 14:00 |
obondarev | o/ | 14:00 |
rubasov | o/ | 14:00 |
ykarel | o/ | 14:00 |
haleyb | o/ | 14:00 |
slaweq | ralonsoh is still off so I will chair today's meeting again | 14:00 |
bcafarel | o/ | 14:01 |
slaweq | I think we can start as we have pretty many folks there already | 14:01 |
slaweq | #topic announcements | 14:01 |
sahid | o/ | 14:01 |
slaweq | I just have regular reminder about release schedule | 14:01 |
slaweq | Bobcat / 2023.2 schedule: https://releases.openstack.org/bobcat/schedule.html | 14:01 |
slaweq | Next milestone is Aug 21-Aug 25 - Final release for non-client libraries | 14:02 |
slaweq | and also PTL and TC nominations will start in 3 weeks (also Aug 21-Aug 25) - please consider preparing Your nomination if You are interested in one of those positions | 14:02 |
slaweq | that's all announcements from me today | 14:03 |
slaweq | do You have anything else? | 14:03 |
slaweq | I guess this means "no" :) | 14:04 |
slaweq | #topic bugs | 14:04 |
slaweq | lucasagomes was bug deputy last week but he forgot to sent summary email | 14:04 |
slaweq | I just pinged him about it before this meeting so please expect email before EOD today | 14:04 |
lucasagomes | sorry folks, I will submit it soon! | 14:05 |
slaweq | in the meantime do You have any bugs You would like to discuss now? | 14:05 |
slaweq | or maybe lucasagomes have already something what would like to bring up here? | 14:05 |
frickler | amorin wanted to discuss one | 14:05 |
lucasagomes | I just started looking into the list, if I find something in the meantime I will bring it up | 14:05 |
amorin | hello! | 14:05 |
frickler | https://bugs.launchpad.net/neutron/+bug/2029722 | 14:06 |
amorin | thanks for the ping :) | 14:06 |
slaweq | hi amorin | 14:06 |
amorin | yup, we discoverd something when using DVR | 14:06 |
amorin | some ip rules are set in order to reach the external gateway of the subnet | 14:06 |
amorin | in a 2 routers architecture, with custom routing rules, it prevent the routing to reach the gw correctly | 14:07 |
amorin | this is not affected no DVR | 14:07 |
amorin | this is not affecting* no DVR | 14:07 |
amorin | we were wondering if someone know why, for DVR, we are using the ip rule to reach the gw | 14:07 |
amorin | and a different routing table | 14:07 |
amorin | instead of the default one | 14:07 |
slaweq | I see that haleyb already started looking at it and I guess that he and maybe obondarev are the best guys to ask why it was originally done like that | 14:09 |
obondarev | there is a great set of articles describing DVR mechanism by Assaf Muller: https://assafmuller.com/category/dvr/ - amorin you may find answers there | 14:09 |
amorin | ack, will read that | 14:10 |
haleyb | it's been a while, but there are typically ip rules to deal with scoping of addresses. | 14:10 |
obondarev | currently I don't remember this detail on ip rule, sorry | 14:10 |
amorin | we implemented a patch to add an extra rule, it works | 14:10 |
amorin | but if the original idea of ip rule is not needed anymore, maybe we can totally get rid of it | 14:11 |
haleyb | as i mentioned in the patch, it could be a certain order of things shows the bug, sometimes we just don't account for things correctly | 14:11 |
haleyb | i think the rules are needed to account for address scopes | 14:12 |
slaweq | amorin so You can probably read Assaf's blog posts about it and we can continue that discussion in the LP and gerrit, is that ok for You? | 14:14 |
haleyb | we can just follow-up in the patch and bug, but i am following it | 14:14 |
amorin | sounds good, thanks | 14:14 |
slaweq | thx amorin for bringing it up here and thx haleyb for taking care of that LP | 14:15 |
slaweq | ok, I think we can move on | 14:15 |
slaweq | #topic community_goals | 14:15 |
slaweq | Consistent and Secure Default RBAC | 14:15 |
slaweq | Patch https://review.opendev.org/c/openstack/neutron/+/886724 should be ready to review now - it still requires neutron-lib bump in the upper-contraints but other than that it looks ok | 14:16 |
slaweq | so please take a look at it if You will have few minutes | 14:16 |
slaweq | I would like to hopefully get it merged in this cycle | 14:16 |
slaweq | next one | 14:17 |
slaweq | Neutron client deprecation | 14:17 |
slaweq | lajoskatona any updates on this one? | 14:17 |
lajoskatona | not much, I worked on the sfc part | 14:17 |
lajoskatona | this is for SDK: https://review.opendev.org/c/openstack/neutron/+/886724 | 14:17 |
lajoskatona | the usual etherpad: https://etherpad.opendev.org/p/python-neutronclient_deprecation | 14:18 |
lajoskatona | please check it, the patches without a 'tick' are something to review | 14:18 |
lajoskatona | this one for fwaas is ready: https://review.opendev.org/c/openstack/python-neutronclient/+/880629 as SKd was released for it | 14:19 |
slaweq | lajoskatona I think You gave wrong link "for SDK" :) | 14:19 |
lajoskatona | sorry, my mind was faster :-) | 14:19 |
slaweq | :) | 14:19 |
lajoskatona | otherwise I started to look into Horizon, and started to play to have SDK behind it | 14:20 |
lajoskatona | that's it for this topic from me | 14:20 |
slaweq | thx | 14:20 |
slaweq | I will finally try to find time to review those patches this week | 14:20 |
slaweq | with that I think we can move on | 14:21 |
lajoskatona | thanks in advance | 14:21 |
slaweq | to the next topic, which is | 14:21 |
slaweq | #topic on_demand | 14:21 |
slaweq | do You have any other neutron related topics to discuss today? | 14:21 |
mlavalle | nope | 14:21 |
frickler | I have a question about IPv6 metadata | 14:21 |
slaweq | frickler sure | 14:21 |
frickler | I see the tests are only done for OVS, and in my local test it doesn't work for OVN, is that a known gap? | 14:22 |
slaweq | yes, it is | 14:22 |
slaweq | OVN metadata agent don't support it (yet) | 14:22 |
frickler | so it is missing in the gap document | 14:23 |
slaweq | I have it on my todo list but not with high priority for now | 14:23 |
slaweq | right, I think we forgot about gaps document | 14:23 |
slaweq | to add it there | 14:23 |
frickler | ok, I can do a patch, need to add some DNS gaps, too, it seems | 14:24 |
frickler | also haleyb do you still plan to followup with https://review.opendev.org/c/openstack/neutron/+/876903 ? | 14:24 |
slaweq | thx frickler | 14:24 |
haleyb | frickler: i need to abandon that and work on a different fix, that didn't work :( | 14:24 |
haleyb | and we don't need to switch to the ec2 one | 14:25 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences https://review.opendev.org/c/openstack/neutron/+/890799 | 14:25 |
frickler | well not switch but add it as an option I though would be a good idea? | 14:25 |
frickler | *thought | 14:26 |
haleyb | i think the problem was that there is no dhcp response value for host routes in ipv6, so it doesn't work | 14:26 |
haleyb | and there was an issue with dnsmasq and addressing, just making sure one host at a time configures the address should be enough | 14:27 |
frickler | but do you need a specific route? wouldn't the default route from the RA be enough? | 14:27 |
haleyb | not when you have an isolated subnet, which is where the dnsmasq issue got into play | 14:27 |
racosta | In the IPv6-only case, the VM generates an LLA address automatically, and this local scope address is not known by ovn-southbound and neutron. At this point we have a hard time! In the current architecture, the metadata makes a proxy and uses the local address of the VM to find the corresponding port to forward and receive the traffic (Port_Binding table) - OVN case. | 14:28 |
haleyb | we decided to just fix it a different way | 14:28 |
frickler | ah, ok, maybe split those topics, then | 14:28 |
haleyb | i'll try and work on that in the next few weeks | 14:29 |
frickler | o.k., I'll wait for that, thx | 14:30 |
slaweq | ok, thank You both for bringing it up here and working on it | 14:31 |
slaweq | I also have 2 quick topics for today on demand section | 14:31 |
slaweq | first one, which I wanted to discuss also with ralonsoh but as he's not here yet, I will ask whole team here: | 14:32 |
slaweq | Do we want to add neutron-core to the openstacksdk/OSC "service-core" group? | 14:32 |
slaweq | See https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034608.html for more details | 14:32 |
slaweq | example patch in project-config: https://review.opendev.org/q/topic:add-cinder-core | 14:32 |
lajoskatona | good idea | 14:32 |
slaweq | IMO it would be good idea to add neutron-core team to that SDK/OSC group also but I wanted to know Your opinion about it | 14:32 |
mlavalle | good idea | 14:33 |
frickler | we have some individuals already added I think | 14:33 |
obondarev | +1 to add | 14:33 |
slaweq | frickler yes, I think that I am there and amotoki | 14:34 |
slaweq | but still having all neutron-cores in that service-core group would be good IMO | 14:34 |
frickler | yes, I'm all for it, just wanted to mention that it could simplify the list a bit | 14:34 |
slaweq | thx | 14:35 |
slaweq | ok, I will propose such patch and also ask ralonsoh to review it as Neutron PTL | 14:35 |
slaweq | and now second topic | 14:35 |
slaweq | As You probably noticed it's that time of the year where user survey questions are going to be prepared for next surver. We have list of current questions, see https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034596.html for details | 14:36 |
slaweq | and we have time until 18th of Aug to propose some changes there in Neutron related questions | 14:37 |
slaweq | I would like to ask You all to check what's there currently and maybe we can discuss about it on the next week's team meeting if we want to change something there | 14:37 |
slaweq | will that work for You? | 14:37 |
lajoskatona | +1 | 14:38 |
frickler | can someone copy the questions into an etherpad? | 14:39 |
slaweq | frickler yes, I will prepare etherpad with those questions today or tomorrow morning | 14:39 |
mlavalle | will look at it | 14:39 |
slaweq | and will send link to it in this channel | 14:39 |
lajoskatona | thanks slaweq | 14:39 |
frickler | cool, thx | 14:39 |
slaweq | and with that we came to the end of the agenda for today | 14:40 |
slaweq | if You don't have any other topics for today, I will give You some time back | 14:40 |
mlavalle | \o/ | 14:41 |
slaweq | and please remember about CI meeting in 19 minutes | 14:41 |
slaweq | it will be on video this week | 14:41 |
lajoskatona | ack | 14:41 |
mlavalle | ack | 14:41 |
slaweq | #endmeeting | 14:41 |
opendevmeet | Meeting ended Tue Aug 8 14:41:19 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:41 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.html | 14:41 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.txt | 14:41 |
opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.log.html | 14:41 |
slaweq | o/ | 14:41 |
bcafarel | noted | 14:41 |
lajoskatona | o/ | 14:41 |
mlavalle | o/ | 14:41 |
rubasov | o/ | 14:42 |
amorin | o/ | 14:43 |
slaweq | #startmeeting neutron_ci | 15:01 |
opendevmeet | Meeting started Tue Aug 8 15:01:05 2023 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
opendevmeet | The meeting name has been set to 'neutron_ci' | 15:01 |
mtomaska | o/ | 15:01 |
slaweq | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:01 |
slaweq | This will be video meeting this time: https://meetpad.opendev.org/neutron-ci-meetings | 15:01 |
slaweq | Grafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=1 | 15:02 |
slaweq | #topic Actions from previous meetings | 15:03 |
slaweq | ralonsoh to check failed neutron.tests.fullstack.test_l3_agent.TestHAL3Agent.test_keepalived_multiple_sighups_does_not_forfeit_primary test | 15:03 |
slaweq | #action ralonsoh to check failed neutron.tests.fullstack.test_l3_agent.TestHAL3Agent.test_keepalived_multiple_sighups_does_not_forfeit_primary test | 15:03 |
slaweq | ykarel to check broken centos periodic jobs | 15:03 |
slaweq | https://bugs.launchpad.net/neutron/+bug/2029335 | 15:04 |
slaweq | https://review.opendev.org/c/openstack/devstack/+/890280 | 15:04 |
slaweq | https://review.opendev.org/#/q/Icd99f467d47aaafaaf3ee8f2a3c4da08842cb672 | 15:04 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences https://review.opendev.org/c/openstack/neutron/+/890799 | 15:06 |
slaweq | mtomaska to check failing neutron-functional-with-sqlalchemy-master periodic job | 15:08 |
mtomaska | https://opendev.org/openstack/neutron/src/branch/master/neutron/db/migration/cli.py#L84 | 15:08 |
slaweq | #action mtomaska to check failing neutron-functional-with-sqlalchemy-master periodic job | 15:14 |
slaweq | #topic Stable branches | 15:14 |
slaweq | #topic Stadium projects | 15:14 |
slaweq | #action lajoskatona will send DNM patch for neutron-dynamic-routing to check jobs | 15:17 |
slaweq | #topic Grafana | 15:17 |
slaweq | #topic Rechecks | 15:19 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences https://review.opendev.org/c/openstack/neutron/+/890799 | 15:22 |
slaweq | #topic Tempest/Scenario | 15:29 |
slaweq | https://cfc27c554a994f35be4e-76bf72ffc642f12cb8e7d8393148d522.ssl.cf5.rackcdn.com/874797/41/gate/neutron-ovn-tempest-ipv6-only-ovs-release/8162e79/testr_results.html | 15:30 |
mtomaska | yep installing master deps right now | 15:32 |
slaweq | #endmeeting | 15:34 |
opendevmeet | Meeting ended Tue Aug 8 15:34:17 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:34 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.html | 15:34 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.txt | 15:34 |
opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.log.html | 15:34 |
opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: DNM: Test patch only https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/890806 | 15:36 |
aprats | Hello there ! Regarding #2029722 I followed obondarev advice and looked at the blog. I didn't find an explanation of why there is this ip rule. One of the explanation I could come up with is that you don't want to route traffic through the snat if it doesn't know the way to route the packet back to the other subnet. With this in mind, my patch seems relevant, only adding an ip rule if when the route exists. | 15:38 |
frickler | lajoskatona: is there anything up with n-d-r or just a general check? | 16:01 |
haleyb | aprats: ack, thanks. So the first router didn't know of the routes to the subnet behind the second router? | 16:29 |
haleyb | can you just put the steps you did to recreate this in the bug? for example the 'openstack...' commands | 16:30 |
lajoskatona | frickler: Hi, just a failure in the periodic job, see: https://zuul.openstack.org/buildset/ee086b0e391f4272aadd9acea19adc4f | 16:30 |
lajoskatona | frickler: and to see if it was an accident or something to take care | 16:30 |
frickler | lajoskatona: seems the failure was reproduced, looks docker related. I can take a closer look tomorrow, thx for spotting that issue | 17:13 |
aprats | haleyb: ack, I will add the commands and a snippet of my tf. On my last try, all the routers had the routes as described in the bug before enabling the gateway on Router Ext. | 17:37 |
lajoskatona | frickler: thanks, tomorrow I am on PTO, but after that if need help or out of time just ping me | 18:02 |
opendevreview | Adam proposed openstack/neutron master: Handle cases where FIP rules are missing from host https://review.opendev.org/c/openstack/neutron/+/890827 | 19:17 |
opendevreview | Brian Haley proposed openstack/neutron master: Update OVN db sync code log messages https://review.opendev.org/c/openstack/neutron/+/890832 | 21:12 |
opendevreview | Merged openstack/neutron stable/yoga: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887603 | 21:34 |
opendevreview | Merged openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 23:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!