Thursday, 2020-06-18

*** rfolco|rover has joined #openstack-qa00:05
*** rfolco|rover has quit IRC00:47
*** brinzhang has joined #openstack-qa01:02
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Work around to make multinode job as n-v  https://review.opendev.org/73641101:08
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Always source lib/apache when lib/tls is used  https://review.opendev.org/69951501:10
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Add GRENADE_USE_EXTERNAL_DEVSTACK to skip part of the setup  https://review.opendev.org/73628301:11
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628401:17
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628401:20
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Update grenade setting for stable/train  https://review.opendev.org/73628501:21
*** Liang__ has joined #openstack-qa01:21
*** Liang__ is now known as LiangFang01:21
*** rfolco|rover has joined #openstack-qa01:23
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Always source lib/apache when lib/tls is used  https://review.opendev.org/69951501:27
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628401:27
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Add GRENADE_USE_EXTERNAL_DEVSTACK to skip part of the setup  https://review.opendev.org/73628301:27
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628401:27
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Update grenade setting for stable/train  https://review.opendev.org/73628501:28
*** paras333 has joined #openstack-qa01:40
*** brinzhang_ has joined #openstack-qa01:59
*** yamamoto has joined #openstack-qa02:01
*** brinzhang has quit IRC02:02
*** paras333 has joined #openstack-qa02:11
*** brinzhang0 has joined #openstack-qa02:26
*** yamamoto has quit IRC02:26
*** yamamoto has joined #openstack-qa02:27
*** yamamoto has quit IRC02:27
*** yamamoto has joined #openstack-qa02:27
*** brinzhang_ has quit IRC02:29
*** paras333 has quit IRC02:42
*** xinranwang_ has joined #openstack-qa02:43
*** brinzhang_ has joined #openstack-qa02:47
*** rcernin has quit IRC02:48
*** brinzhang0 has quit IRC02:50
*** paras333 has joined #openstack-qa02:52
*** paras333 has quit IRC02:56
*** ysirndjuro has quit IRC02:58
*** rcernin has joined #openstack-qa02:59
*** brinzhang_ is now known as brinzhang03:01
*** paras333 has joined #openstack-qa03:03
*** rcernin has quit IRC03:05
*** rcernin has joined #openstack-qa03:20
*** rcernin has quit IRC03:22
*** rcernin has joined #openstack-qa03:22
*** yamamoto has quit IRC03:41
*** yamamoto has joined #openstack-qa03:49
*** markvoelker has quit IRC03:49
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-qa04:33
*** gcheresh has joined #openstack-qa04:49
*** abdysn has joined #openstack-qa05:11
*** udesale has joined #openstack-qa05:42
*** markvoelker has joined #openstack-qa05:51
*** markvoelker has quit IRC05:55
openstackgerritPratik proposed openstack/bashate master: optimize size and time using "--no-cache-dir"  https://review.opendev.org/73644206:10
*** markvoelker has joined #openstack-qa06:19
*** markvoelker has quit IRC06:24
*** rpittau|afk is now known as rpittau06:43
*** gfidente|afk is now known as gfidente06:44
*** slaweq has joined #openstack-qa07:07
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Prepare for dropping keystone admin endpoint  https://review.opendev.org/65149207:11
openstackgerritJens Harbott (frickler) proposed openstack/devstack master: Stop creating a keystone admin endpoint  https://review.opendev.org/73547207:11
*** ralonsoh has joined #openstack-qa07:21
*** tosky has joined #openstack-qa07:25
*** elod has quit IRC07:26
*** LiangFang has quit IRC07:31
*** elod has joined #openstack-qa07:33
*** Liang__ has joined #openstack-qa07:34
*** ccamposr__ has joined #openstack-qa07:42
*** Liang__ has quit IRC07:44
openstackgerritLuigi Toscano proposed openstack/devstack stable/stein: sync-devstack-data: new argument devstack_data_base_dir  https://review.opendev.org/73647507:45
*** ccamposr has quit IRC07:45
*** Liang__ has joined #openstack-qa07:46
*** rcernin has quit IRC07:54
*** rcernin_ has joined #openstack-qa07:54
AJaegerfrickler, ianw, openSUSE is failing on older releases with py27 not found - why is this working for other distros and not openSUSE? See https://review.opendev.org/#/c/735641/07:55
*** jpena|off is now known as jpena07:57
AJaeger(this is for devstack)07:58
openstackgerritMerged openstack/grenade stable/train: Work around to make multinode job as n-v  https://review.opendev.org/73641108:04
AJaegerfrickler, ianw, seems Xenial installs pip via tools/install_pip, see https://zuul.opendev.org/t/openstack/build/32bc241cafb24150aefb122d36666177/log/job-output.txt#380708:05
AJaegermmh, pip is installed for openSUSE as well - and then install_pip uninstalls and fails ;(08:07
*** lucasagomes has joined #openstack-qa08:08
openstackgerritIgnacy Osetek proposed openstack/devstack stable/queens: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73618908:18
*** udesale_ has joined #openstack-qa08:19
*** elod has quit IRC08:19
*** rcernin_ has quit IRC08:20
fricklerAJaeger: seems this has USE_PYTHON3=False, maybe you want to change that to true? otherwise I think we'd need to hold a node to debug further08:20
* frickler sets up a hold anyway, you can choose between recheck and changing to py308:21
*** udesale has quit IRC08:21
openstackgerritLuigi Toscano proposed openstack/grenade stable/train: Always source lib/apache when lib/tls is used  https://review.opendev.org/69951508:22
openstackgerritLuigi Toscano proposed openstack/grenade stable/train: Add GRENADE_USE_EXTERNAL_DEVSTACK to skip part of the setup  https://review.opendev.org/73628308:22
openstackgerritLuigi Toscano proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628408:23
openstackgerritLuigi Toscano proposed openstack/grenade stable/train: Update grenade setting for stable/train  https://review.opendev.org/73628508:23
*** elod has joined #openstack-qa08:26
openstackgerritIgnacy Osetek proposed openstack/devstack master: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73619408:27
*** dtantsur|afk is now known as dtantsur08:29
*** yamamoto has quit IRC08:42
*** yamamoto has joined #openstack-qa08:42
ianwAJaeger: probably just an ensure-pip call in devstack/playbooks/pre.yaml would do it, if it's not being installed otherwise by scripts?08:43
openstackgerritzhufl proposed openstack/tempest master: Fix test_get_service_by_invalid_params for microversion 2.75  https://review.opendev.org/73650308:43
fricklerianw: IMO devstack should work outside the gate, too, so we shouldn't rely on any pre stuff there08:47
*** elod has quit IRC09:00
openstackgerritXinliang Liu proposed openstack/devstack-plugin-ceph master: support focal  https://review.opendev.org/72836209:05
openstackgerritXinliang Liu proposed openstack/devstack-plugin-ceph master: Ceph-iscsi: Use pip to install targetcli-fb  https://review.opendev.org/73375109:05
openstackgerritXinliang Liu proposed openstack/devstack-plugin-ceph master: Make ceph iscsi gateway work again  https://review.opendev.org/73650809:05
*** yamamoto has quit IRC09:12
*** yamamoto has joined #openstack-qa09:13
*** yamamoto has quit IRC09:13
*** vishalmanchanda has joined #openstack-qa09:17
*** elod has joined #openstack-qa09:17
openstackgerritXinliang Liu proposed openstack/devstack-plugin-ceph master: Make ceph iscsi gateway work again  https://review.opendev.org/73650809:18
*** elod has quit IRC09:19
*** yamamoto has joined #openstack-qa09:23
*** elod has joined #openstack-qa09:25
openstackgerritThomas Morin proposed openstack/tempest master: object storage: fix and cleanup header checks  https://review.opendev.org/73629209:35
*** yamamoto has quit IRC09:40
ianwfrickler: fair point, that was kind of the idea that this uncovers things that were hidden in the infra images :)09:51
fricklerianw: yeah, seems to work as intended09:55
*** tkajinam has quit IRC10:05
*** rpittau is now known as rpittau|bbl10:06
openstackgerritThomas Morin proposed openstack/tempest master: object storage: fix and cleanup header checks  https://review.opendev.org/73629210:07
*** yamamoto has joined #openstack-qa10:14
*** Liang__ has quit IRC10:18
openstackgerritFederico Ressi proposed openstack/devstack master: Handle uwsgi install for CentOS/RHEL  https://review.opendev.org/73555810:18
*** chaconpiza has joined #openstack-qa10:19
*** yamamoto has quit IRC10:20
*** yamamoto has joined #openstack-qa10:20
*** yamamoto has quit IRC10:43
*** yamamoto has joined #openstack-qa10:44
*** yamamoto has quit IRC10:44
*** rcernin_ has joined #openstack-qa10:48
*** yamamoto has joined #openstack-qa10:48
*** yamamoto has quit IRC10:53
*** yamamoto has joined #openstack-qa10:54
*** yamamoto has quit IRC10:59
AJaegerfrickler: in train, it should be USE_PYTHON3=False, shouldn't it?11:02
*** udesale_ has quit IRC11:12
*** yamamoto has joined #openstack-qa11:16
*** yamamoto has quit IRC11:20
*** markvoelker has joined #openstack-qa11:24
*** markvoelker has quit IRC11:29
openstackgerritIgnacy Osetek proposed openstack/devstack master: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73619411:31
openstackgerritIgnacy Osetek proposed openstack/devstack stable/queens: Fix for downloading uwsgi on centos in lib/apache  https://review.opendev.org/73618911:32
*** yamamoto has joined #openstack-qa11:34
*** TomStappaerts has quit IRC11:37
*** TomStappaerts has joined #openstack-qa11:38
*** jpena is now known as jpena|lunch11:38
fricklerAJaeger: that's the default probably, but at this point in time I'm not sure we need to keep it that way, py3 should be fully supported as well11:39
fricklerexcept for curiosities like swift11:39
AJaegerfrickler: I'm talking about *Train* here - but let me try...11:40
AJaegeralso, on train the xenial and centos devstack-platform jobs are failing ;811:41
fricklerAJaeger: yes, that seemed to be some tempest/reqs issue, maybe someone will fix that, otherwise we'll just drop them11:42
openstackgerritAndreas Jaeger proposed openstack/devstack stable/train: Set USE_PYTHON3 for openSUSE  https://review.opendev.org/73664711:42
AJaegerfrickler: let's try this ^11:43
openstackgerritPratik Raj proposed openstack/coverage2sql master: optimize size and time using --no-cache-dir  https://review.opendev.org/73665011:47
*** nweinber has joined #openstack-qa11:48
fricklerAJaeger: hmm, I'm just looking at the node I held and it does seem to have USE_PYTHON3=True. it also does have pip2.7. looking at the error again, the search path seems wrong, it doesn't include /usr/local/bin: "which: no pip2.7 in (/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/sbin:/usr/sbin:/sbin:/usr/sbin/)"11:48
fricklerand that's where pip2.7 resides11:48
*** raildo has joined #openstack-qa11:50
openstackgerritPratik Raj proposed openstack/devstack master: optimize size and time using --no-cache-dir  https://review.opendev.org/73665911:52
openstackgerritPratik Raj proposed openstack/devstack-plugin-ceph master: optimize size and time using --no-cache-dir  https://review.opendev.org/73666211:53
fricklerdoes anyone know that --no-cache-dir spammer? looks like automatically generated patches editing comments to utter nonsense11:56
openstackgerritPratik Raj proposed openstack/devstack-tools master: optimize size and time using --no-cache-dir  https://review.opendev.org/73666811:56
AJaegerno, first time spammer ;(11:56
kopecmartin hi, has anyone heard about some changes regarding tox and virtualenv? I noticed some jobs started failing on 'virtualenv: command not found' or 'tox: No such file or directory' .. so i'm tracking now how these dependencies was installed before ..12:04
openstackgerritPratik Raj proposed openstack/grenade master: optimize size and time using --no-cache-dir  https://review.opendev.org/73668012:05
AJaegerkopecmartin: did you read the mailing list? ianw send details there...12:05
kopecmartinoh, i didn't lately .. let me check12:06
openstackgerritPratik Raj proposed openstack/hacking master: optimize size and time using --no-cache-dir  https://review.opendev.org/73668112:06
AJaegerfrickler: "New contributor" - the adjutant change of the series has "Welcome, New Contributor" ;(12:06
kopecmartinAJaeger: i see now, thanks12:06
fricklerAJaeger: working alphabetically, bashate came early, yes12:08
*** gcheresh has quit IRC12:08
AJaegerhttps://review.opendev.org/#/c/736447/ - read the commetns there ;(12:09
*** yamamoto has quit IRC12:12
*** gcheresh has joined #openstack-qa12:14
*** rpittau|bbl is now known as rpittau12:20
*** rh-jelabarre has joined #openstack-qa12:28
openstackgerritMartin Kopec proposed openstack/tempest master: Ensure tox is installed in run-tempest role  https://review.opendev.org/73670112:30
AJaegerfrickler: the changes are all abandoned, read #opendev for details12:33
fricklerAJaeger: yeah I saw that, thanks for the fast action to all involved12:34
*** jpena|lunch is now known as jpena12:38
*** paras333 has joined #openstack-qa13:05
openstackgerritThomas Morin proposed openstack/tempest master: object storage: fix and cleanup header checks  https://review.opendev.org/73629213:07
openstackgerritThomas Morin proposed openstack/tempest master: object storage: fix and cleanup header checks  https://review.opendev.org/73629213:19
*** yamamoto has joined #openstack-qa13:28
*** yamamoto has quit IRC13:34
*** abdysn has quit IRC13:49
gmannianw: frickler for 'virtualenv: command not found' happening on devstack based job, do we need to include 'ensure-virtualenv' role on all subnodes? or that is done in base job. or that you made part of 'ensure-pip' but i think that you said for xenial case only?13:50
openstackgerritDan Smith proposed openstack/devstack master: WIP: Try a more unified glance upload helper  https://review.opendev.org/73623113:53
toskygmann: is it about the failure of the grenade/py2 job in stein (which made you disable it)?13:55
gmanntosky: its in multiple places of multinode job. I thought it is for legacy job only which need d-g fix.13:56
toskyat least the candidate grenade job on train (so from stein) is failing with that error (missing virtualenv)13:59
toskyso maybe something py2/xenial?13:59
gmannthere are other project job also14:00
*** ysirndjuro has joined #openstack-qa14:00
*** rfolco|rover is now known as rfolco14:01
*** rcernin_ has quit IRC14:14
clarkbgmann: I think ensure-virtualenv should happen on all nodes and ensure-pip should be used first if running on xenial14:32
gmannclarkb: yeah, i was searching if zuul job does that or we need to do in devstack base job.14:32
clarkbthe idea is jobs will do it becausethe problems arose when we assumed everyone wanted it14:33
clarkbthe change we made was forcing an opt in14:33
gmannok.14:37
gmannclarkb: do you think doing from devstack master can create issue? i am seeing error only on stable branches for now.14:43
clarkbgmann: hrm I wouldve expected it to fail consistently regardless of branch unless newer devstack installs that stuff on itsown14:45
clarkbdoes latest devstack still use d-g roles?14:45
clarkbmaybe we should identify why current devstack works and backport that fix14:45
gmanni think yes, let me confirm14:45
clarkbgmann: do you have links to failed and successful builds? I'll try to help compare over breakfast14:49
gmannclarkb: this backport is failing on neutron-grenade-dvr-multinode  - https://review.opendev.org/#/c/735948/2 where master is all good.14:50
gmannclarkb: we just prepare the d-g role but test-matrix is removed so we can remove it from here too- https://opendev.org/openstack/devstack/src/branch/master/.zuul.yaml#L21314:55
clarkbI think that may be part of ehy14:56
clarkbtest-matrix uses a venv right?14:56
clarkbbut if we dont use it in master we'd avoid the problem?14:56
clarkbcan we avoid the use of test matrix on stable branches or should we make a fix to install virtualenv? (also still need to confirm that is the issue)14:57
gmannclarkb: ohk, i think that was needed till queens only. let me backport that14:57
gmannclarkb: but ussuri also failing where no test-matrix.14:59
*** Luzi has joined #openstack-qa15:01
clarkbgmann: https://zuul.opendev.org/t/openstack/build/9418dbd339724ad98582c3411cbe50fb/log/job-output.txt#13474 its failing in old on ussuri which is actually train15:03
clarkbgmann: that same code seems to exist on master in lib/infra so I'm trying to figure out why master is ok15:04
gmannyeah,  but i was confused with why other neutron-grenade-multinode job pass..15:05
clarkbya I'm looking at that aspect now15:06
clarkbgmann: I understand it now. If we are running with python2 we set VIRTUALENV_CMD to virtualenv and if we are running with python3 we set it to python3 -m venv15:08
clarkbgmann: it is only failing on python2 because virtualenv is not there15:08
clarkbI'm guessing we don't want to change the python version on those jobs so we should probably add something like if xenial ensure-pip fi ensure-virtualenv15:09
clarkbgmann: do grenade jobs parent to devstack jobs or are they separate? /me is looking now too15:12
clarkbgrenade-base is its own job base15:12
gmannyeah from multinode job not devstack15:13
gmannbut passing one is also py2 -https://zuul.opendev.org/t/openstack/build/a1dedf64ac304c4ba3e17429f6a3d39c/log/job-output.txt#220515:13
openstackgerritVasyl Saienko proposed openstack/tempest master: Update skip check for ServersTestBootFromVolume  https://review.opendev.org/73674215:14
clarkbgmann: https://zuul.opendev.org/t/openstack/build/9ae4c5380deb436692cce72ee3292901/log/job-output.txt#2488 that is a passing one I looked at15:14
*** ccamposr has joined #openstack-qa15:14
clarkbweird15:15
clarkbI wonder if we have a stale image15:16
*** ccamposr__ has quit IRC15:17
gmannthe one you saw is master which is py3 always as ussuri-?master.  but Ussuri job which install train devstack end up with py215:18
gmannbut another confusion here is even ususri grenade base job use USE_PYTHON3=ture so not sure how they endup on py215:19
clarkbgmann: I think that is it15:21
clarkbthe python2 passing job ran in rax-iad15:21
AJaegerfrickler: setting USE_PYTHON3 did not help the train openSUSE job in https://review.opendev.org/736647 . So, if anybody hasan idea why openSUSE fails on train (see also backscroll for what frickler found), would be great15:21
clarkbthe cloud's most recent bionic image is from about June 15 00:00 UTC15:21
clarkbbut the removal of pip and virtualenv merged at about June 15 00:37 UTC15:22
clarkbI've not yet confirmed this for sure as the timestamps are all a bit odd but working on it15:22
gmannAJaeger: does it end up running on py2?15:22
clarkbgmann: I think we should assume the python2 failures can be widespread and we need to add ensure-virtualenv and ensure-pip on grenade and devstack15:23
gmannyeah, train gate is all mixed of py2 and py3.15:23
gmannso ensure-pip and then ensure-virtualenv without condition should work fine for py3 also right?15:24
clarkband I've just foudn our builders aren't building new images because we've run out of disk I think because the old plain images are still around15:24
clarkbgmann: yes that should be safe for both versions15:24
gmannok.15:24
clarkbI'm going to work on cleaning up our builders this morning if you can work on the job side15:24
gmannsure, thanks15:24
gmannand d-g side, what to do on that ?15:25
AJaegergmann: https://zuul.opendev.org/t/openstack/build/fe9a3469360d45a39e2c2e20a394940d/log/job-output.txt#3044 - install_pip does some crazy stuff and then pip27 is in /usr/local/bin but devstack does not find it15:25
gmannask projects to move to zuulv3?15:25
gmannclarkb: found that devstack is all good, ianw did  that in train and ussuri onwards is all py3. I will add on greande side - https://review.opendev.org/#/c/727000/15:29
clarkbfor d-g yes I think the goal should continue to be deleting it from as many places as possible15:30
clarkbgmann: cool thanks15:30
openstackgerritMerged openstack/grenade stable/ussuri: Use ironic-grenade  https://review.opendev.org/73184215:32
*** slaweq has quit IRC15:33
*** gcheresh has quit IRC15:38
openstackgerritGhanshyam Mann proposed openstack/grenade stable/ussuri: Add ensure-virtualenv role  https://review.opendev.org/73675015:39
gmannclarkb: ^^ for testing , added depends-on in this https://review.opendev.org/#/c/735948/15:41
gmannthanks for help.15:41
gmannclarkb: frickler can you review these backports which is needed bymain backport of grenade zuulv3 - https://review.opendev.org/#/c/699515/5 https://review.opendev.org/#/c/736283/415:43
gmannmerging them first will be easy to backport the venv fix also15:44
clarkbdone15:46
gmannthanks15:47
clarkbgmann: for the image cleanup we are landing a change ot upadte nodepool configs which i'll monitor then do any additional cleanup by hand if that becomes necessary15:47
clarkband that should hopefully get us to a more consistent place which mgiht hurt for a minute but help us find any remaining problems quickly15:47
gmann+1.15:48
*** bhagyashris is now known as bhagyashris|away15:49
*** Luzi has quit IRC16:01
*** lucasagomes has quit IRC16:03
toskygmann: so if we have this https://review.opendev.org/#/c/727000/ in grenade too, does it mean we can reenable the py2 grenade job in https://review.opendev.org/#/c/736284/ ?16:03
gmanntosky: testing it on ussuri first to see how it behaves16:04
toskyack16:04
*** ccamposr__ has joined #openstack-qa16:09
*** ccamposr has quit IRC16:11
*** gfidente is now known as gfidente|afk16:49
openstackgerritGhanshyam Mann proposed openstack/tempest master: Fix request body for compute unshelve action API  https://review.opendev.org/67028716:51
*** rpittau is now known as rpittau|afk17:00
openstackgerritMerged openstack/grenade stable/train: Always source lib/apache when lib/tls is used  https://review.opendev.org/69951517:06
openstackgerritMerged openstack/grenade stable/train: Add GRENADE_USE_EXTERNAL_DEVSTACK to skip part of the setup  https://review.opendev.org/73628317:06
*** TomStappaerts has quit IRC17:07
*** TomStappaerts has joined #openstack-qa17:07
*** TomStappaerts has quit IRC17:08
*** dtantsur is now known as dtantsur|afk17:10
*** jpena is now known as jpena|off17:21
*** ccamposr has joined #openstack-qa17:29
*** ccamposr__ has quit IRC17:32
openstackgerritMerged openstack/devstack-plugin-ceph master: support focal  https://review.opendev.org/72836217:44
*** gcheresh has joined #openstack-qa17:55
*** ralonsoh has quit IRC17:57
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Updates script to use latest version of Ceph  https://review.opendev.org/67672218:20
*** gcheresh has quit IRC18:26
kopecmartingmann: hi, how do we handle installation of tox in tempest? python-tempestconf stein job is failing on not having tox installed, however, train and master jobs are fine18:28
clarkbkopecmartin: I believe that the devstack base job is using the ensure-tox role in devstack/playbooks/pre.yaml18:32
clarkber thats not the right path18:33
kopecmartinclarkb: ah, i see, playbooks/tox/pre.yaml ?18:34
clarkbkopecmartin: https://opendev.org/openstack/devstack/src/branch/stable/stein/playbooks/tox/pre.yaml#L8 that path now to double check which jobs use that18:34
clarkbyup18:34
clarkbthat is used by devstack-tox-functional-consumer18:34
clarkbah there it is in devstack/lib/tempest there is a tox install in devstack itself18:35
gmanntox playbooks only for functional jobs18:35
clarkbkopecmartin: gmann https://opendev.org/openstack/devstack/src/branch/stable/stein/lib/tempest#L691-L708 that should install it for any devstack job that enables the tempest devstack service18:36
gmannyeah, kopecmartin you have link for stein patch failing for tox not there?18:37
kopecmartingmann: f.e this patch https://review.opendev.org/#/c/736527/218:39
kopecmartinclarkb: gmann: the jobs in python-tempestconf use a devstack base job but i think they don't use devstack to install tempest, which could explain it18:40
kopecmartinhowever it's weird, that it works that way for train and master18:40
gmannkopecmartin: ohk, how it install then?18:41
kopecmartingmann: here is the playbook: https://github.com/openstack/python-tempestconf/blob/master/playbooks/python-tempestconf-tempest-devstack.yaml18:41
gmannkopecmartin: so this is first place you are preparing the tempest venv right - https://github.com/openstack/python-tempestconf/blob/505e20a4dd771536257a4c4db645da707f727668/roles/install-plugins/tasks/main.yaml#L318:51
kopecmartingmann: yes, but that role is executed only in python-tempestconf-tempest-devstack-admin-plugins job which works18:52
kopecmartingmann: hmm, yeah, that means the tempest must be installed by devstack somehow, not specifically set in our jobs, so default behaviour probably18:53
kopecmartingmann: stein job differs from train/master one only by setting override-checkout and branch, f.e. https://github.com/openstack/python-tempestconf/blob/master/.zuul.yaml#L166-L16818:54
*** vishalmanchanda has quit IRC18:55
gmannkopecmartin: i am trying to understand the flow where you are running tox from cloned tempest18:55
*** ysirndjuro has left #openstack-qa18:55
*** ysirndjuro has quit IRC18:55
kopecmartingmann: based on the playbook i sent above, we run devstack (run-devstack) role, then we prepare tempest by running tempest roles such as setup-tempest-run-dir, setup-tempest-data-dir, acl-devstack-files18:56
kopecmartingmann: then we generate our own tempest.conf which is put to a directory where devstack installed tempest18:57
kopecmartingmann: and then we run run-tempest role18:57
gmanni missed this in that role-  chdir: "{{devstack_base_dir}}/tempest"18:57
gmanngot it18:58
kopecmartingmann: any chance devstack in stein branch doesn't install tox? that's the only difference i can think of18:58
gmannkopecmartin: may be it depends on image but i think you can use ensure-tox role for sure as you are preparing the tempest vnenv  or the way devstack does18:59
gmannv19:00
gmannhttps://opendev.org/openstack/devstack/src/branch/stable/stein/lib/tempest#L691-L70819:00
kopecmartingmann: btw, how should it work? should be roles independent in meaning if the role uses tox it should also check if it's installed? otherwise the role depends on parents which may or may not install the dependency19:04
kopecmartinbut on the other side it would led to  many checks on different levels i guess19:04
gmannkopecmartin: ensure-tox does verify if it is installed19:08
kopecmartingmann: yes, the question is if want something like this or not: https://review.opendev.org/#/c/736701/19:09
gmannkopecmartin: by having tox_envlist default as smoke in run-tempest role, i think it is fine to do that and let users use run-tempest independently19:20
gmannthough i am not sure why python-tempestconf job run-tempest twice 1. before plugins install and 2. after plugins install19:21
gmannif some day you remove the first run-tempest then i will suggest to add ensure-tox in your install-plugins role too as you are creating tox env there at very first time19:21
gmannkopecmartin: ttyl, need to go for lunch.19:22
*** gmann is now known as gmann_afk19:22
openstackgerritJens Harbott (frickler) proposed openstack/devstack stable/train: DNM: Test with additional path  https://review.opendev.org/73681219:29
*** yamamoto has joined #openstack-qa19:32
*** xinranwang_ has quit IRC19:33
artomWhat's the etiquette around adding a waiter that'll be used only in a plugin?19:36
artomSpecifically, a nova service waiter?19:36
artomAdding code to tempest that would be dead in tempest seems weird19:37
*** gmann_afk is now known as gmann19:37
artomBut... it's a generic enough thing that I could see others using19:37
*** yamamoto has quit IRC19:37
gmannartom: what waiter? service up or not? or some resource status?19:37
artomgmann, yeah, talking about specifically Nova's os-services API, waiting for a service to be up/down/whatever19:38
artomOh.19:38
artom*facepalm*19:38
artomNothing in tempest would be starting/stopping these19:38
artomSo nothing would need to wait.19:38
gmannartom: yeah, adding in plugin side is needed not in tempest as no service control in tempest.19:38
artomgmann, yep, that seems obvious in retrospect. Sorry for the noise.19:39
gmannnp!19:40
gmannclarkb: frickler ensure-virtualenv fix the grenade issue, can you review -https://review.opendev.org/#/c/736750/1 (passed the earlier failing multinode grenade dvr job in https://review.opendev.org/#/c/735948/3)19:42
gmannkopecmartin: i am +2 on this - https://review.opendev.org/#/c/736701/119:42
kopecmartingmann: thanks19:45
openstackgerritDan Smith proposed openstack/devstack master: WIP: Try a more unified glance upload helper  https://review.opendev.org/73623119:45
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Add ensure-virtualenv role  https://review.opendev.org/73681919:51
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628420:00
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Update grenade setting for stable/train  https://review.opendev.org/73628520:01
gmannwe should have grenade jobs on train working now ^^, waiting for results20:02
toskyyay!20:04
toskythanks20:04
*** gmann is now known as gmann_afk20:10
*** ccamposr has quit IRC20:12
*** nweinber has quit IRC20:14
*** ccamposr has joined #openstack-qa20:18
*** ccamposr has quit IRC20:19
clarkbgmann_afk: done. Maybe ianw can approve it when the day starts on that side of the planet20:21
*** slaweq has joined #openstack-qa20:23
*** rfolco has quit IRC20:52
*** gcheresh has joined #openstack-qa20:54
*** gcheresh has quit IRC21:00
*** raildo has quit IRC21:10
*** rcernin_ has joined #openstack-qa21:54
*** markvoelker has joined #openstack-qa21:58
*** markvoelker has quit IRC22:02
*** gmann_afk is now known as gmann22:04
*** tonyb has joined #openstack-qa22:05
gmannclarkb: thanks22:05
toskygmann: interesting; grenade is going to fail on https://review.opendev.org/736284, but it passed on https://review.opendev.org/#/c/736285/22:06
gmannmultinode job? checking22:07
toskyno, the base grenade job22:11
toskyand that doesn't make much sense (unless I misread the logs)22:12
*** rcernin_ has quit IRC22:13
*** slaweq has quit IRC22:13
openstackgerritGhanshyam Mann proposed openstack/grenade stable/train: Setup native grenade Zuul v3 jobs  https://review.opendev.org/73628422:19
gmanntosky: ^^, squash the branch setting backport also as it was doing train->master upgrade22:19
gmannlet's see if tat error continue on py2 job we will make it voting and move forward and later we can debug the py2 job.22:20
toskyoh, right22:23
toskynew was initially checking out as train, but the logic was against master22:23
toskyplease note there is devstack stein backport as dependency, which is needed in order to have proper logs22:24
toskythis one : https://review.opendev.org/#/c/736475/22:24
gmannyeah, i saw that. once things is green i will review that.22:24
gmannthanks for that22:24
toskylet's see tomorrow o/22:25
*** tosky has quit IRC22:25
gmanni will check after my bike ride if anything extra needed22:26
*** gfidente|afk has quit IRC22:29
*** ccamposr has joined #openstack-qa22:33
*** tkajinam has joined #openstack-qa23:00
*** rcernin_ has joined #openstack-qa23:15
*** rcernin_ has quit IRC23:15
*** rcernin has joined #openstack-qa23:16
*** ccamposr__ has joined #openstack-qa23:17
openstackgerritGhanshyam Mann proposed openstack/grenade master: Adding document on Extended Maintainance testing policy  https://review.opendev.org/73686623:18
*** ccamposr has quit IRC23:20
*** yamamoto has joined #openstack-qa23:34
*** yamamoto has quit IRC23:39
openstackgerritGhanshyam Mann proposed openstack/grenade master: Adding document on Extended Maintainance testing policy  https://review.opendev.org/73686623:48
*** rfolco has joined #openstack-qa23:58

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