Tuesday, 2019-03-12

*** cshen has joined #openstack-qa00:15
*** cshen has quit IRC00:20
*** gfidente|afk has quit IRC00:22
*** lbragstad has quit IRC00:49
*** dave-mccowan has joined #openstack-qa00:54
openstackgerritSergey Vilgelm proposed openstack/patrole master: Updates Launchpad references to Storyboard  https://review.openstack.org/57626101:09
*** whoami-rajat has joined #openstack-qa01:12
*** mvkr has quit IRC01:36
*** mvkr has joined #openstack-qa01:38
*** mvkr has quit IRC01:49
*** mvkr has joined #openstack-qa01:52
*** hongbin has joined #openstack-qa01:53
*** cshen has joined #openstack-qa02:15
*** cshen has quit IRC02:20
*** openstackstatus has quit IRC02:22
*** openstack has joined #openstack-qa02:23
*** ChanServ sets mode: +o openstack02:23
*** openstackstatus has joined #openstack-qa02:23
*** ChanServ sets mode: +v openstackstatus02:23
*** lbragstad has joined #openstack-qa02:58
*** lbragstad has quit IRC03:07
*** yamamoto has joined #openstack-qa03:30
*** hongbin has quit IRC03:33
*** dave-mccowan has quit IRC03:40
*** rh-jelabarre has quit IRC03:51
*** ramishra has joined #openstack-qa04:02
*** mordred has quit IRC04:14
*** afazekas has quit IRC04:14
*** mordred has joined #openstack-qa04:21
*** yamamoto has quit IRC04:22
*** yamamoto has joined #openstack-qa04:22
*** markvoelker has joined #openstack-qa04:35
*** mordred has quit IRC04:37
*** markvoelker has quit IRC04:40
*** mordred has joined #openstack-qa04:46
*** markvoelker has joined #openstack-qa05:06
*** sridharg has joined #openstack-qa05:22
*** larainema has joined #openstack-qa05:26
*** rf0lc0 has joined #openstack-qa06:00
*** slaweq_ has joined #openstack-qa06:01
*** slaweq has quit IRC06:07
*** rfolco|ruck|off has quit IRC06:07
*** andreaf has quit IRC06:09
*** andreaf has joined #openstack-qa06:12
*** e0ne has joined #openstack-qa06:18
*** masayukig[m] is now known as masayukig[m]106:35
*** masayukig[m]1 is now known as masayukig[m]206:35
*** masayukig has joined #openstack-qa06:41
*** Luzi has joined #openstack-qa06:53
*** ccamposr has joined #openstack-qa07:06
*** masayukig has quit IRC07:22
*** cshen has joined #openstack-qa07:34
*** pcaruana has joined #openstack-qa07:39
*** gcheresh has joined #openstack-qa07:40
*** pcaruana has quit IRC07:43
*** pcaruana has joined #openstack-qa07:43
*** kopecmartin|off is now known as kopecmartin07:53
*** rpittau|afk is now known as rpittau07:59
*** e0ne has quit IRC08:17
*** e0ne has joined #openstack-qa08:23
*** iurygregory has joined #openstack-qa08:35
*** e0ne has quit IRC08:39
*** jpena|off is now known as jpena08:44
*** dtantsur|afk is now known as dtantsur08:52
*** iurygregory has quit IRC09:06
*** lucasagomes has joined #openstack-qa09:16
dtantsurhi folks! has anyone seen No .testr.conf file was found for local execution in stable/queens grenade?09:20
dtantsurwhy is it even looking for .testr.conf if the new version uses stestr?09:21
*** iurygregory has joined #openstack-qa09:31
*** rf0lc0 is now known as rfolco|ruck09:34
dtantsur(some discussion going on on #openstack-stable right now)09:35
*** tosky has joined #openstack-qa09:44
*** gfidente has joined #openstack-qa09:45
openstackgerritNoam Angel proposed openstack/tempest master: allow create_server with vnic_type and port_profile from kwargs  https://review.openstack.org/63338310:08
*** e0ne has joined #openstack-qa10:16
*** gvrangan has joined #openstack-qa10:17
*** iurygregory has quit IRC10:29
*** iurygregory has joined #openstack-qa10:30
fricklertosky: commented on your patches, sorry if I'm being too critical/skeptical. I really do appreciate all the work you are putting into this11:07
toskyfrickler: I guess there is a request for structural changes11:08
fricklerdtantsur: do you have a patch/log to look at? there was some issue with patches being backported that shouldn't have been, not sure if it was queens or pike, though11:08
toskyfrickler: the issue with the enable_plugin written last is a blocker for the grenade plugin11:09
toskygrenade job*11:09
dtantsurfrickler: http://logs.openstack.org/85/641585/1/check/ironic-grenade-dsvm/623a2f5/logs/grenade.sh.txt.gz#_2019-03-07_08_34_11_668 (also see discussion on #openstack-stable)11:10
*** chkumar|pto is now known as chandankumar11:11
toskybecause enable_plugins set at the beginning means that it is parsed at the beginning11:12
fricklerdtantsur: hmm, the failure seems to be on the "old" side, i.e. pike. do you also have a pointer to a run where this was passing earlier? or may this have been broken going unnoticed for some time?11:22
dtantsurfrickler: the last change to ironic merged in early Feb11:23
dtantsurfrickler: it was https://review.openstack.org/#/c/635062/ I think11:23
*** dave-mccowan has joined #openstack-qa11:24
openstackgerritLuigi Toscano proposed openstack-dev/devstack master: zuul job: write the enable_plugin lines last  https://review.openstack.org/63872111:31
toskyrebased, I had to do it anyway11:32
*** chandankumar is now known as chkumar24611:35
fricklerdtantsur: hmm, the failing test has os-testr=1.0.0 installed in the smoke venv, the passing test doesn't have that http://logs.openstack.org/85/641585/1/check/ironic-grenade-dsvm/623a2f5/logs/grenade.sh.txt.gz#_2019-03-07_08_34_10_138 vs. http://logs.openstack.org/62/635062/1/gate/ironic-grenade-dsvm/aaaa5fb/logs/grenade.sh.txt.gz#_2019-02-06_20_53_00_27411:37
fricklerdtantsur: this may need https://review.openstack.org/#/c/639504/1 as a fix, though I'm not 100% sure it is the same issue. gmann or tosky may know better11:38
dtantsurright, maybe11:38
toskyfrickler: oh, I thought that it affected only the native Zuul v3 jobs11:39
toskybut if those patches could be merged (master and all backports), that would unlock few sahara pike patches too11:40
*** cshen has quit IRC11:51
*** rh-jelabarre has joined #openstack-qa11:53
*** jpena is now known as jpena|lunch11:56
fricklerdtantsur: tosky: so it is https://review.openstack.org/#/c/563617/2/lib/tempest which is breaking you, because it is applying old reqs to ironic-tempest-plugin installation, when it really should be using master reqs12:01
fricklerwhich is a similar issue to the pending patch above, but not the same12:02
*** cshen has joined #openstack-qa12:02
toskyfrickler: iirc there is another patch12:02
toskyfrickler: https://review.openstack.org/#/q/project:openstack-dev/devstack+branch:stable/pike+status:open+owner:%22Ghanshyam+Mann+%253Cgmann%2540ghanshyammann.com%253E%2212:03
*** iurygregory has quit IRC12:03
*** iurygregory has joined #openstack-qa12:03
fricklertosky: https://review.openstack.org/#/c/640420/1/lib/tempest looks like it fixes this, yes12:04
dtantsurthanks frickler !12:17
*** markvoelker has quit IRC12:18
*** gvrangan has quit IRC12:22
*** raildo has joined #openstack-qa12:29
*** e0ne has quit IRC12:36
*** iurygregory has quit IRC12:39
*** mjturek has joined #openstack-qa12:40
*** mjturek has quit IRC12:45
*** yamamoto has quit IRC12:47
*** yamamoto has joined #openstack-qa12:48
*** mriedem has joined #openstack-qa12:53
*** yamamoto has quit IRC12:53
*** jpena|lunch is now known as jpena13:00
*** slaweq_ is now known as slaweq13:08
*** yamamoto has joined #openstack-qa13:12
*** gfidente has quit IRC13:13
*** lbragstad has joined #openstack-qa13:17
*** dave-mccowan has quit IRC13:22
*** irclogbot_2 has quit IRC13:26
*** e0ne has joined #openstack-qa13:28
*** irclogbot_2 has joined #openstack-qa13:28
*** dave-mccowan has joined #openstack-qa13:34
*** lennyb has quit IRC13:51
*** awaugama has joined #openstack-qa13:56
*** openstack has joined #openstack-qa15:38
*** ChanServ sets mode: +o openstack15:38
*** openstackstatus has joined #openstack-qa15:39
*** ChanServ sets mode: +v openstackstatus15:39
*** e0ne has quit IRC15:45
*** dulek has joined #openstack-qa15:51
*** gcheresh has quit IRC15:52
*** e0ne has joined #openstack-qa15:58
openstackgerritTom Barron proposed openstack-dev/devstack master: Safety check for python version in get_pip_command  https://review.openstack.org/62949316:11
openstackgerritMerged openstack-dev/devstack master: Use master upper-constraints when installing tempest plugins  https://review.openstack.org/63963016:15
*** cshen has quit IRC16:23
gmanndansmith: frickler yeah once we get those merged, devstack will install tempest plugins with master constraint consistently16:49
dansmithgmann: do you need me to do something?16:49
gmannfrickler: master patch is merged now, could you check these too - https://review.openstack.org/#/c/639503/ and https://review.openstack.org/#/c/639504/16:50
gmanndansmith: sorry wrong pin. please ignore16:51
gmannit was for dtantsur16:51
dansmithgmann: np, thanks for making me feel important briefly :D16:51
gmann:)16:51
*** ccamposr has quit IRC16:52
*** ccamposr has joined #openstack-qa16:52
gmannlbragstad: yeah, or let me recheck the base one. because we need to depends-on 642102/6 not latest16:53
dtantsurdansmith: glad that you asked, could fix the ironic gate? :)16:53
dtantsurthanks gmann!16:54
dansmithheh16:54
gmannianw: for you, master patch is merged now. - https://review.openstack.org/#/q/topic:bug/1816022+(status:open+OR+status:merged)16:55
*** e0ne has quit IRC16:55
*** rpittau is now known as rpittau|afk16:59
*** kopecmartin is now known as kopecmartin|off17:05
openstackgerritSergey Vilgelm proposed openstack/patrole master: DNM: Retrying create_test_server  https://review.openstack.org/64282917:15
*** agurenko has quit IRC17:19
lbragstadgmann 642102/6 ?17:20
lbragstadhttps://review.openstack.org/#/c/624218/8 needs to depend on it - but the dependency there looks ok17:22
gmannlbragstad: sorry this one - /624218/617:22
lbragstadah17:22
lbragstadwhy patch set 6 and not latest?17:22
gmannlbragstad: for testing if that fix the test failure occurred in PS 6. keystone PS8 has depends-on remove admin domain scope patch17:23
gmannonce it is confirmed then we can go as you mentioned. 624218 depends on tempest fix17:24
lbragstadok - so you're just going to have 642102 depend on 624218/6 to verify it?17:25
lbragstadand then we'll update 624218 depend on the latest 642102?17:27
gmannyeah17:27
lbragstadok - cool17:28
lbragstadgmann want me to update that?17:28
*** ccamposr has quit IRC17:28
gmanni did recheck as of now, once we confirm the fix then I will ping you so that you can update keystone patch17:29
lbragstadoh - i thought the depends-on here was ok? https://review.openstack.org/#/c/624218/8//COMMIT_MSG@1917:30
*** gfidente is now known as gfidente|afk17:31
openstackgerritGhanshyam Mann proposed openstack/tempest master: DNM: testing with using cred domain  https://review.openstack.org/64210217:33
gmannlbragstad: i see, you updated keystone one. ^^ done17:33
*** dtantsur is now known as dtantsur|afk17:39
*** mattw4 has joined #openstack-qa17:41
*** cshen has joined #openstack-qa17:43
*** e0ne has joined #openstack-qa18:02
lbragstadgmann yup!18:02
lbragstadthanks18:02
*** e0ne has quit IRC18:06
*** cshen has quit IRC18:08
*** shardy has quit IRC18:09
openstackgerritGhanshyam Mann proposed openstack/tempest master: Use cresential domain instead of creating new one  https://review.openstack.org/64210218:10
*** sshnaidm is now known as sshnaidm|afk18:11
*** cshen has joined #openstack-qa18:13
openstackgerritGhanshyam Mann proposed openstack-dev/devstack stable/rocky: Use master upper-constraints when installing tempest plugins  https://review.openstack.org/64041818:14
openstackgerritGhanshyam Mann proposed openstack-dev/devstack stable/queens: Use master upper-constraints when installing tempest plugins  https://review.openstack.org/64041918:14
openstackgerritGhanshyam Mann proposed openstack-dev/devstack stable/pike: Use master upper-constraints when installing tempest plugins  https://review.openstack.org/64042018:14
gmannfrickler: ^^updated those for cherry pick msg. need your +2 again18:15
*** sridharg has quit IRC18:18
*** cshen has quit IRC18:18
*** e0ne_ has joined #openstack-qa18:21
*** gmann is now known as gmann_afk18:26
*** pcaruana has quit IRC18:39
dirkfrickler: thanks!18:41
*** e0ne_ has quit IRC18:41
*** jpena is now known as jpena|off18:44
openstackgerritMerged openstack-dev/devstack master: zuul job: write the enable_plugin lines last  https://review.openstack.org/63872118:45
*** cshen has joined #openstack-qa18:46
*** fnaval has quit IRC18:47
openstackgerritSergey Vilgelm proposed openstack/patrole master: DNM: Retrying create_test_server  https://review.openstack.org/64282918:48
*** cshen has quit IRC18:50
*** yamamoto has quit IRC19:13
*** larainema has quit IRC19:36
*** yamamoto has joined #openstack-qa19:49
*** yamamoto has quit IRC19:54
*** gfidente|afk has quit IRC20:09
*** e0ne has joined #openstack-qa20:20
*** dave-mccowan has quit IRC20:24
*** cshen has joined #openstack-qa20:46
*** chkumar246 is now known as chandankumar20:47
openstackgerritMerged openstack-dev/devstack master: Capture the content of 'audit.log' file  https://review.openstack.org/63875720:51
*** cshen has quit IRC20:51
*** gmann_afk is now known as gmann20:54
*** mjturek has quit IRC20:57
*** mriedem has quit IRC21:05
*** mriedem has joined #openstack-qa21:08
*** raildo has quit IRC21:10
gmanntosky: devstack_tempest_plugin name is confusing, let's name it tempest_plugin - https://review.openstack.org/#/c/639374/221:17
toskygmann: I thought it would have been better to put it under the devstack scope to avoid collisions, but no problem21:18
gmannhumm. devstack and tempest both have plugins but yeah name collisions is something we need to care also21:20
gmanni think tempest_plugins is fine hoping no one will declare same name var21:22
toskyI need to change the name of the variable in devstack_local_conf.py too, then21:22
toskyor maybe not21:22
toskywhen you say "add more documentation", do you mean adding it to doc/source ?21:23
gmannthere you do not need21:23
toskythere is an example, but we don't really document the various devstack_* keys, for example21:23
gmannREADME.rst only is enough21:24
gmannso that when people check this var definition to understand then they know about its behavior with TEMPEST_PLUGINS21:24
toskyoh, sure21:24
*** e0ne has quit IRC21:26
openstackgerritLuigi Toscano proposed openstack-dev/devstack master: zuul: new variable to easily populate TEMPEST_PLUGINS  https://review.openstack.org/63937421:26
openstackgerritSergey Vilgelm proposed openstack/patrole master: DNM: Retrying create_test_server  https://review.openstack.org/64282921:29
*** mattw4 has quit IRC21:30
*** mattw4 has joined #openstack-qa21:30
openstackgerritMerged openstack-dev/devstack master: Switch GIT_BASE to https://  https://review.openstack.org/64067421:35
openstackgerritMerged openstack-dev/devstack master: Remove git:// openstack references  https://review.openstack.org/64067521:35
openstackgerritMerged openstack-dev/devstack master: support python 3 on centos 7  https://review.openstack.org/64000221:35
openstackgerritLuigi Toscano proposed openstack-dev/grenade master: WIP Setup grenade-base, grenade, grenade-{py3,postgresql}  https://review.openstack.org/54893621:46
openstackgerritLuigi Toscano proposed openstack-dev/grenade master: DNM - grenade-heat-test job  https://review.openstack.org/63977421:46
*** awaugama has quit IRC21:51
*** mattw4 has quit IRC21:52
*** rcernin has joined #openstack-qa22:01
*** whoami-rajat has quit IRC22:12
openstackgerritMerged openstack-dev/devstack master: Replace deprecated brctl with ip commands  https://review.openstack.org/61735422:22
gmannlbragstad: one test is fixed but test_create_is_domain_project is failing - http://logs.openstack.org/18/624218/8/check/tempest-full/22ce7a2/testr_results.html.gz22:24
gmanneither i can add the hack to change the domain of self.projects_client to the created project id (which is doman) or we go and remove the domain scope itself which you are doing initially in 62479422:25
gmannand I do not know how many cases there will be for POST projects when you will add the domain roles for POST project22:26
*** mattw4 has joined #openstack-qa22:26
*** tosky has quit IRC22:43
*** cshen has joined #openstack-qa22:48
*** mriedem has quit IRC22:51
*** cshen has quit IRC22:52
openstackgerritNicolas Helgeson proposed openstack/tempest master: Immutable user source: v3 test_inherits  https://review.openstack.org/63153023:02
openstackgerritNicolas Helgeson proposed openstack/tempest master: Update v3 identity inherits tests to work w/ pre-prov  https://review.openstack.org/62828423:03
*** mattw4 has quit IRC23:06
*** mattw4 has joined #openstack-qa23:10
openstackgerritSergey Vilgelm proposed openstack/patrole master: DNM: Retrying create_test_server  https://review.openstack.org/64282923:20
*** slaweq has quit IRC23:24
openstackgerritMerged openstack-dev/devstack stable/queens: use master upper-constraints when building tempest venv  https://review.openstack.org/63950323:45
openstackgerritMerged openstack-dev/devstack stable/pike: use master upper-constraints when building tempest venv  https://review.openstack.org/63950423:45
*** yamamoto has joined #openstack-qa23:54
openstackgerritGhanshyam Mann proposed openstack-dev/devstack stable/queens: Remove use of git -C option from tempest venv creation  https://review.openstack.org/64292723:58
openstackgerritGhanshyam Mann proposed openstack-dev/devstack stable/pike: Remove use of git -C option from tempest venv creation  https://review.openstack.org/64292823:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!