Tuesday, 2024-07-02

opendevreviewRodolfo Alonso proposed openstack/neutron master: Reorder subnet RBAC policy check strings  https://review.opendev.org/c/openstack/neutron/+/92319507:39
ralonsohracosta, ^^ check this patch in your deployment07:40
opendevreviewPavlo Shchelokovskyy proposed openstack/ovn-octavia-provider master: Do not change LB ip_port_mappints for IPv4  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92319608:03
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Release note for ``neutron-ovn-maintenance-worker`` process  https://review.opendev.org/c/openstack/neutron/+/92319808:19
opendevreviewRodolfo Alonso proposed openstack/neutron master: Release note for ``neutron-periodic-workers`` process  https://review.opendev.org/c/openstack/neutron/+/92319908:25
opendevreviewMerged openstack/os-vif unmaintained/zed: Update .gitreview for unmaintained/zed  https://review.opendev.org/c/openstack/os-vif/+/91774209:49
opendevreviewElod Illes proposed openstack/os-vif unmaintained/yoga: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/os-vif/+/92320810:05
*** ykarel__ is now known as ykarel10:37
ykarelfrickler, hi, you got chance to look at https://github.com/cirros-dev/cirros/pull/116 ?10:38
fricklerykarel: sorry, not yet, hopefully later this week10:42
ykarelfrickler, thx10:57
opendevreviewRodolfo Alonso proposed openstack/neutron master: Reorder subnet RBAC policy check strings  https://review.opendev.org/c/openstack/neutron/+/92319511:13
racostaralonsoh, I'll check it, thanks!11:29
opendevreviewBence Romsics proposed openstack/os-vif master: Do not add taps in trunk bridges to the dead vlan  https://review.opendev.org/c/openstack/os-vif/+/92303611:51
racostaralonsoh, I'm applying your change to subnet policies but the Neutron version for OpenStack Yoga doen't have the ADMIN_OR_NET_OWNER_MEMBER rule...11:56
racostaSo, I think the change won't be useful in this case. Please, correct me if I'm wrong.11:58
racostaI don't know if the StackIT folks have already upgraded their OpenStack but they also reported the bug in the same Yoga based version. I didn't see Max here, maybe felixhuettner[m] can help with the Neutron version.12:06
felixhuettner[m]we currently run zed12:07
racostathanks Felix;) so, Zed also doesn't have this policy rule.12:11
ralonsohracosta, no, this rule is not in Zed and this is an error12:28
ralonsohin any case, I would like you to measure, in the API call, how much time it takes to (1) do the SQL query and (2) apply the policies12:28
ralonsohin your case12:28
ralonsohyou should be able to measure the NeutronDbPluginV2.get_subnets method and the logs return the API call times12:30
opendevreviewliuyulong proposed openstack/neutron master: Check subnet service_type for subnet_create  https://review.opendev.org/c/openstack/neutron/+/90122512:59
opendevreviewliuyulong proposed openstack/neutron master: Refactor external-network update to router gateway cascading effect  https://review.opendev.org/c/openstack/neutron/+/90839312:59
opendevreviewBence Romsics proposed openstack/neutron master: DNM test os-vif change in neutron gate  https://review.opendev.org/c/openstack/neutron/+/92323713:09
opendevreviewliuyulong proposed openstack/neutron master: Always get local vlan from port other_config  https://review.opendev.org/c/openstack/neutron/+/92304013:12
opendevreviewLajos Katona proposed openstack/neutron-lib master: api-ref: Add delete port bindings  https://review.opendev.org/c/openstack/neutron-lib/+/92324013:43
opendevreviewBence Romsics proposed openstack/neutron master: DNM test os-vif change in neutron gate  https://review.opendev.org/c/openstack/neutron/+/92323713:44
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Jul  2 14:00:16 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'networking'14:00
haleybPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh14:00
mtomaskao/14:00
slaweqo/14:00
ralonsohhello14:00
ykarelo/14:00
obondarevo/14:00
haleybhi everyone, let's get started14:01
lajoskatonao/14:01
haleyb#topic announcements14:01
haleybWe are now in Dalmatian release week (R - 13)14:01
haleyb#link https://releases.openstack.org/dalmatian/schedule.html14:01
haleybThis is the Dalmation-2 milestone - most libraries have been released14:02
bcafarellate o/14:02
rubasovo/14:02
haleybD-3 is week of Aug 26th, which will be feature freeze, so 8 weeks away14:03
haleyband library freeze is the week before that14:04
haleyberr, non-client libraries14:04
haleybso if you have reviews that need reviews, feel free to use the priority +1/+2 tag so they are in that dashboard14:05
haleyb#link https://review.opendev.org/dashboard/?title=Neutron+Priorities+Dashboard&foreach=(project:openstack/neutron+OR+project:openstack/neutron-lib+OR+project:openstack/neutron-tempest-plugin+OR+project:openstack/python-neutronclient+OR+project:openstack/neutron-specs+OR+project:openstack/networking-bagpipe+OR+project:openstack/networking-bgpvpn+OR+project:openstack/networking-odl+OR+project:openstack/networking-ovn+OR+projec14:05
haleybt:openstack/networking-sfc+OR+project:openstack/neutron-dynamic-routing+OR+project:openstack/neutron-fwaas+OR+project:openstack/neutron-fwaas-dashboard+OR+project:openstack/neutron-vpnaas+OR+project:openstack/neutron-vpnaas-dashboard+OR+project:openstack/os-ken+OR+project:openstack/ovsdbapp)+status:open&High+Priority+Changes=label:Review-Priority=2&Priority+Changes=label:Review-Priority=1&Blocked+Reviews=label:Review-Priorit14:05
haleyby=-114:05
ralonsohhmmmm14:05
ralonsohcan you paste that in https://paste.opendev.org ?14:06
haleybsure, was wondering what the hmmm was for14:06
slaweqI would like You to look at https://review.opendev.org/c/openstack/neutron-lib/+/921649 so I added review priority +1 for it14:07
haleyb#link https://paste.opendev.org/show/bTecGXQhGNXcYhNCdYRk/14:08
ralonsohwe are not releasing n-lib now14:08
haleybsorry, the captcha was not liking me14:08
ralonsohI talked to release folks yesterday14:08
ralonsohand they said that there was a n-lib release just 2 weeks ago14:08
ralonsohbut we can propose a new n-lib releases patch at any time14:09
slaweqahh, ok14:09
slaweqso in that way it is not that urgent :)14:09
slaweqthx ralonsoh 14:09
haleyband ovsdbapp and os-ken and neutronclient have released recently, so we are good there14:11
haleybJuly 4th is a US Holiday, PTL and others will likely be out the 5th as well, so plan accordingly14:12
haleybReminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers14:12
lajoskatona+1, take a rest and enjoy :-)14:13
haleybralonsoh: i think there was just one RFE from liushy, if you decide to have a drivers meeting this friday based on the responses14:13
ralonsohhow many people is going to attend?14:14
ralonsohplease vote^14:14
haleyb-114:14
ralonsoh+114:14
slaweq+114:14
lajoskatona+114:14
ralonsohykarel, ?14:15
ykarel+114:15
ralonsohin any case, 4 cores is not quorum14:15
ralonsohI would cancel the meeting this Friday14:15
* slaweq needs to commute to other location and will be only lurking at the channel on phone for some time14:15
haleybright, and mlavelle was a -1 from last week14:15
haleybi will cancel this week's drivers meeting14:16
ralonsohthanks14:16
haleybi will add the TC meeting is today, and eventlet removal will be discussed14:17
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/14:18
haleybslawek had added to on-demand but putting here since it impacts us14:18
slaweqYes, I just wanted to let everyone know about it14:18
slaweqThx for bringing it up haleyb 14:19
haleyband looks like it's at 15:00 utc, which conflicts with ci meeting14:20
haleybhmm, that is maybe wrong14:21
slaweqYou mean tx meeting? It is at 18utc14:21
ralonsohright, I though this topic was discussed at 18utc, on zoom14:21
haleyb#link https://wiki.openstack.org/wiki/Meetings/TechnicalCommittee#Next_Meeting14:22
haleybi clicked on the board meeting by accident14:22
haleybso that is today at 18:00 utc14:23
haleybi had no other announcements14:23
haleyb#topic bugs14:24
haleybralonsoh was the bug deputy last week14:24
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/A37YRRUCMNEM6Z5S4OHVLHZVIEJQJTVN/14:24
ralonsohall addressed, I think14:24
haleybyes, looks like all have owners, most have patches, and the one rfe14:25
haleybralonsoh: thanks for the email and work on the dhcp-agent workers14:26
haleyb#link https://paste.opendev.org/show/b8JUaVPaZNIRA1cm08fw/14:26
haleybit seems we didn't need those workers all along14:26
ralonsohI think so, yes, I'll continue with the patch and the documentation14:26
ralonsoh(one issue down for the eventlet deprecation)14:27
haleybanyone have other bugs to discuss?14:28
haleybThis week lajoskatona is the bug deputy, next week will be ykarel14:28
haleybis that good for both?14:29
ykarelack14:29
lajoskatonaI am on it14:29
haleybCurrent bug count this week: 707, down 25 from two weeks ago as i went through the old bug etherpad14:29
haleyb#link https://etherpad.opendev.org/p/dalmatian-neutron-bug-squash14:30
haleybi need to regenerate that to include some more bug, and push the tool i used to generate it14:30
haleyblajoskatona: it looks like there are still changes related to the os-ken bug(s) that need merging, will take a look after meeting14:32
haleyb#link https://review.opendev.org/q/project:openstack/os-ken+status:open14:32
lajoskatonaseems most of the mare green14:32
lajoskatonabut as I see mostly with old zuul runs :-(14:33
haleybyes, unmaintained gates are crufty14:33
haleybi will skip ahead to community as i don't see any active specs14:34
haleyb#topic community-goals14:34
haleybslaweq might be driving (?) so don't know about S-RBAC status14:35
haleybwe can discuss when he gets back or after meeting14:36
haleyb#topic on-demand14:36
haleybralonsoh: did many respond to your doodle?14:37
ralonsohhttps://doodle.com/meeting/organize/id/e3KAwKRe14:37
ralonsoh4 people14:37
ralonsohyou can't on wednesday, right?14:38
ralonsohit's a pity because I can't attend today at 15UTC14:38
haleybralonsoh: no, wednesday i have meetings all morning :(14:38
ralonsohok, so I'll abandon the idea, maybe in some months I'll be able to join these meetings again14:39
ralonsohthat's all14:40
rubasovhere is an os-vif bugfix proposal where I would appreciate neutron eyes too: https://review.opendev.org/c/openstack/os-vif/+/92303614:40
ykarelmay be we can consider Monday?14:41
ralonsohbtw, we didn't talk about the eventlet deprecation in the community goals section14:41
ykarelthe poll didn't allowed to vote today as the day was gone14:41
ralonsohykarel, that's in the pool and this option is not considered14:41
haleybmonday had some X's14:42
ykarelyeap because it didn't allowed to vote likely14:42
ralonsohright, monday is not an option14:42
ykareli wanted to vote there but it didn't allowed14:42
ralonsohok14:42
ykarelnot sure about lajoskatona 14:42
ralonsohlajoskatona, ?14:42
ralonsohin any case, mlavalle voted no last week14:43
haleybralonsoh: i didn't see eventlet deprecation in community goals section, maybe i forgot to add it14:43
ykarellikely as he is out all week 14:43
ykarelk can check this later14:43
ralonsohperfect, I'll check it next week14:43
lajoskatonafor me Monday can work, wednesday is tricky, but in worst case that can also 14:43
ralonsohhaleyb, I wanted to highlight this: https://review.opendev.org/c/openstack/oslo-specs/+/92259714:44
ralonsohlajoskatona, ok so mlavalle is the only one that voted X, and I can blackmail him14:44
ralonsohso that could work14:44
ralonsohI'll wait for the next week14:44
ralonsohplease check the oslo-specs patch: this is introducing the option to select from eventlet and kernel threads (using cotyledon and futurist)14:45
ralonsohthat will help us with the migration of the agents, so please spend 10 mins reading it14:45
ralonsohthanks!14:45
lajoskatonarubasov: the os-vif patch is related to  https://bugs.launchpad.net/neutron/+bug/2069543 (instance tap port has dead vlan when parent port for trunk )14:45
haleyback, will take a look and google 'cotyledon' :)14:46
lajoskatona?14:46
rubasovyes, it is, earlier I had both a neutron and an os-vif patch14:46
rubasovbut the current iteration is os-vif only14:46
slaweqHi. I'm back now14:46
slaweqSorry14:47
slaweqDo you want update about S-RBAC?14:47
ralonsohrubasov, only os-vif patch is needed?14:47
haleybslaweq: once we are done talking about this os-vif change14:47
rubasovralonsoh: at least Sean suggested that14:47
slaweqSure14:47
rubasovI'm only 90% convinced14:48
ralonsohok, I'll check it tomorrow morning, thanks for it!14:48
rubasovthanks!14:48
haleybslaweq: ok, any updates on S-RBAC?14:48
slaweqYes14:49
slaweqLast week I was checking with gmann and it seems that on neutron side we are good with using service tokens. It needs to be also done on the nova's side now14:50
slaweqSecond thing is that I'm starting work on the MANAGER role in neutron14:50
slaweqSo please expect some patches related to that soon14:50
slaweqThat's all for today14:50
ralonsoh+114:50
haleybslaweq: great, thanks14:51
haleybralonsoh: since i'm thinking about it, do you need more reviews on any wsgi changes?14:51
ralonsohthe OVN one (one sec)14:52
haleybi've at least seen the devstack changes green which is good14:52
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/92247014:52
ralonsohyes, the devstack ones, if possible14:52
ralonsohhttps://review.opendev.org/c/openstack/devstack/+/92261114:52
ralonsohhttps://review.opendev.org/c/openstack/devstack/+/92212514:52
haleyback, will review after meeting14:53
haleybany other topics today?14:53
haleyboh, and CI meeting today, irc i believe14:53
ykarelyeap right it's IRC today14:54
lajoskatona+114:54
haleyb#endmeeting14:54
opendevmeetMeeting ended Tue Jul  2 14:54:30 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:54
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-02-14.00.html14:54
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-02-14.00.txt14:54
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-07-02-14.00.log.html14:54
ralonsohbye14:54
haleybhave a good week everyone!14:54
lajoskatonao/14:55
ykarel#startmeeting neutron_ci15:02
opendevmeetMeeting started Tue Jul  2 15:02:26 2024 UTC and is due to finish in 60 minutes.  The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot.15:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:02
opendevmeetThe meeting name has been set to 'neutron_ci'15:02
ykarelPing list: bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira15:02
lajoskatonao/15:03
slaweqo/15:04
bcafarelo/15:05
haleybo/15:05
ykarelhello everyone, let's start with topics from last week15:07
ykarel#topic Actions from previous meetings15:07
ykarellajoskatona to check fullstack failure test_configurations_are_synced_towards_placement15:08
ykarellajoskatona, you got chance to check that?15:08
lajoskatonaI checked quickly this week, and since than no occurance of it15:08
ykarelyeap me too not noticed again but it happens from time to time15:09
lajoskatonaby the logs timeing issue with the fake placement API in the fullstack framework15:09
lajoskatonayes, that's true15:09
lajoskatonaactually those tests are quite shaky as a fake placement service isrunning to answer API calls from Neutron15:11
ykareldo you think if increasing timeout will help here?15:14
lajoskatonaykarel: i can check that if there can be a place to increase timeout 15:15
lajoskatonayou can add an AP for that15:15
ykarel#action lajoskatona to check timeout increase for test_configurations_are_synced_towards_placement15:16
ykarelthx lajoskatona 15:16
ykarelykarel to open bug for fullstack failures15:16
ykarelhttps://launchpad.net/bugs/207039015:16
ykarelfix for this already merged, and not seen the failure again15:16
ykarel#topic Stable branches15:16
ykarelbcafarel, any updates here?15:17
bcafarelall good from what I saw (it was quiet on backports front also this week)15:17
ykarelk thx15:19
ykarel#topic Stadium projects15:19
ykarelsfc/bagpipe still red15:19
ykarel- https://bugs.launchpad.net/neutron/+bug/206872715:19
ykarel- https://bugs.launchpad.net/neutron/+bug/206745215:19
lajoskatonayes, I pushed a u-c bump patch for exabgp: https://review.opendev.org/c/openstack/requirements/+/92331115:20
lajoskatonaI just realized that is needed :-(15:20
lajoskatonabut I had no time for sfc15:20
ykarelohkk i thought those deps update happens automatically for master branch, may be something changed15:22
lajoskatonaI expected the same :-)15:23
ykarelso we thought right as there is patch for that https://review.opendev.org/c/openstack/requirements/+/922558/15:23
ykarelincluding all other updates15:23
lajoskatonaohhh, ok so automatic bump is still there, but fails on something15:24
lajoskatonaon a lot of things15:25
ykarelyeap there are support removed for older python versions 15:25
ykareljust checked voluptuous py3.8 drop, but there could be other deps issue15:25
lajoskatonanot sure if we can push the sole exabgp bump to fix our gate or have to wait anyway for the group bump15:25
ykareli think we can push that's totally fine15:26
lajoskatona+1, I will update the big bump patch to avoid coflict15:27
ykarelk thx15:27
ykarellet's move15:27
ykarel#topic Rechecks15:27
ykarelall good here, and had just 2/22 bare rechecks15:27
ykarel#topic fullstack/functional15:28
ykareltest_get_all_devices (stable/2023.1)15:28
ykarelhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_aac/periodic/opendev.org/openstack/neutron/stable/2023.1/neutron-functional/aacd179/testr_results.html15:28
ykarelFound old bugs Old bugs:- https://launchpad.net/bugs/2012510 and https://bugs.launchpad.net/neutron/+bug/199734115:28
ykareland for one of them there were some stabilization bits pushed, which not backported to stable/2023.115:28
ykarelso we may try backport for those15:28
ykarelanyone willing to check and propose those?15:29
ykarelk i will do check and propose those15:30
ykarel#action ykarel to check backports for test_get_all_devices functional test failures15:30
ykarel#topic Tempest/Scenario15:30
ykarelopenstacksdk-functional-devstack-networking15:30
ykarelhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_858/periodic/opendev.org/openstack/neutron/master/openstacksdk-functional-devstack-networking/858cdaa/testr_results.html15:31
ykarelhttps://fbb34ddc02d1f109337c-ebf431c06382d97dfdf40c8495b34507.ssl.cf5.rackcdn.com/922829/1/check/openstacksdk-functional-devstack/202851a/testr_results.html15:31
ykarelseen only in master periodic once, but looking opensearch seen similar failures in other openstacksdk jobs15:31
slaweqI will check them as this is my change which probably broke it15:31
ykarelthx slaweq 15:31
ykarelfrom timing i thought to also check https://review.opendev.org/c/openstack/devstack/+/922630 if that triggered it15:32
ykarel#action slaweq to check failures in openstacksdk-functional jobs15:32
ykarel#topic Grafana15:32
ykarelhttps://grafana.opendev.org/d/f913631585/neutron-failure-rate15:33
ykarellet's also have quick look here too15:33
ykarellooks good there are some spikes in check but those looks patch specific as different jobs failing 15:36
ykarelanything to add?15:36
slaweqnope15:37
lajoskatonanope15:37
ykarel#topic On Demand15:37
ykarelanyting you would like to raise here?15:38
lajoskatonanothing from me15:38
slaweqnothing from me neighter15:39
ykareljust that we are considering moving this meeting to Monday an hour earlier, ralonsoh will check/confirm that next week15:39
ykarelmay be will have more participants with that15:39
ykarelthx everyone for joining15:40
ykarel#endmeeting15:40
opendevmeetMeeting ended Tue Jul  2 15:40:21 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-07-02-15.02.html15:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-07-02-15.02.txt15:40
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-07-02-15.02.log.html15:40
lajoskatonao/15:40
lajoskatonaBye15:40
slaweqo/15:40
lajoskatonahaleyb: I have a pending feature for tap-as-a-service: https://review.opendev.org/q/topic:%22bug/2015471%22 , could you please think about it if we can push it in this cycle? Part of it is already merged15:43
fricklerlajoskatona: https://review.opendev.org/c/openstack/requirements/+/923311 should be fine, just waiting for CI results. also don't bother updating the group bump, the bot will do that next saturday anyway15:47
opendevreviewMiro Tomaska proposed openstack/neutron stable/2023.1: [2023.1 Only][OVN] Add the bridge name and datapath type to the port VIF details  https://review.opendev.org/c/openstack/neutron/+/92243115:52
opendevreviewMiro Tomaska proposed openstack/neutron stable/2023.1: [2023.1 Only][OVN] Add the bridge name and datapath type to the port VIF details  https://review.opendev.org/c/openstack/neutron/+/92243115:54
ihrachyshaleyb I reviewed the latest nested snat patch and I agree this is the best approach. I'd like to +2 but before I do I'd like you to check the test comments here - https://review.opendev.org/c/openstack/neutron/+/907504/21..24/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_ovn_client.py and tell me if they make sense.15:54
haleyblajoskatona: i don't see why not, just need reviews, i might have some cycles16:45
haleybihrachys: yes, let me re-work the tests slightly, thanks16:51
opendevreviewMerged openstack/os-ken unmaintained/xena: Raise ValueError in case unpack_from returns zero length  https://review.opendev.org/c/openstack/os-ken/+/92281417:02
opendevreviewBrian Haley proposed openstack/neutron master: Change OVN client to account for nested networks  https://review.opendev.org/c/openstack/neutron/+/90750417:10
lajoskatonahaleyb: thanks17:40

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