Friday, 2020-05-22

*** tosky has quit IRC00:01
*** brinzhang_ has joined #openstack-qa01:07
*** brinzhang has quit IRC01:10
*** brinzhang has joined #openstack-qa03:03
*** brinzhang_ has quit IRC03:06
*** brinzhang_ has joined #openstack-qa03:18
*** paras333 has quit IRC03:18
*** brinzhang has quit IRC03:21
*** elod has quit IRC03:23
openstackgerritzhangboye proposed openstack/openstack-health master: Fix hacking min version to 3.0.1  https://review.opendev.org/73018003:25
*** elod has joined #openstack-qa03:35
*** brinzhang has joined #openstack-qa03:36
*** brinzhang_ has quit IRC03:40
openstackgerritzhangboye proposed openstack/os-testr master: Fix hacking min version to 3.0.1  https://review.opendev.org/73018903:54
*** ramishra has quit IRC04:01
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-qa04:33
*** paras333 has joined #openstack-qa04:52
*** paras333 has quit IRC04:57
*** ramishra has joined #openstack-qa05:08
*** paras333 has joined #openstack-qa05:20
*** paras333 has quit IRC05:24
*** paras333 has joined #openstack-qa05:52
*** paras333 has quit IRC05:57
*** nightmare_unreal has joined #openstack-qa06:18
openstackgerritzhufl proposed openstack/tempest master: Add response schema validation for volumes  https://review.opendev.org/61678306:28
*** ralonsoh has joined #openstack-qa06:33
*** udesale has joined #openstack-qa06:39
*** ramishra has quit IRC06:48
*** slaweq has joined #openstack-qa06:49
*** slaweq has quit IRC06:54
openstackgerritzhufl proposed openstack/tempest master: Adding description for testcases - network part2  https://review.opendev.org/72313106:56
*** slaweq has joined #openstack-qa06:56
openstackgerritzhufl proposed openstack/tempest master: Adding description for testcases - network part2  https://review.opendev.org/72313107:07
*** tosky has joined #openstack-qa07:34
*** jpena|off is now known as jpena07:38
*** gfidente has joined #openstack-qa08:03
*** tkajinam_ has quit IRC08:05
*** ramishra has joined #openstack-qa08:15
*** larainema has joined #openstack-qa08:29
*** paras333 has joined #openstack-qa08:36
*** elod has quit IRC08:43
*** artom has quit IRC08:45
*** artom has joined #openstack-qa08:46
*** elod has joined #openstack-qa08:50
*** elod has quit IRC08:56
*** elod has joined #openstack-qa08:58
*** elod has quit IRC09:10
*** elod has joined #openstack-qa09:10
*** ramishra has quit IRC09:37
openstackgerritzhufl proposed openstack/tempest master: Add response schema validation for volumes  https://review.opendev.org/61678309:44
*** ramishra has joined #openstack-qa10:01
*** gcheresh has joined #openstack-qa10:57
*** brinzhang has quit IRC11:09
*** gcheresh has quit IRC11:13
*** nweinber has joined #openstack-qa11:24
*** jpena is now known as jpena|lunch11:30
*** maciejjozefczyk is now known as mjozefcz|lunch11:30
openstackgerritMerged openstack/tempest master: Change bytes to str in servers_client for python3  https://review.opendev.org/70968011:41
*** paras333 has joined #openstack-qa11:52
*** paras333 has quit IRC11:56
openstackgerritLee Yarwood proposed openstack/devstack master: WIP/DNM tempest: Increase m1.nano RAM to 96MB to avoid tmpfs exhaustion  https://review.opendev.org/73030712:05
*** lajoskatona has joined #openstack-qa12:16
*** mjozefcz|lunch is now known as mjozefcz12:18
*** raildo has joined #openstack-qa12:21
*** jpena|lunch is now known as jpena12:32
*** rh-jelabarre has joined #openstack-qa12:39
*** rcernin has quit IRC12:39
*** rfolco|rover is now known as rfolco|rover|bbl12:52
*** paras333 has joined #openstack-qa12:52
lajoskatonaHi, I have a strange stable/stein issue for networking-bagpipe: https://review.opendev.org/72993612:56
lajoskatonalegacy-tempest-dsvm-networking-bgpvpn-bagpipe fails with this:12:57
lajoskatonaThe specified regex doesn't match with anythingERROR: InvocationError for command /opt/stack/new/tempest/.tox/all-plugin/bin/tempest run --regex '^networking_bgpvpn_tempest\.' --concurrency=4 (exited with code 1)12:57
lajoskatonathe suspicious thing is that for stein the tempest plugin for networking-bgpvpn was in the repo, and after that (from train) was moved to neutron-tempest-plugin12:58
lajoskatonaOne idea is that networking-bgpvpn is not installed and thus tempest can't see the plugin,12:59
toskylajoskatona: can't you just change the regex to point to the test which is neutron-tempest-plugin? Because in stein both tempest and its plugins come from master (unless a specific pinning has been used)13:01
lajoskatonatosky: that was another option in my mind, though its like changing the past :-)13:02
lajoskatonatosky: in bgpvpn repo it is define here: https://opendev.org/openstack/networking-bgpvpn/src/branch/master/devstack/devstack-gate-bagpipe-rc#L713:04
toskylajoskatona: the past would need some changes anyway :) and iirc this was the way followed after moving the tempest plugins had been moved to separate repositories13:04
lajoskatonatosky: is it possible to overwrite that from zuul.yaml? or we have to push a change to stein bgpvpn?13:04
toskylajoskatona: that's a legacy job, so you would need to change that variable in the playbook where it's defined13:05
toskymaybe it's easier to port that job as native13:06
lajoskatonatosky: is that a valid step on stable/stein?13:07
toskyuhm, I see it's just for stable/stein, stable/rocky and stable/queens13:09
toskyporting it would allow you to remove the legacy job from openstack-zuul-jobs13:10
toskygmann: what do you think?13:10
gmanntosky: yeah that is recommended way.13:16
gmannlajoskatona: issue is same you suspected, plugin in installed on system level and tempest did not find that in venv and so does trow error while running the plugin test13:17
gmannand i think that job might be using all-plugins tox env also.13:17
gmannlajoskatona: same issue was for neutron-vpnaas and suggestion was to move that to use the new separate plugin with tox -eall env - http://lists.openstack.org/pipermail/openstack-discuss/2020-January/012241.html13:18
*** udesale_ has joined #openstack-qa13:24
lajoskatonagmann: thanks, I check it13:25
lajoskatonatosky: thanks for the help13:25
gmannlajoskatona: there were few more options also to fix with the tradeoff, let me check13:25
*** udesale has quit IRC13:27
toskylajoskatona: talking about zuulv3 porting: in https://review.opendev.org/#/c/725647/, have seen the error in other network-odl multinode jobs?13:30
gmannlajoskatona: i think vpnaas issues were solved by these -https://review.opendev.org/#/q/topic:fix-stable-gate+(status:open+OR+status:merged13:30
gmannlajoskatona: can you point me the job link, i can debug after my coffee ?13:31
lajoskatonagmann: this one: https://review.opendev.org/72993613:38
lajoskatonagmann: networking-bagpipe pulls networking-bgpvpn and that has in-tree tempest-plugin13:39
gmannok13:51
lajoskatonagmann: thanks for checking it13:54
*** mailingsam has joined #openstack-qa14:21
mailingsamHello, could you review https://review.opendev.org/#/c/72980314:22
mordredgmann: I think we're trying to solve the same problem - your way might be more complete. my way was this: https://review.opendev.org/#/c/728920/14:35
mordredgmann: we'll need the pep8 fix in ksa for it to be fully green - should I try changing the patch that has the pep8 fix in ksa to depends-on your requirements patch?14:36
openstackgerritAndreas Jaeger proposed openstack/grenade master: Cleanup py27 support  https://review.opendev.org/72657714:37
mordredgmann: I think your way is better14:38
openstackgerritLee Yarwood proposed openstack/devstack master: tempest: Increase m1.nano and m1.micro RAM by 64MB to avoid tmpfs exhaustion  https://review.opendev.org/73030714:49
*** jcmdln has quit IRC15:01
*** gcheresh has joined #openstack-qa15:11
amodigmann: lyarwood can i get some input on https://review.opendev.org/#/c/730014/ this is trivial15:14
*** vishalmanchanda has joined #openstack-qa15:28
lyarwoodamodi: looking15:29
*** rfolco|rover|bbl is now known as rfolco|rover15:31
openstackgerritArchit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern  https://review.opendev.org/73001415:40
amodilyarwood: thanks!15:40
openstackgerritArchit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern  https://review.opendev.org/73001415:41
*** gcheresh has quit IRC15:45
*** lajoskatona has left #openstack-qa15:48
openstackgerritArchit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern  https://review.opendev.org/73001416:04
*** gcheresh has joined #openstack-qa16:09
lyarwoodamodi: few more comments, make sure you run $ tox -e pep8 -- -HEAD before you git review16:13
amodilyarwood: yeah my bad, im running this from a remote machine, should install relevant packages thanks again!16:14
lyarwoodamodi: tox should take care of all of that for you16:15
lyarwoodassuming that's installed ;)16:15
amodiyeah it wasn't16:15
*** jcmdln has joined #openstack-qa16:17
*** gcheresh has quit IRC16:20
amodilyarwood: we don't need any "version" assertions, do we? so just snapshotting it at the end is enough16:21
gmannmordred: thanks for depends-on. i was testing py35 green first as it failing on this test - https://5f998ee2a07b29ea81e3-da4d3406a0cf7110a347a18ceb92c153.ssl.cf1.rackcdn.com/728920/3/check/openstack-tox-py35/7a9c384/testr_results.html16:21
gmannthat passed in my DNM patch, did recheck16:26
lyarwoodamodi: https://bugs.launchpad.net/nova/+bug/1717547 - the bug you're trying to reproduce is triggered when creating a snapshot from an instance that was itself booted from an image with a version property set.16:27
openstackLaunchpad bug 1717547 in OpenStack Compute (nova) pike "Creating snapshot fails when image metadata has version field" [Low,Confirmed]16:27
amodilyarwood: yes, that's why i had this set before creating the server16:28
amodilyarwood: so revert to previous patch?16:29
lyarwoodamodi: yeah but my point was that this isn't realistic IMHO16:29
lyarwoodamodi: users would want to version their snapshots this way16:30
lyarwoodamodi: so in the context of this scenario it just makes more sense to add the version to the snapshot image, boot another server and snapshot one final time.16:30
gmannlajoskatona i cannot find networking-bgpvpn tempest plugins installed on system side otherwise all-plugins env could have detect that in tempest venv.16:31
gmanni need more time on that to check why that is not installed or detected, might need some new logging.16:32
amodilyarwood: anything works for me, i guess either way we can trigger the bug, so why not use your suggestion16:34
elodgmann: lajoskatona meanwhile went off for the weekend. about the plugin: could that be the reason of not detecting the bgpvpn tempest plugin because that is installed in different venv?16:39
gmannelod: for stein where bgpvpn tempest plugin is part of in-tree bgpvpn, it should be installed on system wied and all-plugins tox env whch enable the site packages for tempest venv should be able to detect it16:41
gmannone possibility can be bgpvpn tempest plugin is on py2.7 and  tempesr venv is py3 so could not detect. let me check if that is the case16:42
gmannelod: yeah that is the case.16:43
*** jpena is now known as jpena|off16:43
elodgmann: oh, nice catch16:43
gmannelod: for plugins in separate repo, i fixed it by switching to tox 'all' env where plugins also get installed in tempest venv but here we have in-tree plugin16:44
elodgmann: is there a way to force bgpvpn to be installed in py3 venv? (actually I don't even know if bgpvpn tempest works with py3, hmmmm...)16:45
gmannone option is (if we want to avoid to use the out of tree plugins), add bgpvpn itself as tempest plugin so that it get installed in tempest venv also16:46
gmannelod: yeah ^^ that way. let me try.16:46
elodgmann: sounds great, thanks!16:46
openstackgerritArchit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern  https://review.opendev.org/73001416:55
*** cmurphy is now known as cmorpheus17:03
gmannelod:  - https://review.opendev.org/#/c/730378/117:05
gmannelod: can you make depends-on on your fix to tests17:05
gmannwe might need devstack-gate-bagpipe-rc to fix not to DEVSTACK_GATE_TEMPEST_ALL_PLUGINS=1 but let's see17:07
gmannelod: fixing devstack-gate-bagpipe-rc in this and also teting if both fix make gate green- https://review.opendev.org/#/c/730379/17:18
*** udesale_ has quit IRC17:20
openstackgerritAndreas Jaeger proposed openstack/hacking master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72910717:27
openstackgerritAndreas Jaeger proposed openstack/devstack-plugin-ceph master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72917917:30
openstackgerritAndreas Jaeger proposed openstack/bashate master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72913417:31
elodgmann: thanks! I will add the depends-on to patch 730378 and we will see17:32
*** nightmare_unreal has quit IRC17:36
gmannelod: need bgpvon fix also, please use this for depends-on https://review.opendev.org/#/c/730379/117:41
elodgmann: ok, nested depends-on, looks fancy :]17:42
gmannelod: yeah and I am not sure if zuul job one will work or not. let's see17:42
elodgmann: depends-on updated, we will see!17:44
*** tosky has quit IRC17:50
*** tosky has joined #openstack-qa17:51
openstackgerritGhanshyam Mann proposed openstack/devstack stable/train: Use stable constraint for older Tempest  https://review.opendev.org/72972217:57
*** gcheresh has joined #openstack-qa18:09
*** gfidente is now known as gfidente|afk18:11
*** ralonsoh has quit IRC18:14
*** gfidente|afk has quit IRC18:21
openstackgerritSorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Resolve unsafe yaml.load use  https://review.opendev.org/73038918:22
*** artom has quit IRC18:30
*** gcheresh has quit IRC19:05
*** nweinber has quit IRC19:07
*** mailingsam has quit IRC19:16
*** elod has quit IRC19:26
*** elod has joined #openstack-qa19:27
elodgmann: it works! https://zuul.opendev.org/t/openstack/build/9db603a68ac4478793f4f9e1de441aa6/log/job-output.txt#241119:35
gmannelod: yeah, we need to get depends-on merge now.19:36
elodgmann: as you wrote this is not used in train, ussuri and master as this is a legacy job, but won't it break something else, e.g. rocky branch or older branches? There we are not running the latest Tempest, are we?19:47
*** slaweq has quit IRC19:49
gmannelod: yeah, we use older tempest there but doing it via venv is no harm for those also. current and new way both can work for those and stein work with new way only19:49
elodgmann: that sounds good! \o/19:50
gmannwith fix we just make sure plugin is available in tempst venv also so that tempest py3 venv also works fine on py2 jobs19:51
*** slaweq has joined #openstack-qa19:55
elodthanks again gmann, I'm leaving for the day as it's quite late here :) have a nice weekend!19:57
gmannelod: you too and GN.  I will try to catch folks to merge them otherwise on Monday.19:58
*** slaweq has quit IRC19:59
gmannwhoami-rajat: seems like bug in cinder 3.1 microversion, last 2-3 comments - https://review.opendev.org/#/c/616783/20:06
gmann 'visibility' and 'protected' fields are returned in base 3.0 microversion also even they were added in 3.1 and not supposed to be return in <3.120:07
*** rh-jelabarre has quit IRC20:33
*** ccamposr__ has quit IRC20:58
*** raildo has quit IRC21:08
openstackgerritMerged openstack/devstack-plugin-ceph master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72917921:10
*** rfolco|rover has quit IRC21:22
*** paras333 has quit IRC21:46
*** vishalmanchanda has quit IRC21:49
*** paras333 has joined #openstack-qa21:52
*** paras333 has quit IRC21:56
whoami-rajatgmann, those are request params, they're used when sending upload image request to glance22:26
whoami-rajatgmann, they will be included in response before 3.1 as well22:27
whoami-rajati've left a comment on the patch22:27
*** ccamposr has joined #openstack-qa22:56
*** tosky has quit IRC22:59
*** larainema has quit IRC23:00
*** ccamposr__ has joined #openstack-qa23:17
*** ccamposr has quit IRC23:20
*** paras333 has joined #openstack-qa23:37
*** paras333 has quit IRC23:38
*** paras333 has joined #openstack-qa23:38

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