| openstackgerrit | Sampat Ponnaganti proposed openstack/patrole master: [2008683] For testing only - Do not Merge Cleans up volume after test https://review.opendev.org/c/openstack/patrole/+/778573 | 00:04 |
|---|---|---|
| *** lbragstad has quit IRC | 00:23 | |
| *** tosky has quit IRC | 00:28 | |
| openstackgerrit | Ian Wienand proposed openstack/devstack master: iniset testing: test value with equals sign https://review.opendev.org/c/openstack/devstack/+/779408 | 00:37 |
| openstackgerrit | Merged openstack/tempest master: Make _get_router() non-private https://review.opendev.org/c/openstack/tempest/+/778698 | 00:46 |
| *** lbragstad has joined #openstack-qa | 00:52 | |
| openstackgerrit | Merged openstack/tempest master: Make _create_empty_security_group() non-private https://review.opendev.org/c/openstack/tempest/+/778697 | 01:03 |
| openstackgerrit | Merged openstack/tempest master: Make _get_server_port_id_and_ip4() non-private https://review.opendev.org/c/openstack/tempest/+/777089 | 01:10 |
| *** hamalq has quit IRC | 01:24 | |
| openstackgerrit | Ian Wienand proposed openstack/devstack master: test_worlddump: fix file match https://review.opendev.org/c/openstack/devstack/+/779415 | 02:04 |
| openstackgerrit | Merged openstack/devstack stable/train: zuul: clean up some legacy jobs https://review.opendev.org/c/openstack/devstack/+/756666 | 02:20 |
| *** rh-jelabarre has quit IRC | 02:23 | |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Simulate a remote/standalone g-api worker https://review.opendev.org/c/openstack/devstack/+/770487 | 02:25 |
| *** rcernin has quit IRC | 02:25 | |
| *** rcernin has joined #openstack-qa | 02:42 | |
| *** rcernin has quit IRC | 02:48 | |
| *** rcernin has joined #openstack-qa | 02:51 | |
| *** Yarboa has quit IRC | 02:53 | |
| *** Yarboa has joined #openstack-qa | 02:56 | |
| *** ajitha has joined #openstack-qa | 03:14 | |
| *** rcernin has quit IRC | 03:20 | |
| *** rcernin has joined #openstack-qa | 03:40 | |
| *** lbragstad has quit IRC | 03:43 | |
| *** jparker_ has joined #openstack-qa | 03:50 | |
| *** adriant4 has joined #openstack-qa | 03:51 | |
| *** eandersson8 has joined #openstack-qa | 03:51 | |
| *** rcernin_ has joined #openstack-qa | 03:51 | |
| *** raukadah has joined #openstack-qa | 03:54 | |
| *** TerianceVi has joined #openstack-qa | 03:55 | |
| *** freefood has joined #openstack-qa | 03:56 | |
| *** icey_ has joined #openstack-qa | 03:56 | |
| *** rcernin has quit IRC | 03:59 | |
| *** amodi has quit IRC | 03:59 | |
| *** mjturek has quit IRC | 03:59 | |
| *** openstackgerrit has quit IRC | 03:59 | |
| *** lxkong has quit IRC | 03:59 | |
| *** icey has quit IRC | 03:59 | |
| *** jparker has quit IRC | 03:59 | |
| *** dmellado has quit IRC | 03:59 | |
| *** adriant has quit IRC | 03:59 | |
| *** afazekas has quit IRC | 03:59 | |
| *** chandankumar has quit IRC | 03:59 | |
| *** eandersson has quit IRC | 03:59 | |
| *** TViernion has quit IRC | 03:59 | |
| *** tbarron has quit IRC | 03:59 | |
| *** cgoncalves has quit IRC | 03:59 | |
| *** freefood_ has quit IRC | 03:59 | |
| *** adriant4 is now known as adriant | 03:59 | |
| *** eandersson8 is now known as eandersson | 03:59 | |
| *** dmellado has joined #openstack-qa | 04:02 | |
| *** Yarboa has quit IRC | 04:13 | |
| *** Yarboa has joined #openstack-qa | 04:15 | |
| *** vishalmanchanda has joined #openstack-qa | 04:16 | |
| *** vishalmanchanda has quit IRC | 04:21 | |
| *** vishalmanchanda has joined #openstack-qa | 04:21 | |
| *** whoami-rajat_ has joined #openstack-qa | 04:41 | |
| *** whoami-rajat_ is now known as whoami-rajat | 04:46 | |
| *** raukadah is now known as chandankumar | 04:51 | |
| *** lxkong9 has joined #openstack-qa | 05:02 | |
| *** lxkong9 is now known as lxkong | 05:02 | |
| *** iurygregory has quit IRC | 05:26 | |
| *** lxkong has quit IRC | 05:30 | |
| *** rcernin_ has quit IRC | 05:48 | |
| *** rcernin_ has joined #openstack-qa | 05:49 | |
| *** gcheresh has joined #openstack-qa | 06:12 | |
| *** slaweq has joined #openstack-qa | 06:50 | |
| *** cgoncalves has joined #openstack-qa | 07:09 | |
| *** lpiwowar has joined #openstack-qa | 07:19 | |
| *** ccamposr has joined #openstack-qa | 07:28 | |
| *** vishalmanchanda has quit IRC | 07:36 | |
| *** eolivare has joined #openstack-qa | 07:39 | |
| *** lpiwowar has quit IRC | 07:39 | |
| *** lpiwowar has joined #openstack-qa | 07:51 | |
| *** gfidente has joined #openstack-qa | 08:21 | |
| *** rpittau|afk is now known as rpittau | 08:24 | |
| *** rcernin_ has quit IRC | 08:24 | |
| *** jpena|off is now known as jpena | 08:55 | |
| *** tosky has joined #openstack-qa | 09:02 | |
| *** lucasagomes has joined #openstack-qa | 09:05 | |
| *** ajitha has quit IRC | 10:08 | |
| *** lxkong has joined #openstack-qa | 10:21 | |
| *** rcernin_ has joined #openstack-qa | 10:27 | |
| *** iurygregory_ has joined #openstack-qa | 10:45 | |
| *** iurygregory_ is now known as iurygregory | 10:46 | |
| *** icey_ is now known as icey | 11:02 | |
| *** dtantsur|afk is now known as dtantsur | 11:25 | |
| *** rcernin_ has quit IRC | 11:26 | |
| *** Luzi has joined #openstack-qa | 11:27 | |
| *** artom has quit IRC | 11:51 | |
| *** slaweq_ has joined #openstack-qa | 11:52 | |
| *** brinzhang has quit IRC | 11:56 | |
| *** slaweq has quit IRC | 11:59 | |
| *** artom has joined #openstack-qa | 12:25 | |
| *** jpena is now known as jpena|lunch | 12:34 | |
| *** slaweq_ is now known as slaweq | 12:38 | |
| *** lucasagomes has quit IRC | 13:04 | |
| *** rh-jelabarre has joined #openstack-qa | 13:12 | |
| *** vishalmanchanda has joined #openstack-qa | 13:16 | |
| *** nweinber has joined #openstack-qa | 13:21 | |
| *** lbragstad has joined #openstack-qa | 13:24 | |
| *** jpena|lunch is now known as jpena | 13:36 | |
| *** Yarboa has quit IRC | 13:44 | |
| *** Yarboa has joined #openstack-qa | 13:45 | |
| *** tkajinam has quit IRC | 13:53 | |
| *** amodi has joined #openstack-qa | 13:54 | |
| *** masayukig has joined #openstack-qa | 13:59 | |
| masayukig | #startmeeting qa | 14:00 |
| openstack | Meeting started Tue Mar 9 14:00:11 2021 UTC and is due to finish in 60 minutes. The chair is masayukig. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: qa)" | 14:00 | |
| openstack | The meeting name has been set to 'qa' | 14:00 |
| masayukig | Who all here today? | 14:00 |
| kopecmartin | hi o/ | 14:00 |
| gmann | o/ | 14:02 |
| lpiwowar | Hi o/ | 14:03 |
| masayukig | ok, let's start | 14:04 |
| masayukig | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting | 14:04 |
| masayukig | ^^^^ here's the agenda | 14:04 |
| masayukig | #topic Announcement and Action Item (Optional) | 14:04 |
| *** openstack changes topic to "Announcement and Action Item (Optional) (Meeting topic: qa)" | 14:04 | |
| masayukig | I have two things, vPTG April 2021 is coming. I've created an etherpad for that | 14:05 |
| masayukig | #link https://etherpad.opendev.org/p/qa-xena-ptg | 14:05 |
| masayukig | Please put your ideas for the vPTG | 14:06 |
| masayukig | And your name | 14:06 |
| gmann | and we need to select the time slot too | 14:06 |
| gmann | this one right? https://doodle.com/poll/5ixvdcbm488kc2ic | 14:07 |
| masayukig | yea, I've also create that | 14:07 |
| masayukig | gmann: yeah, thanks | 14:07 |
| gmann | March 25th is deadline for team signup, | 14:08 |
| gmann | may be we can add action item to track it. | 14:08 |
| masayukig | yes, we can select topic during office hours | 14:09 |
| gmann | yeah may be during 23rs one or early depends on what all topics we have | 14:09 |
| gmann | basically filling etherpad is priority so that we can decide how much time we need to book the slot | 14:10 |
| masayukig | gmann: +1 yes, right | 14:10 |
| masayukig | 2nd thing, please run for the QA PTL :) We still have no candidates if I don't miss | 14:11 |
| masayukig | That's all from my side for this topic | 14:11 |
| masayukig | #topic Wallaby Priority Items progress | 14:12 |
| *** openstack changes topic to "Wallaby Priority Items progress (Meeting topic: qa)" | 14:12 | |
| masayukig | #link https://etherpad.opendev.org/p/qa-wallaby-priority | 14:12 |
| masayukig | any updates for the priorites? | 14:12 |
| gmann | for 'RBAC testing strategy', we laded alt persona and scope support for specific role last week | 14:12 |
| gmann | I can mark that as completed and work on tests moving to new rbac in next cycle or so | 14:13 |
| masayukig | cool | 14:13 |
| gmann | I will work on 'microversion testing with 'latest'' this week, at least adding job and see what all test failing | 14:14 |
| gmann | that is all from my side | 14:14 |
| kopecmartin | no updates from my side, a few patches for scenario manager goal have been merged | 14:14 |
| masayukig | gmann: Thanks for working on that | 14:15 |
| masayukig | kopecmartin: thanks, cool | 14:15 |
| gmann | np! | 14:15 |
| masayukig | ok, let's move on if nothing else | 14:16 |
| masayukig | #topic OpenStack Events Updates and Planning | 14:16 |
| *** openstack changes topic to "OpenStack Events Updates and Planning (Meeting topic: qa)" | 14:16 | |
| *** lucasagomes has joined #openstack-qa | 14:16 | |
| masayukig | We already talked about this, let's skip | 14:16 |
| *** lucasagomes has quit IRC | 14:16 | |
| masayukig | #topic Gate Status Checks | 14:17 |
| *** openstack changes topic to "Gate Status Checks (Meeting topic: qa)" | 14:17 | |
| *** lucasagomes has joined #openstack-qa | 14:17 | |
| gmann | added etherpad in event section | 14:17 |
| gmann | gate is back on Tempest and <=stable/train | 14:17 |
| gmann | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-March/020923.html | 14:17 |
| gmann | but on Tempest gate unit test failure happening more frequently but not all time, I will open bug for that and see if I can get the root cause. | 14:18 |
| gmann | test_fix_argument_yes test fail | 14:18 |
| masayukig | hmm.. | 14:19 |
| masayukig | gmann: thanks! | 14:20 |
| gmann | and one of the gate failure fixes are still stuck for long | 14:20 |
| gmann | #link https://review.opendev.org/q/I5b2217d85e6871ca3f7a3f6f859fdce9a50d3946 | 14:21 |
| gmann | this unblock the networking-generic-switch and py27 train jobs | 14:21 |
| gmann | #link https://review.opendev.org/c/openstack/networking-generic-switch/+/773335/13 | 14:21 |
| gmann | we need to merge those | 14:21 |
| masayukig | gmann: thanks, but I'm not sure why this isn't merged yet? #link https://review.opendev.org/c/openstack/devstack/+/774711 | 14:23 |
| masayukig | I mean no +A | 14:23 |
| masayukig | It got 2 +2 already | 14:24 |
| gmann | ohk, did not notice that, ianw reviewed it yesterday but not sure he forgot to +A or waiting for other reviewer ? | 14:24 |
| masayukig | yeah, I don't see any reason for non +A | 14:24 |
| gmann | yeah | 14:25 |
| masayukig | I'll check the patch after office hour | 14:25 |
| masayukig | ok, anything else or let's move on to the next topic | 14:26 |
| gmann | thanks, also the backports, basically once those till stable/train merge then networking-generic-switch gate will be green | 14:26 |
| gmann | yeah | 14:26 |
| *** abhishekk is now known as konan | 14:26 | |
| masayukig | #topic Periodic jobs Status Checks | 14:26 |
| *** openstack changes topic to "Periodic jobs Status Checks (Meeting topic: qa)" | 14:26 | |
| masayukig | Periodic stable: #link https://zuul.openstack.org/builds?job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&job_name=tempest-full-train-py3&pipeline=periodic-stable | 14:27 |
| masayukig | Periodic master: #link https://zuul.openstack.org/builds?job_name=tempest-all&job_name=tempest-full-oslo-master&pipeline=periodic | 14:27 |
| masayukig | It looks all green now? | 14:27 |
| masayukig | almost green | 14:27 |
| gmann | yeah train is fixed so green | 14:27 |
| gmann | it went red during get-pip.py issue | 14:27 |
| masayukig | ah, OK, I see. thanks for fixing | 14:28 |
| masayukig | #topic Sub Teams highlights (Sub Teams means individual projects under QA program) | 14:28 |
| *** openstack changes topic to "Sub Teams highlights (Sub Teams means individual projects under QA program) (Meeting topic: qa)" | 14:28 | |
| *** konan is now known as abhishekk | 14:28 | |
| masayukig | Tempest | 14:29 |
| masayukig | #link https://review.openstack.org/#/q/project:openstack/tempest+status:open | 14:29 |
| gmann | I have pushed the release patch for Tempest | 14:29 |
| masayukig | Any patches or topics to discuss? | 14:29 |
| masayukig | gmann: ++ thanks! | 14:29 |
| gmann | #link https://review.opendev.org/c/openstack/releases/+/779088 | 14:29 |
| gmann | this will include two main items 1. tempest run new intreface 2. system scope complete support | 14:30 |
| gmann | once it is released I will ping johnsom to consume the scope support in specific role too. | 14:31 |
| gmann | and whoami-rajat also or alt persona usage | 14:31 |
| gmann | s/or/for | 14:31 |
| masayukig | +1 for the patch | 14:32 |
| gmann | thanks | 14:32 |
| gmann | that is intermediate release for wallaby and we will cut the final release at the end of wallaby release | 14:32 |
| masayukig | yeah. | 14:33 |
| *** Luzi has quit IRC | 14:34 | |
| masayukig | How about Patrole? #link https://review.openstack.org/#/q/project:openstack/patrole+status:open | 14:34 |
| gmann | I have not got much time to spend on patrole, | 14:35 |
| masayukig | thanks, np | 14:35 |
| gmann | and not sure if I can do in his cycle as few things pending on my side for wallaby | 14:35 |
| masayukig | ack, you've done so many thing already, and still have a lot of tasks | 14:36 |
| masayukig | And, anything else on Devstack? #link https://review.openstack.org/#/q/project:openstack/devstack+status:open | 14:37 |
| gmann | there is on on remote glance patch but that is under review now so not stuck | 14:38 |
| masayukig | ok | 14:38 |
| masayukig | I don't see urgent patches on Grenade and Hacking but anything to discuss? #link https://review.opendev.org/q/project:openstack/grenade+status:open | 14:39 |
| masayukig | #link https://review.opendev.org/q/project:openstack/hacking+status:open | 14:39 |
| gmann | if you can check these two | 14:39 |
| gmann | #link https://review.opendev.org/c/openstack/hacking/+/773814 | 14:39 |
| gmann | #link https://review.opendev.org/c/openstack/hacking/+/777636 | 14:40 |
| gmann | they are all green | 14:40 |
| whoami-rajat | thanks gmann for the release update! | 14:41 |
| gmann | whoami-rajat: np! sir | 14:41 |
| gmann | we do not have much change merged in hacking since its last release so we do not need to release it for wallaby as such | 14:41 |
| gmann | it is anyways in independent release model | 14:42 |
| masayukig | +2 and +A for the patches | 14:42 |
| masayukig | ok, move on to the next topic | 14:43 |
| gmann | thanks | 14:43 |
| masayukig | #topic Gate Blocker Fix / Urgent Change | 14:43 |
| *** openstack changes topic to "Gate Blocker Fix / Urgent Change (Meeting topic: qa)" | 14:43 | |
| masayukig | Any blocker for the gate? | 14:43 |
| gmann | nothing i am aware of | 14:44 |
| gmann | except the ones we already talked | 14:44 |
| masayukig | ok, cool | 14:45 |
| masayukig | #topic Open Discussion | 14:45 |
| *** openstack changes topic to "Open Discussion (Meeting topic: qa)" | 14:45 | |
| masayukig | anything else to discuss? I can see no topics on the agenda, though | 14:46 |
| gmann | one topic. | 14:46 |
| dansmith | can I ask about devstack things here? | 14:46 |
| gmann | thanks masayukig for serving as PTL for past 2 cycles. | 14:46 |
| masayukig | gmann: thanks! | 14:46 |
| gmann | PTL nomination deadline is today i think, if anyone would like to take resp for next cycle feel free to submit the candidacy. | 14:47 |
| masayukig | I'm sorry for my narrow bandwidth, though. | 14:47 |
| kopecmartin | masayukig: thank you | 14:47 |
| gmann | that's fine we all are busy in other things | 14:47 |
| masayukig | gmann: kopecmartin thanks | 14:47 |
| gmann | you did your best | 14:48 |
| gmann | that's all | 14:48 |
| gmann | dansmith: go ahead | 14:48 |
| dansmith | heh, sorry | 14:48 |
| masayukig | :) | 14:48 |
| dansmith | so I'm wondering if I could get a +W on this: https://review.opendev.org/c/openstack/devstack/+/770487 it's been sitting for two months and is in front of a chain of two other patches in dependent projects, | 14:48 |
| dansmith | which we need in order to be testing a thing we merged | 14:48 |
| dansmith | the feedback on the latest rev is mostly nit and style stuff, which I'm happy to circle back on, | 14:49 |
| dansmith | but I really don't want to have to make all three dependent patches take another couple days to get approved, because it's already likely it's going to take days to get these merged | 14:49 |
| gmann | dansmith: i see, did not check those comment yet. | 14:49 |
| *** artom has quit IRC | 14:49 | |
| dansmith | is that reasonable if I promise to clean up those quote and naming nits? | 14:50 |
| gmann | i think that is much reasonable and i do not see frickler -1 so not blocking. | 14:50 |
| gmann | I will check after my meeting (1 internal after this office hour) | 14:50 |
| gmann | Tempest one is already +A right? | 14:51 |
| dansmith | okay thanks | 14:51 |
| dansmith | yes | 14:51 |
| gmann | k | 14:51 |
| masayukig | cool | 14:51 |
| masayukig | thanks | 14:52 |
| masayukig | #topic Bug Triage (last 30 min) | 14:52 |
| *** openstack changes topic to "Bug Triage (last 30 min) (Meeting topic: qa)" | 14:52 | |
| masayukig | #link https://etherpad.openstack.org/p/qa-bug-triage-wallaby | 14:52 |
| gmann | we left only 8 min, may be we can chck if any critical bug | 14:52 |
| masayukig | yeah | 14:52 |
| kopecmartin | no critical bugs | 14:53 |
| gmann | only devstack has 2 | 14:53 |
| gmann | #link https://bugs.launchpad.net/devstack/+bugs?search=Search&field.importance=Critical&field.status=New&field.status=Incomplete&field.status=Confirmed&field.status=Triaged&field.status=In+Progress&field.status=Fix+Committed | 14:53 |
| kopecmartin | oh, I meant tempest only :) | 14:53 |
| gmann | but those are fixed, may be its just LP not updated | 14:53 |
| gmann | I will add patches in LP comment and mark fix released | 14:54 |
| masayukig | gmann: cool | 14:54 |
| gmann | kopecmartin: yeah. tempest is all good with your hard work on bugs :) | 14:54 |
| masayukig | thank you! | 14:54 |
| masayukig | anything else to discuss? or let's close the office hour | 14:55 |
| gmann | nothing from my side | 14:55 |
| masayukig | ok, let's close this | 14:55 |
| masayukig | thank you all!!! | 14:55 |
| masayukig | #endmeeting | 14:56 |
| *** openstack changes topic to "Discussion of OpenStack Quality Assurance Projects (tempest, devstack, grenade, and friends) | docs https://wiki.openstack.org/wiki/QA | specs https://review.openstack.org/#/q/project:openstack/qa-specs | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-qa/" | 14:56 | |
| openstack | Meeting ended Tue Mar 9 14:56:00 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:56 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-03-09-14.00.html | 14:56 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-03-09-14.00.txt | 14:56 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-03-09-14.00.log.html | 14:56 |
| gmann | thanks | 14:56 |
| dansmith | does tempest go into feature freeze on thursday like the other projects? | 15:06 |
| tosky | no, it's branchless, but a tag is created at relevant times | 15:09 |
| dansmith | okay cool, so I don't have to worry about an FFE for getting tests in that are pending but not landed by FF? | 15:10 |
| dansmith | that's ... excellent news :) | 15:10 |
| masayukig | tosky: dansmith yeah, you don't have to worry about it for tempest :) | 15:13 |
| dansmith | sweet | 15:13 |
| dansmith | tempest branches though, but I assume it doesn't have strict feature freeze? | 15:14 |
| *** mjturek has joined #openstack-qa | 15:14 | |
| dansmith | sorry, *Devstack* | 15:14 |
| masayukig | yeah, I don't think we have feature freeze for devstack. But let's ask gmann, frickler, ianw opinion just in case. | 15:15 |
| gmann | dansmith: masayukig we do as m-3 | 15:16 |
| gmann | for devstack and tempest | 15:16 |
| masayukig | oh, sorry is that the feature freeze? | 15:17 |
| gmann | https://wiki.openstack.org/wiki/QA/releases#Feature_Freeze | 15:17 |
| masayukig | sorry | 15:17 |
| gmann | yeah we set in last or Train cycle | 15:17 |
| dansmith | oh gosh | 15:18 |
| masayukig | https://releases.openstack.org/wallaby/schedule.html | 15:18 |
| dansmith | there are several tempest patches pending and the way it's going, I'm pretttty worried | 15:18 |
| gmann | dansmith: can you link those we can review on priorty. | 15:19 |
| gmann | this one is on my list for today https://review.opendev.org/c/openstack/tempest/+/774303 | 15:19 |
| dansmith | well, this one for image tasks API in glance: https://review.opendev.org/c/openstack/tempest/+/775679 | 15:20 |
| dansmith | this one for regression cover: https://review.opendev.org/c/openstack/tempest/+/774303 | 15:20 |
| dansmith | this one is very important to us, but already +W: https://review.opendev.org/c/openstack/tempest/+/770520 | 15:20 |
| gmann | yeah | 15:20 |
| gmann | today I will review 774303 | 15:20 |
| dansmith | seems like the tempest gate has been really bad in terms of rechecking those patches, but the meeting this morning said no blockers, so maybe it's better | 15:20 |
| dansmith | gmann: ack thanks, I feel like you do everything, so sorry to add to your list :/ | 15:21 |
| gmann | np!, thanks for bringing those | 15:21 |
| dansmith | gmann: I have also seen this a couple times lately: https://zuul.opendev.org/t/openstack/build/7083050170db411e9ba5418f071df611/log/job-output.txt#6863 | 15:23 |
| dansmith | I wonder if we should try to retry a pip install once or something.. | 15:24 |
| dansmith | logstash shows it's not always exactly that error, so ... maybe not | 15:27 |
| dansmith | so this unit test fail in tempest: https://423b70bc819a8a28a5aa-c8df7fee768f0a38e1c53a0e9f1f4351.ssl.cf5.rackcdn.com/775679/4/check/openstack-tox-py38/1089f44/testr_results.html | 15:50 |
| dansmith | seems to be happening a _lot_: | 15:50 |
| dansmith | https://imgur.com/a/A6ubzV4 | 15:50 |
| dansmith | almost 70 hits just yesterday | 15:50 |
| gmann | dansmith: yeah I will debug that today its happening more frequently now a days | 15:51 |
| gmann | i mean unit test failure | 15:51 |
| dansmith | how important is it? I wonder if just skipping for now would be prudent? | 15:51 |
| dansmith | yeah | 15:51 |
| gmann | it is for check-uuid tool. let me see if I do not find reason today then i will propose for fix and do like do not merge check-uuid changes if any | 15:52 |
| dansmith | okay | 15:53 |
| dansmith | gmann: maybe if both of those tests run in parallel, they both create a tmpdir "module" and then one deletes it while the other is still doing its import? | 16:03 |
| gmann | is it same name they use? | 16:04 |
| lyarwood | ~. | 16:05 |
| gmann | humm | 16:05 |
| *** openstackgerrit has joined #openstack-qa | 16:06 | |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Address feedback from glance-remote patch https://review.opendev.org/c/openstack/devstack/+/779523 | 16:06 |
| dansmith | gmann: no, but they do an import, which probably does an os.walk() which could fail if one disappears after it does the listdir() | 16:07 |
| dansmith | gmann: I'll try to repro locally and then see if I can catch it happening | 16:08 |
| gmann | but its same class tests, they run serially | 16:08 |
| dansmith | ...really? | 16:08 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Address feedback from glance-remote patch https://review.opendev.org/c/openstack/devstack/+/779523 | 16:21 |
| *** lpiwowar has quit IRC | 16:36 | |
| dansmith | yoctozepto: is the -1 just for the cleanup note? | 16:40 |
| yoctozepto | dansmith: no, for all together; don't feel bad; the approach is nice | 16:42 |
| dansmith | okay I'm not sure I understand what you want changed other than the cleanup thing | 16:43 |
| dansmith | I addressed frickler's quoting comments in a patch above | 16:43 |
| dansmith | yoctozepto: I feel bad because this went unreviewed for two months and now that we're in crunch time, every time I have to revise this for nits, I have two other patches in two other projects that have to recheck a bunch of times to get a clean result | 16:46 |
| dansmith | and we're running out of time to be actually testing this thing which is a highlight for the glance wallaby release | 16:47 |
| dansmith | so I just want to make sure I understand what is critical to be fixed, and in the fewest number of round trips | 16:47 |
| *** gcheresh has quit IRC | 16:47 | |
| yoctozepto | dansmith: ack, it's "good enough" in general | 16:48 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Address feedback from glance-remote patch https://review.opendev.org/c/openstack/devstack/+/779523 | 16:48 |
| yoctozepto | left one comment | 16:49 |
| yoctozepto | let's fix these in the followup indeed | 16:49 |
| dansmith | yoctozepto: here's the updated ^ cleanup with the right dir | 16:49 |
| yoctozepto | dansmith: could you also take a look at that L500 | 16:50 |
| gmann | yoctozepto: yeah there is already follow up from dansmith I am checking if anything missing otherwise let's merge that soon as we are m-3 | 16:50 |
| yoctozepto | yeah, I have approved | 16:50 |
| yoctozepto | it's good enough to approve in this shape already | 16:50 |
| gmann | ohk, results also being verified and it was all good | 16:51 |
| yoctozepto | for the record, I did not notice the followup being done | 16:51 |
| gmann | i see | 16:51 |
| dansmith | yoctozepto: it basename's the argument so either works, I can convert to $GLANCE_API_CONF in the follow-up | 16:51 |
| yoctozepto | dansmith: ack | 16:55 |
| yoctozepto | thanks | 16:55 |
| dansmith | yoctozepto: stacking on the cleanup patch with that change now, and will push it up once I know it works | 16:56 |
| *** lucasagomes has quit IRC | 17:02 | |
| *** rpittau is now known as rpittau|afk | 17:05 | |
| dansmith | gmann: kopecmartin: could one of you un- and then re-W this? it's already approved but hasn't entered the gate because it was dependent on the devstack patch: https://review.opendev.org/c/openstack/tempest/+/770520 | 17:09 |
| dansmith | I don't want to have to recheck it to get it to notice :) | 17:09 |
| gmann | dansmith: ack, let's devstack patch merge in case we have to recheck there :) | 17:11 |
| dansmith | okay, just trying to optimize linear time, assuming the devstack patch lands on the first go | 17:12 |
| dansmith | the tempest gate is much smaller than its check, AFAICT, and right now every recheck I do of the other tempest patches fails with something, so I was expecting I'd be more likely to pass gate than check | 17:13 |
| kopecmartin | gmann: sorry, i gave my +w vote the second you wrote the msg :/ | 17:14 |
| kopecmartin | let's hope, the devstack change will pass the gates smoothly | 17:15 |
| gmann | no worry. let see how it goes | 17:15 |
| dansmith | why would we not try to get both in the gate at the same time? I must be missing something. | 17:16 |
| kopecmartin | because if the devstack one fails for any reason, the tempest change will get -2 by zuul and we will need to recheck it | 17:16 |
| kopecmartin | i think | 17:16 |
| dansmith | okay | 17:17 |
| gmann | yeah, unless depends-on is merged before we have to recheck/re-w on tempest noe | 17:18 |
| gmann | one | 17:18 |
| dansmith | okay, I guess at the moment, it seems like we're in wartime and optimizing for time through the gate is ideal | 17:19 |
| dansmith | but fair enough :) | 17:19 |
| gmann | an as tempest one is in gate pipeline and devstack in check, we have to re-w on tempest even devstack pass in first gio | 17:19 |
| gmann | go | 17:19 |
| *** eolivare has quit IRC | 17:22 | |
| openstackgerrit | Pierre Riteau proposed openstack/devstack stable/victoria: Fix DevStack setup on CentOS 8.3 https://review.opendev.org/c/openstack/devstack/+/779544 | 17:27 |
| openstackgerrit | Pierre Riteau proposed openstack/devstack stable/ussuri: Fix DevStack setup on CentOS 8.3 https://review.opendev.org/c/openstack/devstack/+/779545 | 17:28 |
| openstackgerrit | Dan Smith proposed openstack/devstack master: Address feedback from glance-remote patch https://review.opendev.org/c/openstack/devstack/+/779523 | 17:29 |
| dansmith | yoctozepto: ^ | 17:29 |
| gmann | yoctozepto: ianw masayukig anything else I need to fix this? three +2 but need +A - https://review.opendev.org/c/openstack/devstack/+/774711 | 17:42 |
| gmann | kopecmartin: can you please +A this now? https://review.opendev.org/c/openstack/tempest/+/698773 | 17:43 |
| kopecmartin | sure | 17:45 |
| gmann | thanks | 17:45 |
| *** gfidente is now known as gfidente|afk | 17:47 | |
| openstackgerrit | Lee Yarwood proposed openstack/devstack master: Only write out uwsgi configs when deploying API services https://review.opendev.org/c/openstack/devstack/+/779547 | 17:48 |
| dansmith | gmann: just to help us both remember, if we have to re-approve the tempest glance-remote test, I need to squash the region cleanup into it to drop the patch count | 17:49 |
| dansmith | and same for the devstack cleanup I guess | 17:49 |
| gmann | dansmith: yeah, I will say tempest one we can do but devstack is up to you. | 17:51 |
| dansmith | gmann: ack | 17:51 |
| gmann | dansmith: or you think changing the glance conf things in devstack cleanup need to be tested again?' | 17:53 |
| gmann | in that case I think while updating the tempest, make depends-on to devstack cleanup | 17:53 |
| dansmith | gmann: yeah, the devstack cleanup won't have run against the full three-patch job so I'd rather hold off on that if it's all the same | 17:53 |
| gmann | k | 17:54 |
| dansmith | I'm pretty sure it's fine, but at this point I'd rather get in without the cleanups but have tests enabled | 17:54 |
| dansmith | btw, I have run 200,000 tempest unit tests locally and cannot get that uuid test to fail | 17:55 |
| gmann | k | 17:55 |
| gmann | i am also trying that unit tests, and only test_fix_argument_yes fail where other test also import the tmp dir | 17:56 |
| dansmith | you see the fail locally? | 17:56 |
| gmann | I am suspecting the fixture cleanup timing of test_fix_argument_yes itself playing some race | 17:56 |
| gmann | not yet | 17:56 |
| *** amodi has quit IRC | 17:56 | |
| gmann | I am trying with sleep or so but no luck yet. | 17:56 |
| dansmith | okay, I really didn't think those all ran serially inside one class.. are you sure that's how it works? | 17:57 |
| gmann | I think so, class level are serial by default. | 17:57 |
| dansmith | I thought there was another base class to use when you wanted things serialized | 17:57 |
| dansmith | but okay | 17:57 |
| dansmith | unless there is some other test somewhere that is creating a ./$tmp/ module that is getting created and removed at the same time, I dunno what else could be wrong there | 17:58 |
| gmann | did not check stestr param if changed or we are doing parallel for unit test. | 17:58 |
| gmann | yeah | 17:58 |
| *** artom has joined #openstack-qa | 18:01 | |
| *** jpena is now known as jpena|off | 18:02 | |
| *** dtantsur is now known as dtantsur|afk | 18:13 | |
| *** ralonsoh has quit IRC | 18:28 | |
| *** hamalq has joined #openstack-qa | 18:30 | |
| *** ccamposr has quit IRC | 18:32 | |
| *** nweinber has quit IRC | 18:33 | |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Create tmd dir with specific prefix in check-uuid unit tests https://review.opendev.org/c/openstack/tempest/+/779555 | 18:36 |
| gmann | dansmith: kopecmartin ^^ if this can help. i cannot reproduce so donot know actual root cause | 18:36 |
| dansmith | gmann: thats similar to what I was going to try, but since we're already passing the path of the tmpdir, I thought maybe I would need to cd into the tmpdir to prevent the thing from finding the other dir | 18:37 |
| openstackgerrit | Ghanshyam proposed openstack/tempest master: Create tmd dir with specific prefix in check-uuid unit tests https://review.opendev.org/c/openstack/tempest/+/779555 | 18:39 |
| *** nweinber has joined #openstack-qa | 18:52 | |
| *** hamalq has quit IRC | 19:09 | |
| *** hamalq has joined #openstack-qa | 19:09 | |
| *** amodi has joined #openstack-qa | 19:44 | |
| *** lbragstad has quit IRC | 20:01 | |
| *** ccamposr has joined #openstack-qa | 20:15 | |
| *** lbragstad has joined #openstack-qa | 20:15 | |
| openstackgerrit | Merged openstack/devstack master: Simulate a remote/standalone g-api worker https://review.opendev.org/c/openstack/devstack/+/770487 | 20:25 |
| openstackgerrit | Merged openstack/tempest master: Test glance distributed import https://review.opendev.org/c/openstack/tempest/+/770520 | 20:29 |
| *** irclogbot_3 has quit IRC | 20:31 | |
| *** irclogbot_1 has joined #openstack-qa | 20:33 | |
| *** sboyron has quit IRC | 20:38 | |
| *** whoami-rajat has quit IRC | 20:42 | |
| dansmith | gmann: yoctozepto: My blood pressure is returning to normal.. thanks for ^ | 21:06 |
| gmann | heh :). | 21:07 |
| yoctozepto | dansmith: I appreciate good upstream work | 21:08 |
| gmann | checking web-download one too | 21:08 |
| gmann | dansmith: this is merge conflict - https://review.opendev.org/c/openstack/tempest/+/775679/4 | 21:08 |
| dansmith | image-tasks is in the gate | 21:08 |
| dansmith | oh | 21:09 |
| dansmith | hmm | 21:09 |
| gmann | may be due to 770520 | 21:09 |
| openstackgerrit | Sampat Ponnaganti proposed openstack/patrole master: [2008683] Use class level reosurce in testing Instance level resource is causing resource leak https://review.opendev.org/c/openstack/patrole/+/778573 | 21:09 |
| openstackgerrit | Dan Smith proposed openstack/tempest master: Add image task validation https://review.opendev.org/c/openstack/tempest/+/775679 | 21:11 |
| dansmith | gmann: rebased ^ | 21:11 |
| dansmith | definitely good to let that go through check first, as there's non-trivial change | 21:12 |
| gmann | dansmith: +A. yeah checked those. | 21:13 |
| *** Yarboa has quit IRC | 21:13 | |
| *** Yarboa has joined #openstack-qa | 21:16 | |
| dansmith | gmann: that patch already failed the unit test :) https://818dd4daaa8b64e65699-a5f35bd9e6b1ff88ed5feb29b52d40b3.ssl.cf1.rackcdn.com/775679/5/check/openstack-tox-py36/f25b789/testr_results.html | 21:21 |
| gmann | dansmith: humm, this change create temp dir with check-uuid-yes prefix and it is failing with some random dir import | 21:25 |
| dansmith | I'm not sure what you mean...this is the same failure we've been seeing yes? | 21:28 |
| gmann | yeah but dir name i changed in this change which is what used to fail on import in check-uuid. | 21:28 |
| gmann | https://review.opendev.org/c/openstack/tempest/+/779555/1/tempest/tests/lib/cmd/test_check_uuid.py#60 | 21:29 |
| dansmith | gmann: right that one passed right? | 21:29 |
| gmann | no, that test failed but with some other tmp dir error | 21:30 |
| gmann | that is why i am confused | 21:30 |
| gmann | --package arg is being modified by some other tests | 21:31 |
| dansmith | 779555 is Zuul+1 for me | 21:31 |
| gmann | dansmith: ohhh, i thought you pasted 779555 link saying fix itself failing :) | 21:32 |
| gmann | got confused, sorry | 21:32 |
| dansmith | no, I was saying moments after retrying the image-tasks one it was already failing that functional tests | 21:32 |
| dansmith | just like "here we go again" :) | 21:32 |
| gmann | humm | 21:32 |
| gmann | kopecmartin: can we merge this and see if that fix https://review.opendev.org/c/openstack/tempest/+/779555/1 | 21:32 |
| dansmith | the benefit of this ^ is that if we see the -no version in the -yes test fail, we'll know that's the thing that is overlapping | 21:33 |
| gmann | yeah | 21:34 |
| gmann | dansmith: seems 775679 failing on pep8 too but i cannot see log something wrong | 21:34 |
| openstackgerrit | Sampat Ponnaganti proposed openstack/patrole master: [2008683] Uses class level resource in the test https://review.opendev.org/c/openstack/patrole/+/778573 | 21:35 |
| dansmith | gmann: the job link for the pep8 won't even load for me | 21:35 |
| gmann | ditto | 21:35 |
| gmann | how to stop it from gate. it is rebased also | 21:36 |
| dansmith | I can push some trivial change | 21:36 |
| gmann | or rebase on top of it https://review.opendev.org/c/openstack/tempest/+/774303 | 21:36 |
| dansmith | okay | 21:37 |
| gmann | but pep8 you checked locally right ? | 21:37 |
| gmann | if something rebase caused that | 21:37 |
| dansmith | maybe not after the last one, will try again | 21:38 |
| gmann | k | 21:38 |
| dansmith | ooh, yep, I didn't convert one line | 21:39 |
| openstackgerrit | Dan Smith proposed openstack/tempest master: Add image task validation https://review.opendev.org/c/openstack/tempest/+/775679 | 21:41 |
| *** nweinber has quit IRC | 21:48 | |
| openstackgerrit | Merged openstack/tempest master: Remove the unused _api_version variable from volume tests https://review.opendev.org/c/openstack/tempest/+/698773 | 21:50 |
| openstackgerrit | Merged openstack/devstack master: Add a variable to configure the Tempest venv upper constraints https://review.opendev.org/c/openstack/devstack/+/774711 | 21:50 |
| *** Yarboa has quit IRC | 22:14 | |
| *** Yarboa has joined #openstack-qa | 22:16 | |
| *** lxkong has quit IRC | 22:21 | |
| *** lxkong has joined #openstack-qa | 22:22 | |
| *** rcernin has joined #openstack-qa | 22:24 | |
| *** rcernin has quit IRC | 22:30 | |
| *** rcernin has joined #openstack-qa | 22:30 | |
| openstackgerrit | Lee Yarwood proposed openstack/devstack master: WIP cinder: Default CINDER_ISCSI_HELPER to lioadm on Ubuntu https://review.opendev.org/c/openstack/devstack/+/779624 | 22:37 |
| *** tkajinam has joined #openstack-qa | 23:00 | |
| *** adriant has quit IRC | 23:11 | |
| *** adriant has joined #openstack-qa | 23:12 | |
| *** Yarboa has quit IRC | 23:24 | |
| *** Yarboa has joined #openstack-qa | 23:35 | |
| dansmith | gmann: this is about to pass check if you can re-+W: https://review.opendev.org/c/openstack/tempest/+/775679 | 23:40 |
| dansmith | I pushed up to fix that pep8 thing since you did last | 23:40 |
| dansmith | the other ones failed gate and are having to recheck now | 23:40 |
| gmann | dansmith: done | 23:43 |
| dansmith | thanks | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!