Monday, 2024-11-18

opendevreviewTakashi Kajinami proposed openstack/neutron master: Drop reference to removed services  https://review.opendev.org/c/openstack/neutron/+/93547603:58
opendevreviewTakashi Kajinami proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536806:10
opendevreviewTakashi Kajinami proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536806:14
opendevreviewTakashi Kajinami proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536807:03
ralonsohtkajinam, hi! I was checking the same07:28
ralonsohis that image enough? has everything needed to run frr?07:28
tkajinamquay.io/frr:9.0.0 is the image used in integration tests of os_ken07:31
tkajinamso I believe it is enough07:31
ralonsohtkajinam, it is failing in zuul07:31
ralonsohtkajinam, ykarel, lajoskatona I'm going to limit os-ken in neutron-tempest-plugin until this issue is fixed07:36
tkajinamugg07:36
ralonsohwe can update the os-ken version in the patch fixing it07:36
tkajinamralonsoh, you should downgrade it in global upper constraints07:36
ralonsohwe can, for now, only limit it in n-t-p07:36
tkajinamI don't know if that works because all libraries are installed according to u-c07:36
tkajinamif you cap the version then it fails because of inconsistency between local req vs global u-c07:37
ralonsohI'll try, I prefer to keep this local07:37
ralonsohbut no one else is enforcing a newer version of os-ken07:37
tkajinamok07:37
tkajinamhmmm os_ken.tests.integrated.common.docker_base.CommandError yeah it fails07:37
ralonsohso I prefer to keep it local to neutron (for now)07:37
ralonsohwe need better error description there07:38
tkajinamyeah I don't know why it does not show the actual output07:38
tkajinamI have to leave for some time but will come back later to look into it07:38
lajoskatonaack07:38
lajoskatonaI just check the last logs07:38
ykarelyes we need to pin in upper-constarints if we want to go with limit07:38
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Limit temporarily the os-ken version to <2.11.0  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536607:39
lajoskatonaThe docker stderr is this if I understand well this causes the exception: "stderr: Unable to find image 'quay.io/frr:9.0.0' locally"07:40
ralonsohlajoskatona, this is because we need to run docker build first, I think07:41
lajoskatonatkajinam: ---^07:41
ralonsohhttps://docs.frrouting.org/projects/dev-guide/en/latest/building-docker.html07:45
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536807:55
ykarelralonsoh, also getting u-c pin ready with https://review.opendev.org/c/openstack/requirements/+/935485 08:07
opendevreviewLajos Katona proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536808:11
ralonsohlajoskatona, hey, I pushed a PS before you08:13
ralonsohykarel, let's wait for the patch08:13
ykarelralonsoh, yes just getting it ready as an alternative08:13
ralonsohfor sure, thanks!08:14
ykarelas seems other patch will need more adjustment08:14
ykarellike for stable versions of the jobs08:14
ralonsohykarel, but the stable versions should not retrieve os-ken 2.1008:16
ralonsoh2.1108:17
ykarelbut frr to quoga rename in test will be impacted08:17
ralonsohthe problem will be with the older n-t-p versions, that won't have FRR08:17
ykareli meant for :from os_ken.tests.integrated.common import frr only relevant for >=2.11.008:18
ralonsohyes08:18
ykarelso this will not work with stable versions of the jobs, let's wait for the results08:18
ralonsohshould we pin n-t-p? because so far the current code won't work with 2024.2 and older verions08:18
ralonsohright08:18
ykarelwe normally keep n-t-p unpinned until we transition to unmaintained08:19
ykarelso was thinking if we can avoid the pin08:20
ralonsohI think we should bring back quaga in os-ken just for older versions08:20
ralonsohwith a warning message08:20
ralonsohthat would be the best way, IMO08:20
ykarelyes and will require other os-ken release08:21
ralonsohok, on my way!08:21
ralonsohI'll push a os-ken patch08:21
ykarelthx08:21
lajoskatonaralonsoh: sorry, I checked 1 minute before, and it seems was too late...08:21
ralonsohlajoskatona, no problem, your version uses quai and is newer08:21
*** dmellado0755391 is now known as dmellado07553908:34
*** elodilles is now known as elodilles_pto08:37
ykarelralonsoh, ohkk also since it's tempest venv, master constraints being used so all stable branches also impacted08:41
ykarelso seems 935368 may work for stable too from tempest perspective08:42
ralonsohperfect then08:42
ralonsohin any case, it is not recommended to change the testing method in stable branches08:42
ykarelyes +1 to ^, as will impact distributions like RDO09:02
ralonsohlajoskatona, I'm going to update the patch09:05
ralonsoh2024-11-18 08:58:23.501 90250 ERROR os_ken.tests.integrated.common.docker_base [-] stderr: /usr/bin/pkill: unrecognized option: S09:05
ralonsohgive me a couple of mins09:05
lajoskatonaralonsoh: ack09:07
opendevreviewRodolfo Alonso proposed openstack/os-ken master: Change the order and name of the pkill signal  https://review.opendev.org/c/openstack/os-ken/+/93549009:36
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Change references from Quagga to FRR  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93536809:38
ykarelralonsoh_, so not going with: "bring back quaga in os-ken just for older versions"?10:04
*** ralonsoh_ is now known as ralonsoh10:04
ykarelif yes we can get things unblocked first with requirements pin https://review.opendev.org/c/openstack/requirements/+/93548510:04
ykareland have more time to fix things10:05
ralonsohykarel, yes, I'll try it now. But we need also to change the n-t-p code10:05
ralonsohykarel, before merging the requirements patch, let's wait for the last execution10:05
ykarelalso i doubt if depends-on os-ken will work10:05
ykarelfor the tempest bits10:06
ralonsohyes because the job has this project in the dependencies10:06
ralonsohykarel, ok no, it overwritten with other version10:07
ralonsohit is*10:07
opendevreviewyatin proposed openstack/neutron unmaintained/xena: [Xena Only] exclude dhcp tests in ovn jobs  https://review.opendev.org/c/openstack/neutron/+/93549710:26
ralonsohlajoskatona, ykarel I'm deploying a ndr env, but the tests are now passing, at least the setup resources phase, with https://review.opendev.org/c/openstack/os-ken/+/93549011:06
ralonsohlocally, of course11:06
ralonsoheg: https://paste.opendev.org/show/bOrl9iDS3wn586RKX2jG/11:08
ykarelralonsoh, ack11:51
opendevreviewRodolfo Alonso proposed openstack/os-ken master: Remove frr.conf config lines with errors  https://review.opendev.org/c/openstack/os-ken/+/93550411:53
lajoskatonaralonsoh: ack, thanks11:56
opendevreviewRodolfo Alonso proposed openstack/os-ken master: Revert "Use FRR instead of Quagga when testing"  https://review.opendev.org/c/openstack/os-ken/+/93550511:58
ralonsohso let's revert the patch and then provide both implementations, making them selectable 11:58
ralonsohonce FRR is used and tested in n-d-r, we can remove the Quagga one, if needed11:59
ralonsohfolks: https://review.opendev.org/c/openstack/os-ken/+/93550513:34
ralonsohthen we need to block 2.11.0 and 2.11.1 in requirements13:34
opendevreviewRodolfo Alonso proposed openstack/os-ken master: Remove frr.conf config lines with errors  https://review.opendev.org/c/openstack/os-ken/+/93550413:35
toreAnyone know how the "extra properties" feature are supposed to be used? I need to add some site-specific metadata to network objects, and from the CLI help description, --extra-property seems like a good fit. However I can't quite get it to work:13:40
tore$ openstack network set --extra-property type=int,name=customerid,value=42 provnet13:40
toreBadRequestException: 400: Client Error for url: https://openstack/api/network/v2.0/networks/90e3fc3a-edfb-41a3-93fc-e779602cf4a3, Unrecognized attribute(s) 'customerid'13:40
toreIs it necessary to configure a list of recognised extra properties somewhere?13:41
ralonsohtore, you can modify the resource API parameters, not new ones13:48
ralonsohin a network, for example, what you see in "openstack network show"13:48
toreHm. Okay. It does seem like all the named parameters shown there have dedicated set commands, e.g., openstack network set --description foobar in order to set 'description', and it seems implied that --extra-property is to set attributes that are, well, extra - in addition to those shown by default.13:50
torebut you're right, "openstack network set --extra-property type=str,name=description,value=foobar123 provnet" seems to do the same thing as "openstack network set --description foobar123 provnet", so I guess I'm just confused as to how that would be an "extra" property13:53
slaweqtore this "extra-property" was added to cover case where 3rd party plugins have some additonal attributes which we don't know in neutron14:59
slaweqneutron client had support for this "just like that" but in osc we had to add something what would allow to pass such unknown attributes to the neutron server and this is exactly for that15:00
slaweqykarel ralonsoh do we have ci meeting today?15:02
mlavalledo we have CI meeting today?15:02
ykarelslaweq, yes. starting it now, got a bit late15:03
ykarel#startmeeting neutron_ci15:03
opendevmeetMeeting started Mon Nov 18 15:03:19 2024 UTC and is due to finish in 60 minutes.  The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot.15:03
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:03
opendevmeetThe meeting name has been set to 'neutron_ci'15:03
ykarelPing list: bcafarel, lajoskatona, slawek, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira15:03
mlavalleykarel: video or irc?15:03
slaweqykarel sure :)15:03
slaweqo/15:03
mlavalle\o15:03
ykarelIRC15:03
haleybo/15:04
bcafarelo/ but in another meeting15:04
ralonsohhi15:04
ykarelhello everyone, let's start with the topics15:05
ykarel#topic Actions from previous meetings15:05
ykarelralonsoh to check cleanup issue in test_dvr_ha_router_interface_mtu_update15:05
ralonsohI filled a LP bug and working on it15:05
ykarel#link https://bugs.launchpad.net/neutron/+bug/208815415:06
ykarel^ right?15:06
ralonsohyes, I was looking for that15:06
ralonsohexactly15:06
ykarelthx ralonsoh 15:06
ykarelralonsoh to open bug for sdk quota zone failure issue15:07
ralonsohworking on it, I have a patch (that is not working(15:07
ralonsohhttps://review.opendev.org/c/openstack/openstacksdk/+/93476215:07
ralonsohI need to check how to retrieve the dns.quota15:07
ralonsohbut low priority, is a random issue15:07
ykarelthx ralonsoh 15:07
ykarelykarel to check for failure in test_no_value_set_for_other_config_raises_exception15:08
ykarelBased on the symptoms Looks similar to https://bugs.launchpad.net/neutron/+bug/196174015:08
ykarelas can see interface getting added/deleted/added quickly15:08
lajoskatonao/15:08
ykarelykarel to log bug for ovn sync test failures in functional tests15:09
ykarelOpened https://bugs.launchpad.net/neutron/+bug/208842315:09
ykarelslawek to check https://bugs.launchpad.net/neutron/+bug/208360915:09
ykarelfor this we have some recent updates15:13
ykarelhttps://review.opendev.org/c/openstack/neutron/+/93496015:13
slaweqI was checking that recently15:13
ykarelhttps://review.opendev.org/c/openstack/neutron/+/93467715:13
slaweq I described in https://bugs.launchpad.net/neutron/+bug/2083609/comments/10 what I have found 15:14
slaweqI don't know anything else and I have no idea if we can somehow fix it really15:15
ykarelme neither :(15:16
ykarelthx slaweq for looking into it, if anyone has idea on this please have a look and update the bug15:16
ykarel#topic Stable branches15:17
ykarelstable branches are impacted by os-ken release https://bugs.launchpad.net/neutron/+bug/208828515:17
ykarelbcafarel, anything to add ?15:17
bcafarelsome back-and-forth in the reviews for 2023.1 unmaintained transition, but nothing bad15:18
lajoskatonawith os-ken what is the full solution? I saw patches from ralonsoh (thanks for it)15:19
bcafarelRodolfo backported FT fix https://review.opendev.org/q/I18cfb18abe481f47db870f210188e1a570844077 , should make stable CI even greener15:19
ykarelyes 2023.1 CI also kind of broken with some projects yet to transition to unmaintained15:19
ralonsohwe are revirting the patch in os-ken15:19
ralonsohif needed, we can propose a FRR container but not remoiving the quagga one15:19
lajoskatonaahh, ok, but we dont have to backport the frr patches?15:19
ralonsohand make it configurable15:20
ralonsohno15:20
lajoskatonaralonsoh: ack, let's have both than15:20
ykarellajoskatona, to unblock CI we going with pin https://review.opendev.org/c/openstack/requirements/+/93548515:20
ykareland then we can have os-ken revert + release + u-c update15:21
ykarelor can also check the proposals from ralonsoh before the release15:21
ykareli.e to have something compatible with stable branches15:21
ralonsohand we have the broken releases blocked --> os-ken!=2.11.0,!=2.11.115:21
ralonsohhttps://review.opendev.org/c/openstack/requirements/+/935485/2/global-requirements.txt#35615:21
opendevreviewRodolfo Alonso proposed openstack/neutron master: Replace ``ReaderWriterLock`` with ``threading.RLock``  https://review.opendev.org/c/openstack/neutron/+/93552415:23
ykarelk that's it for stable15:23
ykarel#topic Stadium projects15:23
ykarelweekly run red for all stadium projects15:23
ykareltempest jobs with os-ken release https://bugs.launchpad.net/neutron/+bug/208828515:24
ralonsohpfffff15:24
ykarelunit test jobs broken by https://review.opendev.org/c/openstack/neutron/+/93419015:24
ykarelfor tempest one we should have that soon unblocked15:24
ykarelfor unit test not checked if we have the patches open15:25
ykarellajoskatona, anything else to add for stadiums?15:25
lajoskatonayeah that is due to the os-ken issue15:25
lajoskatonanothing other than those ones15:25
ykarelfor unit test ones will be need another neutron release? or just update in stadium to point to new location will be enough?15:26
lajoskatonagood question15:27
ralonsohI think they have neutron as dependency and use master15:27
ykarelfrom neutron.tests.unit.db import test_db_base_plugin_v2 -> from neutron.tests.common import test_db_base_plugin_v215:27
ralonsohso it shouldn't be needed15:27
ykarelok then just the fixes on all stadium is needed15:28
ykarelany volunteer to send those patches?15:28
ralonsohI can do it15:28
lajoskatonaI can15:28
ralonsohlajoskatona, ok, all yours, I'm still fixing other issues15:28
ralonsohthanks!!15:28
lajoskatona:-)15:29
ykarel#action lajoskatona to send stadium fixes for test_db_base_plugin_v2 renames15:29
ykarelthx lajoskatona ralonsoh 15:29
ykarel#topic Rechecks15:29
ykarelwe still have some rechecks with known issues15:30
ykarelgood bare recheks down this week, 2/27. Let's keep avoiding bare rechecks15:30
ykarel#topic fullstack/functional15:30
ykareltest_update_minimum_bandwidth_queue15:31
ykarelhttps://613acde820ad3e40c738-ebad28ee85ce63233d9ad88daf84e2e7.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-sqlalchemy-master/646ee8a/testr_results.html15:31
ykarelfound an old bug https://bugs.launchpad.net/neutron/+bug/202574015:31
ykarelwhere these tests were transitioned to serial but we still hitting it 15:31
ykarelnot frequently though15:31
ykareltest_non_existing15:32
ykarelhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_22b/934652/3/gate/neutron-functional-with-uwsgi/22bd251/testr_results.html15:32
ykarelseen this also once15:32
ykarelseems due to some slowness on pg to be created and ready when it was looked15:33
ralonsohtest_non_existing --> I think we can add eventwaits15:33
ralonsohsame as in other OVN tests15:33
ralonsohlet me check this one15:34
ykarelyes +115:34
ralonsohI'll create a LP bug15:34
ykarelralonsoh, to open lp for test_non_existing15:34
ralonsohabout the min-qos, because I wrote the code and the test, I'll check it too15:34
ykarel#action ralonsoh to open lp for test_non_existing ft tests15:34
ykarelralonsoh, thx15:35
ykarel#action ralonsoh to check for test_update_minimum_bandwidth_queue15:35
opendevreviewMerged openstack/os-ken master: Revert "Use FRR instead of Quagga when testing"  https://review.opendev.org/c/openstack/os-ken/+/93550515:35
ykarelralonsoh, both these are seen once recently, so can do check in lower priority15:35
ykareleven in later weeks, so not to hurry15:35
ralonsohperfect15:35
ykarel#topic Tempest/Scenario15:36
ykarel#info os-ken issue https://bugs.launchpad.net/neutron/+bug/2088285 impacting most of the branches15:36
ykarelneutron-ovn-tempest-full-multinode-ovs-master failed thrice during last week where many tests failed as neutron api fails to respond in 90 seconds, last three run passed though 15:36
ykarelhttps://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-full-multinode-ovs-master15:36
ykareljust raising here to follow up next week, as latest runs are good15:37
ykareland don't know if there were anything that broke and fix it15:37
ykarelso will watch for next runs15:37
ykarel#topic Periodic15:38
ykarelunmaintained/xena needs https://review.opendev.org/c/openstack/neutron/+/93549715:38
ykarelplease review ^15:38
ykarelunmaintained/2021 needs https://review.opendev.org/c/openstack/ironic/+/93403615:38
ykarelbut now 2021 is blocked as not all projects transitioned to unmaintained, and default branch not switched to unmaintained15:39
ykarel#topic Grafana15:39
ykarelhttps://grafana.opendev.org/d/f913631585/neutron-failure-rate15:39
ykarellet's have a quick look here too15:40
ykarellooks normal as we can see some spikes in gate for known issues15:42
slaweqNeutron-tempest-plugin jobs in check queue looks like totally broken15:43
ykarelyeap that's the os-ken issue15:43
slaweqahh, ok15:43
ykarel#topic On Demand15:43
ykarelanything else you would like to raise?15:43
ralonsohyes15:43
ralonsohhttps://review.opendev.org/c/openstack/releases/+/93552715:44
ralonsohthat's all15:44
slaweq+1 from me15:44
lajoskatona+1 , 15:44
fricklerralonsoh: is that fixing all the neutron issues? I can likely single-approve then15:45
ralonsohfrickler, yes15:45
fricklerok, will approve once CI is done, thx15:45
ykarelthx everyone for joining15:45
ykarel#endmeeting15:45
opendevmeetMeeting ended Mon Nov 18 15:45:58 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:45
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-11-18-15.03.html15:45
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-11-18-15.03.txt15:45
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-11-18-15.03.log.html15:45
lajoskatonao/15:46
ralonsohbye15:46
slaweqo/15:46
mlavalle\o15:47
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add Octavia OVN DBs sync cmd  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92574716:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add octavia_client with openstacksdk  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93069216:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add LB sync logic  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92532416:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add Listener sync logic  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93125016:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add Pool sync logic  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93126616:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add Member sync logic  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93126716:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add Health monitor sync logic  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/93128816:05
opendevreviewRico Lin proposed openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903916:05
ricolinslaweq:  patch set update with wording ^^^16:08
slaweqricolin thx, I will check them tomorrow morning my time16:10
ricolinWould be gratefully if we can have more eyes on that patch set, it's a large one, but really nice improve. 16:10
ricolinslaweq:  thank you!16:10
opendevreviewMerged openstack/neutron unmaintained/xena: [Xena Only] exclude dhcp tests in ovn jobs  https://review.opendev.org/c/openstack/neutron/+/93549717:21
opendevreviewIhar Hrachyshka proposed openstack/neutron master: functional: Don't assume privsep-helper is installed in virtualenv  https://review.opendev.org/c/openstack/neutron/+/93525118:20
noonedeadpunkslaweq: if you're checking octavia provider for OVN, pls take a look at https://review.opendev.org/c/openstack/ovn-octavia-provider/+/933879 as well :D18:26
*** dmellado0755393 is now known as dmellado07553922:09
opendevreviewMerged openstack/neutron stable/2024.2: Set distributed flag to NB_Global  https://review.opendev.org/c/openstack/neutron/+/93520523:52
opendevreviewMerged openstack/neutron stable/2024.1: Set distributed flag to NB_Global  https://review.opendev.org/c/openstack/neutron/+/93520623:52
opendevreviewMerged openstack/neutron stable/2023.2: Set distributed flag to NB_Global  https://review.opendev.org/c/openstack/neutron/+/93520723:52

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