Tuesday, 2022-02-08

*** pojadhav|afk is now known as pojadhav02:36
*** jpena|off is now known as jpena08:03
jm1Hello :) Atm devstack has issues with reboots, e.g. swift and cinder will fail (kind of) because their file based storage wont be recreated on boot. Is this by intention/design?08:21
jm1If not, maybe we should fix this? E.g. https://review.opendev.org/c/openstack/devstack/+/82791808:22
*** akekane_ is now known as abhishekk08:31
fricklerjm1: I'd say we don't intentionally break on reboot, but we also don't test that scenario, so it will be difficult to detect regressions, in particular for non-default scenarios. but I think patches to improve things should be welcome09:08
*** lifeless_ is now known as lifeless09:24
opendevreviewLiZekun proposed openstack/hacking master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/hacking/+/82825609:34
opendevreviewLiZekun proposed openstack/hacking master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/hacking/+/82825609:35
*** chaconpiza__ is now known as chaconpiza09:42
jm1frickler: hmmm... to catch regressions, couldn't we/I add a reboot to the zuul playbooks just at the end of the devstack setup?10:23
fricklerjm1: I don't think we would want to do this for every job, but maybe a dedicated job that does this and runs tempest afterwards would be feasible. you may want to look at the fips jobs for how to reboot and reconnect with zuul again10:38
jm1frickler: ok, will first fix cinder/lvm else it is broken from the start ^^10:39
opendevreviewJakob Meng proposed openstack/devstack master: Made LVM backing disk persistent  https://review.opendev.org/c/openstack/devstack/+/82828011:07
jm1submitted a patch against devstack which fixes reboot issues with LVM backing files. https://review.opendev.org/c/openstack/devstack/+/82828011:10
opendevreviewJakob Meng proposed openstack/devstack master: Added Zuul job devstack-reboot  https://review.opendev.org/c/openstack/devstack/+/82828912:06
*** pojadhav is now known as pojadhav|afk12:33
*** dmellado_ is now known as dmellado13:20
*** pojadhav|afk is now known as pojadhav13:31
*** Guest2040 is now known as spotz14:37
opendevreviewJakob Meng proposed openstack/devstack master: Added Zuul job devstack-reboot  https://review.opendev.org/c/openstack/devstack/+/82828914:47
opendevreviewAttila Fazekas proposed opendev/elastic-recheck master: Revert "Add query for WARN: failed: route add -net bug 1857151"  https://review.opendev.org/c/opendev/elastic-recheck/+/82835514:52
*** akaha|rover is now known as akahat|dinner14:55
kopecmartin#startmeeting qa15:00
opendevmeetMeeting started Tue Feb  8 15:00:19 2022 UTC and is due to finish in 60 minutes.  The chair is kopecmartin. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'qa'15:00
kopecmartin#link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Weekly_QA_Team_meeting15:00
kopecmartinagenda ^^15:00
gmanno/15:00
kopecmartinhi gmann, let's start15:02
kopecmartin#topic Announcement and Action Item (Optional)15:02
kopecmartinwe have PTL elections coming up15:02
kopecmartin#link https://governance.openstack.org/election/15:03
gmannyeah, nomination is open now until 15th Feb15:03
fricklerkopecmartin: do you want to run again?15:03
fricklerI would support that15:03
gmannI hope so :)15:03
kopecmartini wanted to ask first whether there is anyone who would like to run for qa PTL15:04
gmannI too support for sure15:04
gmannnot me15:04
alexYefimov_kopecmartin ++15:04
kopecmartinif not, then yes, i'd like to run again15:04
gmannthanks kopecmartin 15:05
frickler+115:05
kopecmartinthanks :) 15:05
kopecmartin#topic Yoga Priority Items progress15:06
kopecmartin#link https://etherpad.opendev.org/p/qa-yoga-priority15:06
kopecmartinany updates on the priority items?15:06
gmannon RBAC, I am trying to finish the new policy change in Nova first and then will go through the devstack required changes based on what new default nova end up with15:06
kopecmartinregarding FIPS, we've merged some patches , i need to check what else is needed15:07
gmanntwo nodeset for centos stream is also merged15:07
gmannthere are multinode job adding patch but that is failing somewhere. I commented on that15:07
kopecmartingmann: thanks 15:08
kopecmartinregarding rbac, let me know if i can do anything15:08
kopecmartini'd like to get to the point where we can see where tempest stands on a s-rbac env15:08
gmannyeah, policy defaults are again changing on service side so tempest needs to hold it until they are ready15:09
fricklerthere's also this fix for neutron system scope, which needs reviews https://review.opendev.org/c/openstack/devstack/+/82685115:09
gmann keystone is already ready, nova will be in Yoga release along  with other like neutron or so15:10
gmannfrickler: ack, will check15:10
kopecmartinthat one is on my list too15:11
fricklerfor keystone the changes in devstack are also still pending, I just rechecked two, the next needs rebasing15:11
gmannyeah, i saw those. waiting for the gate result15:12
gmannalso we need to move those setting from keystone if it work before test phase. need to check my old patch which was failing earlier 15:13
kopecmartinCleanup of duplicated scenario.manager - one of our interns is working on this, a good task for initial onboarding 15:14
gmann+1, nice15:14
kopecmartinand i have no updates for the rest of the items15:14
kopecmartinmoving on15:15
kopecmartin#topic OpenStack Events Updates and Planning15:15
kopecmartinwe'll start gathering ptg topics once the Z release is named15:15
kopecmartinshould be done anytime soon ,right?15:15
fricklerIIUC it depends on trademark checking15:16
kopecmartingood, thanks15:16
gmannkopecmartin: yeah, 1 more week and we will be ready.15:17
kopecmartinbtw, team registration is open, i have in my TODO, i'll fill the registration form and will follow up with a doodle to find the best time slots for our sessions15:17
kopecmartingmann: perfect15:17
gmann+115:17
kopecmartin#topic Gate Status Checks15:17
kopecmartin#link https://review.opendev.org/q/label:Review-Priority%253D%252B2+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking)15:17
gmannkopecmartin: I will also schedule TC+PTL sessions early PTG week, just to note for scheduling QA sessions. 15:17
fricklerF35 I approved before the meeting15:18
fricklerand notified neutron+nova15:18
kopecmartingmann: ack15:18
kopecmartinfrickler: thanks15:18
gmannfrickler: do you know centos-stream8 is stable now, so that we can again make the jobs voting ?15:19
fricklerI saw a couple of failures in the F35, so IMO it is not stable15:19
gmannthey are not stable seems https://zuul.openstack.org/builds?job_name=tempest-integrated-compute-centos-8-stream&job_name=tempest+tempest-full-py3-centos-8-stream&skip=015:19
fricklerin the F35 change15:19
gmannfrickler: yeah15:20
fricklerand centos-9-stream is hard broken with some attachment failures15:20
fricklervery likely some libvirt/qemu version issue with nova15:21
fricklersomeone was looking at that, but I lost track whether there was any progress15:21
fricklergmann: maybe you can pick that up in the nova meeting?15:22
gmannk15:22
gmannsure15:22
kopecmartinbtw, it seems we broke neutron-tempest-with-uwsgi job15:22
kopecmartin#link https://bugs.launchpad.net/neutron/+bug/196002215:22
kopecmartinwith 15:22
kopecmartin#link https://review.opendev.org/c/openstack/tempest/+/81408515:22
kopecmartini'm looking into it , i'm failing to install the env locally15:23
kopecmartinwill try again , after that I'll leave it up to zuul and move the experiments on how to fix that to the WIP reviews 15:24
fricklerkopecmartin: let me know if you want a help node for debugging15:24
fricklerheld15:24
*** pojadhav is now known as pojadhav|dinner15:24
kopecmartinfrickler: i have an internal openstack where i can spawn one .. the problem i have is with the installation, it fails for various reasons 15:25
kopecmartinat least it gives me more insights into devstack 15:25
gmannkopecmartin: so it is failing because we are waiting for it ? 15:25
kopecmartinit looks like that, although i doesn't make sense to me right now, however, if would be too big coincidence if that review is not related15:26
kopecmartinthe job started falling after the review got merged15:26
gmannhumm15:27
kopecmartin#topic Periodic jobs Status Checks15:28
fricklerif it is 100% failure, testing against a revert should be pretty easy to confirm that?15:28
kopecmartinfrickler: yup, i can propose a dnm patch just to confirm that that's the real culprit 15:28
kopecmartin#link https://zuul.openstack.org/builds?job_name=tempest-full-xena-py3&job_name=tempest-full-wallaby-py3&job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&job_name=tempest-full-train-py3&pipeline=periodic-stable15:29
kopecmartin#link https://zuul.openstack.org/builds?project=openstack%2Ftempest&project=openstack%2Fdevstack&pipeline=periodic15:29
fricklerfor the periodic jobs, I noticed there isn't a xena one15:29
gmannbut it seems failing only on victoria and ussuri ?15:29
fricklerand train also no longer runs, so it can be removed from that query15:29
kopecmartinoh15:29
gmannfrickler: ohk, did I miss that15:29
kopecmartingmann: it seems the job is running only on victoria and ussuri 15:30
gmannkopecmartin: ohk15:30
gmannstable jobs are up to date https://github.com/openstack/tempest/blob/master/zuul.d/project.yaml#L16615:31
frickleroh, full-xena, not -py315:31
fricklerthen just the query above is wrong15:31
kopecmartinyeah, my bad15:32
gmann#link https://zuul.openstack.org/builds?job_name=tempest-full-xena&job_name=tempest-full-wallaby-py3&job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&pipeline=periodic-stable15:32
gmann^^ this one we can use15:32
gmannyeah I removed the py3 from xena onwards as everything is py315:32
kopecmartini've fixed the link in the agenda15:33
gmannthanks15:33
kopecmartinfrickler: good catch!15:33
kopecmartinanyway, periodic jobs seem fine15:34
kopecmartin#topic Sub Teams highlights15:34
kopecmartinChanges with Review-Priority == +115:34
kopecmartin#link https://review.opendev.org/q/label:Review-Priority%253D%252B1+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking)15:34
kopecmartini see 2 reviews, will try to check them soon15:34
kopecmartin#topic Open Discussion15:35
kopecmartinanything for the open discussion?15:35
fricklerthere was this idea proposed of doing reboot checks for devstack15:35
fricklerwhich might be useful it we get it to work in a stable way15:36
fricklerbut let me know if you disagree, then we can stop investing work15:37
kopecmartininteresting, what would it be about exactly? restarting the env after deployment and checking whether is up and running?15:37
frickleryes, my idea would be "run devstack", "reboot", "run tempest"15:37
fricklerthe reboot part is happing in the fips jobs anyway, but before devstack15:38
fricklerbut this way rebooting during a CI run isn't a new thing anymore15:39
gmannwhat is it about to catch/test ?15:39
kopecmartinor maybe , devstack, tempest, reboot, some subset of tempest? the subset can grow after it's stable15:39
fricklercurrently it fails because swift and cinder loopback setup isn't rebootsafe15:39
gmannand as separate job right not by default in all devstack jobs?15:39
fricklersee https://review.opendev.org/c/openstack/devstack/+/82828015:40
frickleryes, I would want a dedicated job15:40
fricklerjm1 may even be still around who proposed this15:41
frickleranyway, we don't need a long discussion right now, just wanted to raise awareness for this idea15:42
jm1https://review.opendev.org/c/openstack/devstack/+/82828915:42
kopecmartinthat's interesting, i don't see a reason why not to try that in one separate job and let's see where it gets us15:42
gmannok, looks good idea. +115:43
kopecmartinjm1: thanks, added to my watchlist15:43
kopecmartinanything else for the open discussion?15:43
fricklerjust one other headsup15:44
fricklerI started integrating cirros CI in opendev15:44
fricklersince the old travis setup no longer exists15:45
frickleranyone wanting to help and learn about zuul is welcome to join15:45
fricklerhttps://review.opendev.org/c/cirros/cirros/+/82791615:45
fricklermy current painpoint is how to collect artifacts and where to store them15:46
fricklerand that's it from me15:46
kopecmartinfrickler: thanks for the headsup15:47
kopecmartinone last thing to do15:47
kopecmartin#topic Bug Triage15:47
kopecmartin#link https://etherpad.opendev.org/p/qa-bug-triage-yoga15:48
kopecmartin bug numbers are recorded at ^15:48
kopecmartin#link https://bugs.launchpad.net/tempest/+bug/191104415:48
kopecmartingmann: ^15:48
kopecmartinwe can proceed with unittest2 removal, right?15:48
kopecmartinthere is already a patch proposed 15:48
kopecmartin#link https://review.opendev.org/c/openstack/tempest/+/82619115:48
gmannkopecmartin: about to ask the link. I think yes as stable/train does not use tempest master. but let me check again in case15:50
gmannI will review it today15:50
kopecmartinit doesn't, we pinned tempest for train15:51
kopecmartin#link https://opendev.org/openstack/devstack/src/commit/8a22f7380c7029e931fe9103320f24a223b619d1/stackrc#L31915:51
kopecmartingmann: ok, thanks15:51
kopecmartinthat's all from my side15:51
kopecmartinthank you everyone15:51
kopecmartinsee you around15:51
kopecmartin#endmeeting15:52
opendevmeetMeeting ended Tue Feb  8 15:52:15 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:52
opendevmeetMinutes:        https://meetings.opendev.org/meetings/qa/2022/qa.2022-02-08-15.00.html15:52
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/qa/2022/qa.2022-02-08-15.00.txt15:52
opendevmeetLog:            https://meetings.opendev.org/meetings/qa/2022/qa.2022-02-08-15.00.log.html15:52
frickleroh, wait15:52
fricklermeh15:52
kopecmartinoh, sorry15:52
kopecmartinwhat's up?15:52
fricklermy bad, I was too slow15:52
fricklerthat was an idea in the patch that made chmod -x for stackrc15:52
gmannkopecmartin: i mean in case anyone else need unittest2 than stable/train. train is all fine. 15:53
fricklerwhich is to drop the shebang headers for all files that are only included and not meant to be executed on their own15:53
fricklerwhich I think is sensible15:53
fricklerbut currently our bashate configuration explicitly requires this15:53
fricklerso I was wondering what was the historic reasoning for that and whether it still holds15:54
fricklersee https://review.opendev.org/c/openstack/devstack/+/827815/115:55
fricklerthe solution would be to split the list in https://opendev.org/openstack/devstack/src/branch/master/tox.ini#L17-L34 and have one part running without the shebang test15:56
fricklerif nobody has a good argument not to do that, I will prepare a patch with that15:57
gmannnot sure about reason. But i agree that it make sense to remove it. 15:57
gmannit will be more clear in term they are not meant to be executed as alone 15:58
*** pojadhav|dinner is now known as pojadhav15:58
fricklero.k., great15:59
kopecmartini have no idea why it was made executable , it's good as it is now, without x permission15:59
opendevreviewAndre Aranha proposed openstack/devstack stable/xena: Add openstack-two-node-centos-8-stream  https://review.opendev.org/c/openstack/devstack/+/82824415:59
gmannI think we removed it few file from tempest too very long ago. by masayukig I think. 16:00
abhishekkcan anyone from the team help to understand this failure?16:14
abhishekkhttps://review.opendev.org/c/openstack/glance/+/82511516:14
gmannabhishekk: because of this https://review.opendev.org/c/openstack/tempest/+/825112/6/tempest/config.py#87916:36
gmannthat config option is removed in depends-on tempest patch16:37
abhishekkohh16:38
abhishekkthank you, will have a look16:38
gibigmann: about the rescue issue in the centos 9 stream job. In that job I see libvirt 8.0.0 and qemu 6.2.0 that is way newer than what we use in the ubuntu jobs (libvirt 6.0.0 and qemu 4.2.0) 16:46
gibigmann: and nova fails to detach the volume from the guest via libvirt even after retries 16:47
gibigmann: so if I have to guess then something is changed in the libvirt between 6.0.0 and 8.0.0 that makes this faulty16:48
gmanngibi: ohk, thanks for debugging. yeah, let me open bug in devatack then in case anyone from centos9-stream side check it and fix. 16:50
gibigmann: OK. thanks!16:50
opendevreviewAbhishek Kekane proposed openstack/tempest master: Add image cache client  https://review.opendev.org/c/openstack/tempest/+/82511116:51
opendevreviewAbhishek Kekane proposed openstack/tempest master: Add tempest coverage for new cache API  https://review.opendev.org/c/openstack/tempest/+/82511216:51
*** jpena is now known as jpena|off17:04
*** akahat|dinner is now known as akahat|rover17:05
opendevreviewMartin Kopec proposed openstack/tempest master: Revert "Fix test_rebuild_server test by waiting for floating ip"  https://review.opendev.org/c/openstack/tempest/+/82824517:18
gmannkopecmartin: frickler gibi FYI, logged devstack bug for centos9-stream job failure https://bugs.launchpad.net/devstack/+bug/196034617:57
opendevreviewJakob Meng proposed openstack/devstack master: Added Zuul job devstack-reboot  https://review.opendev.org/c/openstack/devstack/+/82828918:09
opendevreviewJakob Meng proposed openstack/devstack master: Added Zuul job devstack-reboot-tempest  https://review.opendev.org/c/openstack/devstack/+/82828919:14
opendevreviewJakob Meng proposed openstack/devstack master: Added Zuul job devstack-reboot-tempest  https://review.opendev.org/c/openstack/devstack/+/82828919:27
*** fpantano is now known as fmount19:29
*** osmanlicilegi is now known as Guest120:56
opendevreviewBrian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints  https://review.opendev.org/c/openstack/devstack/+/71051922:17
opendevreviewBrian Haley proposed openstack/tempest master: Add an IPv6 tunnel endpoint job definition  https://review.opendev.org/c/openstack/tempest/+/82843122:20
opendevreviewMerged openstack/devstack master: Bump fedora-latest to F35  https://review.opendev.org/c/openstack/devstack/+/82757622:34
opendevreviewMerged openstack/devstack master: Fix deployment of Neutron with enforced scopes  https://review.opendev.org/c/openstack/devstack/+/82685122:39
opendevreviewMerged openstack/devstack master: Revert "Disable enforcing scopes in Neutron temporary"  https://review.opendev.org/c/openstack/devstack/+/82685222:41

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