Monday, 2021-03-08

*** tosky has quit IRC00:42
*** ricolin has quit IRC00:46
*** ricolin has joined #openstack-qa00:50
*** rcernin has quit IRC01:12
*** rcernin_ has joined #openstack-qa01:12
openstackgerritDan Smith proposed openstack/tempest master: Add image task validation  https://review.opendev.org/c/openstack/tempest/+/77567901:18
*** rh-jelabarre has joined #openstack-qa02:21
*** rh-jelabarre has quit IRC02:43
*** vishalmanchanda has joined #openstack-qa03:00
*** rcernin_ has quit IRC03:14
*** rcernin_ has joined #openstack-qa03:21
*** ramishra has joined #openstack-qa03:41
dansmithI've seen this a couple times now.. wonder if we have a new gate blockage queued up for milestone week? https://zuul.opendev.org/t/openstack/build/380bc6752f6443f8871d5b1c35a49bd4/log/job-output.txt#422104:40
openstackgerritDan Smith proposed openstack/devstack master: Fix get-pip URL  https://review.opendev.org/c/openstack/devstack/+/77912904:43
openstackgerritDan Smith proposed openstack/devstack stable/train: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77913004:51
*** ajitha has joined #openstack-qa05:26
*** gcheresh has joined #openstack-qa05:43
openstackgerritWenping Song proposed openstack/tempest master: Remove usage of six  https://review.opendev.org/c/openstack/tempest/+/77686307:08
*** slaweq has joined #openstack-qa07:10
*** rcernin_ has quit IRC07:14
*** ralonsoh has joined #openstack-qa07:23
*** ccamposr has joined #openstack-qa07:35
openstackgerritFelix Huettner proposed openstack/tempest master: Reactivate image before deletion  https://review.opendev.org/c/openstack/tempest/+/77916807:52
*** eolivare has joined #openstack-qa07:53
*** sboyron has joined #openstack-qa08:12
*** rpittau|afk is now known as rpittau08:23
*** whoami-rajat_ has joined #openstack-qa08:26
*** tosky has joined #openstack-qa08:34
*** brinzhang has joined #openstack-qa08:34
*** brinzhang0 has quit IRC08:36
*** gfidente has joined #openstack-qa08:54
openstackgerritLee Yarwood proposed openstack/devstack master: Update Cirros to 0.5.2  https://review.opendev.org/c/openstack/devstack/+/77917908:54
*** lucasagomes has joined #openstack-qa08:57
*** jpena|off is now known as jpena08:58
openstackgerritCarlos Gonçalves proposed openstack/devstack master: Re-apply "Fix rdo-release install"  https://review.opendev.org/c/openstack/devstack/+/77608409:26
*** Luzi has joined #openstack-qa09:31
openstackgerritElod Illes proposed openstack/devstack stable/stein: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77918909:39
openstackgerritLee Yarwood proposed openstack/tempest master: compute: Add [compute-feature-enabled]ide_bus flag  https://review.opendev.org/c/openstack/tempest/+/77563009:49
openstackgerritElod Illes proposed openstack/devstack stable/rocky: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919510:06
openstackgerritElod Illes proposed openstack/devstack stable/queens: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919610:08
openstackgerritElod Illes proposed openstack/devstack stable/pike: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919710:10
*** mugsie__ is now known as mugsie10:11
elodhi QA stable cores: train and older branches are blocked due to pip2.7 bootstrap script link change. There is a trivial path update fix for it. Note: to be able to merge the train version (due to grenade), the stein version needs to be merged first. Could you please review this? https://review.opendev.org/c/openstack/devstack/+/77918911:44
elod(and the rest afterwards :))11:45
elodthanks in advance!11:45
openstackgerritElod Illes proposed openstack/devstack stable/rocky: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919512:12
openstackgerritElod Illes proposed openstack/devstack stable/queens: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919612:13
*** jpena is now known as jpena|lunch12:31
*** artom has joined #openstack-qa12:45
*** iurygregory has quit IRC12:46
*** jpena|lunch is now known as jpena13:24
gmannelod: thanks for fix, checking13:31
elodgmann: well, actually dansmith proposed the 1st patch, I just backported the fix where it is also needed :)13:33
gmannelod: lgtm, waiting for result on queens, rocky, rest all is +2 from me13:33
elodgmann: thanks o/13:33
gmann+1, thanks dansmith too.13:33
gmannwill keep eyes on those13:33
*** nweinber has joined #openstack-qa13:41
*** rh-jelabarre has joined #openstack-qa13:41
*** rh-jelabarre has quit IRC13:45
*** rh-jelabarre has joined #openstack-qa13:52
*** mnaser has joined #openstack-qa14:03
*** amodi has joined #openstack-qa14:09
openstackgerritSampat Ponnaganti proposed openstack/patrole master: [2008683] Cleans up volume after test  https://review.opendev.org/c/openstack/patrole/+/77857314:13
dansmithgmann: \o/14:24
dansmithgmann: hopefully someone will repay the favor my +Wing my devstack patch for distributed-import ;P14:24
gmanndansmith: ah yeah, I will bring that in tomorrow QA office hour too14:25
dansmithack, thanks ;)14:26
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: nova: Make configure_ceph_nova multinode compatible  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/75632314:26
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: zuul: Introduce a multinode ceph job  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/71162514:26
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: tempest: Use a more appropriate name for the skip list  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77090614:27
lbragstadgmann https://github.com/openstack/keystone-tempest-plugin/blob/master/keystone_tempest_plugin/tests/rbac/v3/base.py#L20-L41 is a pretty re-usable test class, is there a good place to stash that in tempest?14:44
lbragstadthe implementation has been copied in glance and in cinder, but it might make sense to just put that in tempest proper14:44
*** Luzi has quit IRC14:49
openstackgerritMerged openstack/tempest master: Make _create_security_group() non-private  https://review.opendev.org/c/openstack/tempest/+/77693215:02
toskytime to start merging the get-pip fixes starting from pike? The pike job is green15:03
gmannlbragstad: nice idea, let me check cinder and glance tests if they need something from base volume/image test class from tempest.15:07
gmannlbragstad: but yes, keeping this base class in tempest can be reusable.15:08
lbragstadfwiw - i haven't landed a patch in cinder-tempest-plugin, yet15:08
lbragstadbut glance and keystone do have base RBAC test classes15:08
gmannfrickler: yoctozepto : can you please check this get-pip things to unblock the Tempest master and <=stable/train gate https://review.opendev.org/q/Id62e91b1609db4b1d2fa425010bac1ce77e9fc5115:08
gmannlbragstad: ack.15:09
gmannlbragstad: will check in evening and get back to you.15:09
lbragstadhttps://review.opendev.org/c/openstack/cinder-tempest-plugin/+/772915 is the cinder tempest plugin patch where we're talking about re-usability15:09
lbragstadok15:09
toskyyep :)15:09
lbragstadi'm going to keep writing cinder tests on top of the existing patches15:10
lbragstadand i'll just pull the base implementation out later15:10
toskysure15:10
gmannlbragstad: I think you are not consuming the tempest volume base class right?15:12
lbragstadno - it's just a class that inherits from object15:13
gmannok.15:13
openstackgerritArtom Lifshitz proposed openstack/whitebox-tempest-plugin master: WIP: TripleO multinode standalone job  https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/76286615:14
*** ccamposr__ has joined #openstack-qa15:51
*** ccamposr has quit IRC15:51
yoctozeptogmann: approving them15:52
*** ccamposr has joined #openstack-qa15:57
*** ccamposr__ has quit IRC15:57
gmannyoctozepto: thanks16:00
*** gmann is now known as gmann_afk16:01
*** ajitha has quit IRC16:15
*** ajitha has joined #openstack-qa16:41
openstackgerritLance Bragstad proposed openstack/tempest master: WIP: Add a base protection test class to tempest.lib.base  https://review.opendev.org/c/openstack/tempest/+/77929716:52
dansmithyoctozepto: hey, we're friends right?16:52
lbragstadgmann_afk ^ that's going to fail pretty bad... but i'm stashing it in review for now until we know where exactly we want to put that16:53
*** lucasagomes has quit IRC17:00
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: zuul: Introduce a multinode ceph job  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/71162517:03
openstackgerritLee Yarwood proposed openstack/devstack-plugin-ceph master: tempest: Use a more appropriate name for the skip list  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/77090617:03
*** eolivare has quit IRC17:03
*** jpena is now known as jpena|brb17:15
*** iurygregory has joined #openstack-qa17:17
*** hamalq has joined #openstack-qa17:29
*** jpena|brb is now known as jpena17:37
yoctozeptodansmith: hey, that's kind of an odd question to ask but the answer is yes, there is nothing wrong between us as far as I am aware; I hope you see it the same :P ; why the question? :D17:43
dansmithyoctozepto: wanted to ask a favor amongst buddies, but I'm on a call at the moment :P17:43
yoctozeptodansmith: haha, all right17:44
*** iurygregory has quit IRC17:49
*** gfidente is now known as gfidente|afk18:00
*** rpittau is now known as rpittau|afk18:16
*** amodi has quit IRC18:16
*** dtantsur is now known as dtantsur|afk18:17
*** ccamposr has quit IRC18:24
*** amodi has joined #openstack-qa18:27
*** jpena is now known as jpena|off18:30
*** amodi has quit IRC18:34
dansmithyoctozepto: sorry about that.. my joke was going to be a lot more funny if that call hadn't been in the way :P18:39
dansmithyoctozepto: so we have this glance feature that underwent a lot of drama and landed late, which we really want to see some "soak time" in exposure to the "firehose" before the release18:40
dansmithand to do that, we need this devstack patch which has been sitting for a while: https://review.opendev.org/c/openstack/devstack/+/77048718:40
dansmithyoctozepto: I know we're wedged right now, but if you could let me know if you have any critical feedback I can work on in parallel, or slap a +2 on there to help make it an easy approve when the gate is unwedged, I'd really really appreciate it18:41
*** iurygregory has joined #openstack-qa18:46
*** stand has quit IRC19:15
*** stand has joined #openstack-qa19:22
openstackgerritMerged openstack/devstack stable/stein: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77918919:28
openstackgerritMerged openstack/devstack stable/rocky: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919519:28
*** gmann_afk is now known as gmann19:32
gmannlbragstad: ack.19:37
*** amodi has joined #openstack-qa19:41
*** whoami-rajat_ is now known as whoami-rajat19:44
yoctozeptodansmith: you shall retry this joke again some time :P19:50
dansmithyoctozepto: you (apparently) know me well... buddy :D19:50
yoctozeptodansmith: hah, let it be19:51
yoctozeptoI opened a tab with this change19:51
dansmithyoctozepto: thanks19:51
yoctozeptoI will try to get to it tomorrow19:51
yoctozepto:-)19:52
dansmithappreciate it :)19:52
*** gcheresh has quit IRC20:17
openstackgerritMerged openstack/devstack stable/queens: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919620:23
*** Yarboa has quit IRC20:33
*** Yarboa has joined #openstack-qa20:33
*** gcheresh has joined #openstack-qa20:46
*** vishalmanchanda has quit IRC20:47
*** arxcruz is now known as arxcruz|off21:04
*** nweinber has quit IRC21:30
*** Yarboa has quit IRC21:33
*** Yarboa has joined #openstack-qa21:35
*** ccamposr has joined #openstack-qa21:51
*** rcernin has joined #openstack-qa21:54
*** gcheresh has quit IRC21:58
*** whoami-rajat has quit IRC22:06
*** slaweq has quit IRC22:15
*** gfidente|afk has quit IRC22:26
*** ajitha has quit IRC22:35
gmannfrickler: reminder for these, these fixes the networking-generic-switch stable/train gate  - https://review.opendev.org/q/I5b2217d85e6871ca3f7a3f6f859fdce9a50d394622:53
gmannianw: ^^ if you are around22:54
ianwgmann: looking22:57
gmannthanks22:57
openstackgerritLance Bragstad proposed openstack/tempest master: Update volume client to show encrypted volume metadata  https://review.opendev.org/c/openstack/tempest/+/77938222:59
ianwgmann: presumably when testing py27, etc. with old tempest, TEMPEST_BRANCH is set to a stable/* ?  could you set TEMPEST_VENV_UPPER_CONSTRAINTS to the same branch as TEMPEST_BRANCH?23:08
ianwor does it need to be something else entirely?23:08
*** ccamposr has quit IRC23:09
gmannianw: it depends on Tempest old tag. Say Tempest 26.0.0 is tag for end of support for stein was released in wallaby stable/stein-wallaby works fine.23:10
gmannsame branch work for sure23:11
gmannianw: like this https://review.opendev.org/c/openstack/devstack/+/775052/6/stackrc#32023:12
ianwgmann: ok, i dropped a couple of questions in review; maybe i'm not fully understanding23:14
ianws/maybe/most likely/ :)23:14
gmannchecking..23:14
gmannianw: just replied, please check if that clarify the use case https://review.opendev.org/c/openstack/devstack/+/774711/8/stackrc#30123:20
openstackgerritMerged openstack/devstack stable/train: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77913023:23
openstackgerritMerged openstack/devstack stable/pike: Fix get-pip URL for py27  https://review.opendev.org/c/openstack/devstack/+/77919723:23
ianwgmann: ok, so the plan is to check in the compatible requirements file in tempest?  or devstack?23:26
gmannianw: in devstack when devstack setup the tempest virtual env23:27
gmannand install tempest and tempest  plugins in that venv23:27
ianwok, so do we need this change on master?23:27
gmannianw: nice question, with the master change, we do not need to add the logic each time in stable branch goes to EM state. I defaulted it to 'master' for stable/train - master and as soon as we need to use old Tempest  we just update TEMPEST_BRANCH and TEMPEST_VENV_UPPER_CONSTRAINTS23:29
gmannbasically keep it configurable so that any job or stable branch can just configure both together23:30
ianwgmann: thanks for explaining.  the variable name feels a bit odd but i see how we've ended up here.  lgtm23:39
gmannianw: thanks. yeah, its for only tempest virtual env , and could not find better name23:41

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