Tuesday, 2024-01-02

« Sunday, 2023-12-31 Index Wednesday, 2024-01-03 »
opendevreviewMerged openstack/neutron master: [UT] OVN fake resources factory method should return instance  https://review.opendev.org/c/openstack/neutron/+/90413312:13
ralonsoh(one sec, opening the links)14:00
ralonsoh#startmeeting networking14:00
opendevmeetMeeting started Tue Jan  2 14:00:52 2024 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. 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
mlavalle\o14:00
ralonsohhello all14:00
ralonsohPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki 14:01
slaweqHi and HNY!14:01
ralonsohHNY!14:01
bcafarelo/ and HNY all!14:01
ykarelo/ HNY all !!14:01
elvirao/ hny14:02
ralonsohthis is going to be a short meeting, we'll review the calenda, bugs and on demand topics14:02
ralonsohnext week haleyb will lead the meeting again14:02
slaweq++14:02
ralonsohso let's start14:02
ralonsoh#topic announcements14:02
ralonsoh#link https://releases.openstack.org/caracal/schedule.html14:02
ralonsohmost important thing: caracal milestone 214:02
ralonsoh#link https://releases.openstack.org/caracal/schedule.html#c-214:03
ralonsohmost relevant features should be approved and under review right now14:03
ralonsohas commented 2 weeks ago, some projects have a spec freeze now14:03
ralonsohwe don't but we should not delay that14:03
ralonsohI'm guilty of not spending time last weeks on spec reviewal14:04
ralonsohbut I'll ping you in Friday for that14:04
bcafarelthat time of year it is quite allowed14:04
ralonsohand last topic in this section is the Ussuri EOL patch14:04
ralonsoh#link https://review.opendev.org/c/openstack/releases/+/90329414:04
ralonsohas commented (in the patch)14:04
ralonsohthere are some open reviews14:05
ralonsohbut I'm against merging them without the proper testing14:05
ralonsoh(backports have removed the FT)14:05
ralonsohbecause are not working, so... no14:05
ralonsohso please comment on the open patches or in the releases one14:05
ralonsohsomething else?14:06
ralonsohok, let's move on14:06
mlavallenot from me14:06
ralonsoh#topic bugs14:07
ralonsohwe have (should have) 2 reports14:07
ralonsohfrom bcafarel: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/N7UNL4EI3ZJELP5FMLVCFAONUWTLVD3J/14:07
bcafarelquite a long report :)14:07
ralonsohheheheh14:07
ralonsohthe other one is missing14:07
ralonsohbut I collected the pending LP bugs14:07
ralonsohso we have 3 to discuss today14:07
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/204710114:08
ralonsohThe test case for 'rbac_policy_quota' execution reports an error.14:08
ralonsohif I'm not wrong, there is a opne review for this one14:08
ralonsohone sec...14:08
ralonsohyes14:08
ralonsoh#link https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/90410614:08
ralonsohso I'll update the LP bug and assign it to liwejian14:08
ralonsohnext one14:09
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/204709214:09
ralonsoh"We need a way to distinguish physical_network"14:09
ralonsohto be honest, I'm not sure aboyt what the bug is requesting14:09
ralonsohI commented in c#1 with other options (policies, RBACs)14:09
ralonsohbut I have no idea what AZ means in this context14:10
ralonsohany idea?14:10
fricklerjust guessing maybe they want to use tags instead. but your question for feedback will hopefully help14:10
ralonsohbut tags are available for members, not only admins14:11
ralonsohmaybe is asking for a way of grouping the physnets14:12
ralonsohin any case, more feedback will be needed for this bug/feature14:12
fricklerit all depends on the actual use case, which is missing, ack14:12
ralonsohok, let's wait for more information from the reported then14:13
ralonsohlast bug is from previous meetings14:13
ralonsoh#link https://bugs.launchpad.net/neutron/+bug/203642314:13
ralonsoh"subnet's gateway ip can be unset while attached to router"14:13
ralonsohi14:13
ralonsohit is expired but still relevant14:13
ralonsohreviewing the code we don't handle correctly the subnet GW IP update14:14
ralonsohin the L3 agent or, as reported in the bug, in the OVS agent14:14
ralonsohso, IMO, if we are not handling this event, we should prevent that action (if the subnet is attached to a router)14:15
ralonsoh^^ do you agree?14:15
ykarel+114:16
ralonsohok, I'll take this one then. Because that implies an API change, I don't think we'll be able to backport it. And, of course, we need to document that change. I'll push a patch soon.14:17
slaweq++14:17
mtomaska+1 as well. just finished reading14:17
slaweqAnd probably some new shim API extension may be needed then14:17
ralonsohyes, exactly14:17
*** elvira1 is now known as elvira14:17
ralonsohthanks folks14:18
ralonsohthat's all I have in this topic14:18
ralonsohany other pending bug?14:18
ralonsohand as commented, last section today14:18
ralonsoh#topic on_demand14:18
ralonsoh^^ anything you want to add?14:19
mtomaskayes I have a quick one14:19
ralonsohsure14:19
mtomaskathis patch. https://review.opendev.org/c/openstack/neutron/+/903572/9 I am adding a new RPC method which already exists in RPC interfaces. One way to solve it would be to start using oslo.messaging nemaspaces. Any concern if I start doing that?14:20
mtomaskawe currently set all RPC namespaces to None https://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/constants.py#L566-L58114:20
ralonsohbut we are already using it 14:20
ralonsohnamespace=constants.RPC_NAMESPACE_DHCP_PLUGIN14:20
ralonsohright?14:20
mtomaskaNo we are not. I did that in my patch just to see if it will work. Notice the FIXME I left14:21
ralonsohno no, I mean from the base code14:21
ralonsoh            version='1.0')14:21
ralonsohsorry14:21
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/903572/9/neutron/agent/dhcp/agent.py#b86914:21
mtomaskayeah and that is None if I am reading that correctly https://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/constants.py#L57114:22
slaweqyes, all those are set to None currently14:23
ralonsohok, I'm stupid! sorry14:23
slaweqhaha14:23
mtomaskaso essentially we have one namespace None  :)14:23
ralonsohwhat method is clashing with "get_ports"?14:23
slaweqIMHO You can propose to change that and we can try to start using them14:23
mtomaskaSo I am wondering if  it would be ok to start using namespaces after all these years of just using None14:24
mtomaskametadata agent get_ports is clashing https://opendev.org/openstack/neutron/src/branch/master/neutron/agent/metadata/agent.py#L7014:24
slaweqTODO comment in that neutron-lib contants file is very old :)14:24
ralonsohright, I see 14:25
ralonsohso for now I would (1) use get_dhcp_ports and (2) enable the namespaces 14:25
mtomaskaother way to "fix it" would be to not use the same method_name 14:25
ralonsoh(2) is something that could introduce some errors14:26
ralonsohand could be a potential issue during upgrades14:26
mtomaskahmm I would be for one option or the other but not both14:26
ralonsohagents with None and server with a string14:26
slaweqright, how it will work with upgrade if we will change namespaces?14:26
ralonsohenabling the namespace names could be potentially problematic14:26
ralonsohwhen the server is upgraded, the non-updated clients will fail14:27
ralonsohall of them14:27
ralonsohbut should be tested14:27
slaweqralonsoh: this IMO means that we can't do it really14:27
ralonsohbecause the migration alternative (if that really breaks the upgade process)14:28
ralonsohis to introduce two listeners in the servers14:28
ralonsohone with and another one without namespace14:28
ralonsoh^^ during 2 releases14:29
slaweqbut then mtomaska will still need to go with 1) approach for his patch14:29
ralonsohyes 14:29
slaweqralonsoh please remember about SLURP releases :)14:29
ralonsohthat is straightforward14:29
ralonsohslaweq, C will start with the migration 14:29
ralonsohand E will keep them14:29
ralonsohyes, sorry14:29
ralonsoh2 SLURP releases14:30
ralonsohso 4 in total14:30
slaweq3 in total: C (SLURP), D (non SLURP) and E (SLURP) :)14:31
slaweqso what I propose is:14:31
slaweq1. try to use namespaces and test it with grenade jobs to see how it works14:32
mtomaska1. I did that already. see patchset 9. grenade passed14:32
slaweq2. if that will work fine we can go with this approach IMHO, if there will be problem with upgrade as we are expecting, we will need to:14:32
slaweq3. do what ralonsoh said about two listeners for now and You should then go with different name of Your callback method in Your patch mtomaska 14:33
ralonsohgrenade is updating all services at the same time14:34
mtomaskaI also need that patch for downstream 16.2 async release :) 14:34
ralonsohwe don't have updated API and old agents14:34
mtomaskaand 17.1.314:34
slaweqmtomaska You changed that namespace for DHCP agent only14:34
slaweqand in grenade job this agent is only running on controller which is updated14:34
mtomaskaslaweq, o ok, you mean change all namespaces14:35
slaweqon "compute" node, which is not upgraded to new version there is ovs agent running14:35
slaweqmtomaska yes, You will need to change at least namespace used by the OVS agent to test with grenade job 14:35
slaweqand/or metadata agent14:35
ralonsohexactly14:35
slaweqand L3 agent14:36
slaweqthose 3 are running in compute1 node: https://7b4153b100b6f570d4fc-1ca4271300c9fb8c2811a73921fc79fe.ssl.cf2.rackcdn.com/903572/9/check/neutron-ovs-grenade-multinode/d255232/compute1/logs/index.html14:36
slaweqbut not DHCP agent unfortunately14:36
ralonsohIMO, for your patch to be merged (and backportable is D/S), the method should be just "get_dhcp_ports"14:37
ralonsohthe RPC namespace should be another LP bug14:37
ralonsohthis is not trivial14:37
mtomaskayes, a simple change is snowballing :) 14:38
ralonsohperfect! thanks slaweq for the grenade information14:38
ralonsohI'll open a RFE to cover the RPC namespace changes14:38
mtomaskaok. Let me rename it to get_dhcp_ports . Then I will document what we discussed here in a LP 14:38
ralonsohmtomaska, will you open this LP bug?14:39
mtomaskayes14:39
ralonsohperfect14:39
mtomaskathank you for the info all14:39
ralonsohanything else you want to bring to the on demand section?14:39
ralonsoh2 heads-up14:40
ralonsohin 20 mins we have the CI meeting here in this channel14:40
ralonsohand I forgot the bug deputies14:40
ralonsohThis week elvira is the bug deputy, next week will be slaweq14:40
ralonsohack ^?14:40
slaweqsure14:41
ralonsohI'll ping elvira offline14:41
ralonsohthank you all, see you14:41
ralonsoh#endmeeting14:41
opendevmeetMeeting ended Tue Jan  2 14:41:35 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:41
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-02-14.00.html14:41
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-02-14.00.txt14:41
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-02-14.00.log.html14:41
elviraralonsoh: ack!14:44
ralonsohthanks14:44
ykarel#startmeeting neutron_ci15:01
opendevmeetMeeting started Tue Jan  2 15:01:35 2024 UTC and is due to finish in 60 minutes.  The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
opendevmeetThe meeting name has been set to 'neutron_ci'15:01
mlavalle\o15:01
ralonsohhi15:01
ykarelping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira15:01
bcafarelo/ again15:01
ykarel This will be video meeting today: https://meetpad.opendev.org/neutron-ci-meetings15:02
ykarel#topic Actions from previous meetings15:03
ykarelralonsoh to cleanup test_get_datapath_id15:03
ykarel#link https://review.opendev.org/c/openstack/neutron/+/90407615:03
ykarelykarel to check failures in test_agent_resync_on_non_existing_bridge15:04
ykarelhttps://64984dc0bc4d6446d774-b4e19c98e06eb23bd3ecd1208eab0884.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/stable/wallaby/neutron-functional/35cecc0/testr_results.html15:09
ykarelhttps://64984dc0bc4d6446d774-b4e19c98e06eb23bd3ecd1208eab0884.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/stable/wallaby/neutron-functional/35cecc0/controller/logs/dsvm-functional-logs/neutron.tests.functional.agent.ovn.metadata.test_metadata_agent.TestMetadataAgent.test_agent_resync_on_non_existing_bridge/testrun.txt15:09
ykarelneutron.privileged.agent.linux.ip_lib.NetworkInterfaceNotFound: Network interface tapovn-780d921 not found in namespace ovnmeta-ovn-780d924a-eef9-4b4c-a6d8-e128f36d9b87.15:09
slaweqhttps://opendev.org/openstack/neutron/src/branch/master/neutron/privileged/agent/linux/ip_lib.py#L21015:10
ralonsohhttps://review.opendev.org/q/Ie195ad596fd148708fc30946bde964d52444afee15:11
ykarel#action ralonsoh to check and propose fix for handling ip route commands handling15:12
ykarelykarel to check/confirm if issue is same as 2007166 and push backports to skip test15:13
ykarel#link https://review.opendev.org/q/topic:%22bug/2007166%22+branch:stable/2023.115:13
ykarel#topic Stable branches15:13
ykarelall good15:13
ykarel#topic Stadium projects15:14
ykarelall green15:14
ykarel#topic Rechecks15:14
ykarel#topic Periodic15:15
ykarelhttps://zuul.openstack.org/builds?job_name=neutron-ovn-tripleo-ci-centos-9-containers-multinode&project=openstack/neutron15:15
ralonsoh+1 to remove the Wallaby centos9 job15:16
ykarel#action ralonsoh to send patch to drop tripleo jobs from wallaby15:17
ykarel#topic Grafana15:17
ykarelhttps://grafana.opendev.org/d/f913631585/neutron-failure-rate15:17
ykarel#topic On Demand15:18
ykarel#endmeeting15:34
opendevmeetMeeting ended Tue Jan  2 15:34:57 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:34
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-02-15.01.html15:34
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-02-15.01.txt15:34
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-02-15.01.log.html15:34
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [stable-only] Remove the "neutron-ovn-tripleo-ci-centos-9-containers-multinode" job  https://review.opendev.org/c/openstack/neutron/+/90449815:40
ykarelralonsoh, i think can drop tripleo.yaml fully i.e all those jobs15:41
ralonsohright15:41
ralonsohI'll push a new ps15:41
ykarelthx15:41
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [stable-only] Remove all TripleO CI jobs  https://review.opendev.org/c/openstack/neutron/+/90449815:46
opendevreviewMerged openstack/neutron stable/2023.1: [ovn][ipv6] Skip test_update_router_admin_state  https://review.opendev.org/c/openstack/neutron/+/90416715:46
opendevreviewMerged openstack/neutron stable/2023.1: [ovn][ipv6] Add some more tests to skiplist  https://review.opendev.org/c/openstack/neutron/+/90416815:46
opendevreviewMerged openstack/neutron stable/2023.1: Remove obsolete PID files before start  https://review.opendev.org/c/openstack/neutron/+/90314916:29
opendevreviewMerged openstack/neutron stable/zed: [OVN] DB sync host/physnet - filter on agent_type  https://review.opendev.org/c/openstack/neutron/+/90368516:31
opendevreviewMiro Tomaska proposed openstack/neutron master: DHCP agent get ovn dhcp port from server  https://review.opendev.org/c/openstack/neutron/+/90357216:38
opendevreviewMiro Tomaska proposed openstack/neutron stable/2023.1: Metadata: handle process exceptions  https://review.opendev.org/c/openstack/neutron/+/89377517:35
ralonsohmtomaska, https://review.opendev.org/c/openstack/neutron/+/903572/10/neutron/agent/dhcp/agent.py#93019:44
ralonsohif you change that, I'll +2 the patch19:45
mtomaskaack. Ill do it right now19:48
mtomaskathanks19:48
opendevreviewMiro Tomaska proposed openstack/neutron master: DHCP agent get ovn dhcp port from server  https://review.opendev.org/c/openstack/neutron/+/90357219:51
ralonsoh+219:52
« Sunday, 2023-12-31 Index Wednesday, 2024-01-03 »

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