*** tosky has quit IRC | 00:01 | |
*** brinzhang_ has joined #openstack-qa | 01:07 | |
*** brinzhang has quit IRC | 01:10 | |
*** brinzhang has joined #openstack-qa | 03:03 | |
*** brinzhang_ has quit IRC | 03:06 | |
*** brinzhang_ has joined #openstack-qa | 03:18 | |
*** paras333 has quit IRC | 03:18 | |
*** brinzhang has quit IRC | 03:21 | |
*** elod has quit IRC | 03:23 | |
openstackgerrit | zhangboye proposed openstack/openstack-health master: Fix hacking min version to 3.0.1 https://review.opendev.org/730180 | 03:25 |
---|---|---|
*** elod has joined #openstack-qa | 03:35 | |
*** brinzhang has joined #openstack-qa | 03:36 | |
*** brinzhang_ has quit IRC | 03:40 | |
openstackgerrit | zhangboye proposed openstack/os-testr master: Fix hacking min version to 3.0.1 https://review.opendev.org/730189 | 03:54 |
*** ramishra has quit IRC | 04:01 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-qa | 04:33 | |
*** paras333 has joined #openstack-qa | 04:52 | |
*** paras333 has quit IRC | 04:57 | |
*** ramishra has joined #openstack-qa | 05:08 | |
*** paras333 has joined #openstack-qa | 05:20 | |
*** paras333 has quit IRC | 05:24 | |
*** paras333 has joined #openstack-qa | 05:52 | |
*** paras333 has quit IRC | 05:57 | |
*** nightmare_unreal has joined #openstack-qa | 06:18 | |
openstackgerrit | zhufl proposed openstack/tempest master: Add response schema validation for volumes https://review.opendev.org/616783 | 06:28 |
*** ralonsoh has joined #openstack-qa | 06:33 | |
*** udesale has joined #openstack-qa | 06:39 | |
*** ramishra has quit IRC | 06:48 | |
*** slaweq has joined #openstack-qa | 06:49 | |
*** slaweq has quit IRC | 06:54 | |
openstackgerrit | zhufl proposed openstack/tempest master: Adding description for testcases - network part2 https://review.opendev.org/723131 | 06:56 |
*** slaweq has joined #openstack-qa | 06:56 | |
openstackgerrit | zhufl proposed openstack/tempest master: Adding description for testcases - network part2 https://review.opendev.org/723131 | 07:07 |
*** tosky has joined #openstack-qa | 07:34 | |
*** jpena|off is now known as jpena | 07:38 | |
*** gfidente has joined #openstack-qa | 08:03 | |
*** tkajinam_ has quit IRC | 08:05 | |
*** ramishra has joined #openstack-qa | 08:15 | |
*** larainema has joined #openstack-qa | 08:29 | |
*** paras333 has joined #openstack-qa | 08:36 | |
*** elod has quit IRC | 08:43 | |
*** artom has quit IRC | 08:45 | |
*** artom has joined #openstack-qa | 08:46 | |
*** elod has joined #openstack-qa | 08:50 | |
*** elod has quit IRC | 08:56 | |
*** elod has joined #openstack-qa | 08:58 | |
*** elod has quit IRC | 09:10 | |
*** elod has joined #openstack-qa | 09:10 | |
*** ramishra has quit IRC | 09:37 | |
openstackgerrit | zhufl proposed openstack/tempest master: Add response schema validation for volumes https://review.opendev.org/616783 | 09:44 |
*** ramishra has joined #openstack-qa | 10:01 | |
*** gcheresh has joined #openstack-qa | 10:57 | |
*** brinzhang has quit IRC | 11:09 | |
*** gcheresh has quit IRC | 11:13 | |
*** nweinber has joined #openstack-qa | 11:24 | |
*** jpena is now known as jpena|lunch | 11:30 | |
*** maciejjozefczyk is now known as mjozefcz|lunch | 11:30 | |
openstackgerrit | Merged openstack/tempest master: Change bytes to str in servers_client for python3 https://review.opendev.org/709680 | 11:41 |
*** paras333 has joined #openstack-qa | 11:52 | |
*** paras333 has quit IRC | 11:56 | |
openstackgerrit | Lee Yarwood proposed openstack/devstack master: WIP/DNM tempest: Increase m1.nano RAM to 96MB to avoid tmpfs exhaustion https://review.opendev.org/730307 | 12:05 |
*** lajoskatona has joined #openstack-qa | 12:16 | |
*** mjozefcz|lunch is now known as mjozefcz | 12:18 | |
*** raildo has joined #openstack-qa | 12:21 | |
*** jpena|lunch is now known as jpena | 12:32 | |
*** rh-jelabarre has joined #openstack-qa | 12:39 | |
*** rcernin has quit IRC | 12:39 | |
*** rfolco|rover is now known as rfolco|rover|bbl | 12:52 | |
*** paras333 has joined #openstack-qa | 12:52 | |
lajoskatona | Hi, I have a strange stable/stein issue for networking-bagpipe: https://review.opendev.org/729936 | 12:56 |
lajoskatona | legacy-tempest-dsvm-networking-bgpvpn-bagpipe fails with this: | 12:57 |
lajoskatona | The 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 |
lajoskatona | the 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-plugin | 12:58 |
lajoskatona | One idea is that networking-bgpvpn is not installed and thus tempest can't see the plugin, | 12:59 |
tosky | lajoskatona: 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 |
lajoskatona | tosky: that was another option in my mind, though its like changing the past :-) | 13:02 |
lajoskatona | tosky: in bgpvpn repo it is define here: https://opendev.org/openstack/networking-bgpvpn/src/branch/master/devstack/devstack-gate-bagpipe-rc#L7 | 13:04 |
tosky | lajoskatona: the past would need some changes anyway :) and iirc this was the way followed after moving the tempest plugins had been moved to separate repositories | 13:04 |
lajoskatona | tosky: is it possible to overwrite that from zuul.yaml? or we have to push a change to stein bgpvpn? | 13:04 |
tosky | lajoskatona: that's a legacy job, so you would need to change that variable in the playbook where it's defined | 13:05 |
tosky | maybe it's easier to port that job as native | 13:06 |
lajoskatona | tosky: is that a valid step on stable/stein? | 13:07 |
tosky | uhm, I see it's just for stable/stein, stable/rocky and stable/queens | 13:09 |
tosky | porting it would allow you to remove the legacy job from openstack-zuul-jobs | 13:10 |
tosky | gmann: what do you think? | 13:10 |
gmann | tosky: yeah that is recommended way. | 13:16 |
gmann | lajoskatona: 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 test | 13:17 |
gmann | and i think that job might be using all-plugins tox env also. | 13:17 |
gmann | lajoskatona: 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.html | 13:18 |
*** udesale_ has joined #openstack-qa | 13:24 | |
lajoskatona | gmann: thanks, I check it | 13:25 |
lajoskatona | tosky: thanks for the help | 13:25 |
gmann | lajoskatona: there were few more options also to fix with the tradeoff, let me check | 13:25 |
*** udesale has quit IRC | 13:27 | |
tosky | lajoskatona: talking about zuulv3 porting: in https://review.opendev.org/#/c/725647/, have seen the error in other network-odl multinode jobs? | 13:30 |
gmann | lajoskatona: i think vpnaas issues were solved by these -https://review.opendev.org/#/q/topic:fix-stable-gate+(status:open+OR+status:merged | 13:30 |
gmann | lajoskatona: can you point me the job link, i can debug after my coffee ? | 13:31 |
lajoskatona | gmann: this one: https://review.opendev.org/729936 | 13:38 |
lajoskatona | gmann: networking-bagpipe pulls networking-bgpvpn and that has in-tree tempest-plugin | 13:39 |
gmann | ok | 13:51 |
lajoskatona | gmann: thanks for checking it | 13:54 |
*** mailingsam has joined #openstack-qa | 14:21 | |
mailingsam | Hello, could you review https://review.opendev.org/#/c/729803 | 14:22 |
mordred | gmann: 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 |
mordred | gmann: 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 |
openstackgerrit | Andreas Jaeger proposed openstack/grenade master: Cleanup py27 support https://review.opendev.org/726577 | 14:37 |
mordred | gmann: I think your way is better | 14:38 |
openstackgerrit | Lee Yarwood proposed openstack/devstack master: tempest: Increase m1.nano and m1.micro RAM by 64MB to avoid tmpfs exhaustion https://review.opendev.org/730307 | 14:49 |
*** jcmdln has quit IRC | 15:01 | |
*** gcheresh has joined #openstack-qa | 15:11 | |
amodi | gmann: lyarwood can i get some input on https://review.opendev.org/#/c/730014/ this is trivial | 15:14 |
*** vishalmanchanda has joined #openstack-qa | 15:28 | |
lyarwood | amodi: looking | 15:29 |
*** rfolco|rover|bbl is now known as rfolco|rover | 15:31 | |
openstackgerrit | Archit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern https://review.opendev.org/730014 | 15:40 |
amodi | lyarwood: thanks! | 15:40 |
openstackgerrit | Archit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern https://review.opendev.org/730014 | 15:41 |
*** gcheresh has quit IRC | 15:45 | |
*** lajoskatona has left #openstack-qa | 15:48 | |
openstackgerrit | Archit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern https://review.opendev.org/730014 | 16:04 |
*** gcheresh has joined #openstack-qa | 16:09 | |
lyarwood | amodi: few more comments, make sure you run $ tox -e pep8 -- -HEAD before you git review | 16:13 |
amodi | lyarwood: yeah my bad, im running this from a remote machine, should install relevant packages thanks again! | 16:14 |
lyarwood | amodi: tox should take care of all of that for you | 16:15 |
lyarwood | assuming that's installed ;) | 16:15 |
amodi | yeah it wasn't | 16:15 |
*** jcmdln has joined #openstack-qa | 16:17 | |
*** gcheresh has quit IRC | 16:20 | |
amodi | lyarwood: we don't need any "version" assertions, do we? so just snapshotting it at the end is enough | 16:21 |
gmann | mordred: 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.html | 16:21 |
gmann | that passed in my DNM patch, did recheck | 16:26 |
lyarwood | amodi: 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 |
openstack | Launchpad bug 1717547 in OpenStack Compute (nova) pike "Creating snapshot fails when image metadata has version field" [Low,Confirmed] | 16:27 |
amodi | lyarwood: yes, that's why i had this set before creating the server | 16:28 |
amodi | lyarwood: so revert to previous patch? | 16:29 |
lyarwood | amodi: yeah but my point was that this isn't realistic IMHO | 16:29 |
lyarwood | amodi: users would want to version their snapshots this way | 16:30 |
lyarwood | amodi: 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 |
gmann | lajoskatona i cannot find networking-bgpvpn tempest plugins installed on system side otherwise all-plugins env could have detect that in tempest venv. | 16:31 |
gmann | i need more time on that to check why that is not installed or detected, might need some new logging. | 16:32 |
amodi | lyarwood: anything works for me, i guess either way we can trigger the bug, so why not use your suggestion | 16:34 |
elod | gmann: 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 |
gmann | elod: 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 it | 16:41 |
gmann | one 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 case | 16:42 |
gmann | elod: yeah that is the case. | 16:43 |
*** jpena is now known as jpena|off | 16:43 | |
elod | gmann: oh, nice catch | 16:43 |
gmann | elod: 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 plugin | 16:44 |
elod | gmann: 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 |
gmann | one 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 also | 16:46 |
gmann | elod: yeah ^^ that way. let me try. | 16:46 |
elod | gmann: sounds great, thanks! | 16:46 |
openstackgerrit | Archit Modi proposed openstack/tempest master: Add image "version" metadata in test_snapshot_pattern https://review.opendev.org/730014 | 16:55 |
*** cmurphy is now known as cmorpheus | 17:03 | |
gmann | elod: - https://review.opendev.org/#/c/730378/1 | 17:05 |
gmann | elod: can you make depends-on on your fix to tests | 17:05 |
gmann | we might need devstack-gate-bagpipe-rc to fix not to DEVSTACK_GATE_TEMPEST_ALL_PLUGINS=1 but let's see | 17:07 |
gmann | elod: 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 IRC | 17:20 | |
openstackgerrit | Andreas Jaeger proposed openstack/hacking master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729107 | 17:27 |
openstackgerrit | Andreas Jaeger proposed openstack/devstack-plugin-ceph master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729179 | 17:30 |
openstackgerrit | Andreas Jaeger proposed openstack/bashate master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729134 | 17:31 |
elod | gmann: thanks! I will add the depends-on to patch 730378 and we will see | 17:32 |
*** nightmare_unreal has quit IRC | 17:36 | |
gmann | elod: need bgpvon fix also, please use this for depends-on https://review.opendev.org/#/c/730379/1 | 17:41 |
elod | gmann: ok, nested depends-on, looks fancy :] | 17:42 |
gmann | elod: yeah and I am not sure if zuul job one will work or not. let's see | 17:42 |
elod | gmann: depends-on updated, we will see! | 17:44 |
*** tosky has quit IRC | 17:50 | |
*** tosky has joined #openstack-qa | 17:51 | |
openstackgerrit | Ghanshyam Mann proposed openstack/devstack stable/train: Use stable constraint for older Tempest https://review.opendev.org/729722 | 17:57 |
*** gcheresh has joined #openstack-qa | 18:09 | |
*** gfidente is now known as gfidente|afk | 18:11 | |
*** ralonsoh has quit IRC | 18:14 | |
*** gfidente|afk has quit IRC | 18:21 | |
openstackgerrit | Sorin Sbarnea (zbr) proposed opendev/elastic-recheck master: Resolve unsafe yaml.load use https://review.opendev.org/730389 | 18:22 |
*** artom has quit IRC | 18:30 | |
*** gcheresh has quit IRC | 19:05 | |
*** nweinber has quit IRC | 19:07 | |
*** mailingsam has quit IRC | 19:16 | |
*** elod has quit IRC | 19:26 | |
*** elod has joined #openstack-qa | 19:27 | |
elod | gmann: it works! https://zuul.opendev.org/t/openstack/build/9db603a68ac4478793f4f9e1de441aa6/log/job-output.txt#2411 | 19:35 |
gmann | elod: yeah, we need to get depends-on merge now. | 19:36 |
elod | gmann: 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 IRC | 19:49 | |
gmann | elod: 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 only | 19:49 |
elod | gmann: that sounds good! \o/ | 19:50 |
gmann | with fix we just make sure plugin is available in tempst venv also so that tempest py3 venv also works fine on py2 jobs | 19:51 |
*** slaweq has joined #openstack-qa | 19:55 | |
elod | thanks again gmann, I'm leaving for the day as it's quite late here :) have a nice weekend! | 19:57 |
gmann | elod: you too and GN. I will try to catch folks to merge them otherwise on Monday. | 19:58 |
*** slaweq has quit IRC | 19:59 | |
gmann | whoami-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.1 | 20:07 |
*** rh-jelabarre has quit IRC | 20:33 | |
*** ccamposr__ has quit IRC | 20:58 | |
*** raildo has quit IRC | 21:08 | |
openstackgerrit | Merged openstack/devstack-plugin-ceph master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729179 | 21:10 |
*** rfolco|rover has quit IRC | 21:22 | |
*** paras333 has quit IRC | 21:46 | |
*** vishalmanchanda has quit IRC | 21:49 | |
*** paras333 has joined #openstack-qa | 21:52 | |
*** paras333 has quit IRC | 21:56 | |
whoami-rajat | gmann, those are request params, they're used when sending upload image request to glance | 22:26 |
whoami-rajat | gmann, they will be included in response before 3.1 as well | 22:27 |
whoami-rajat | i've left a comment on the patch | 22:27 |
*** ccamposr has joined #openstack-qa | 22:56 | |
*** tosky has quit IRC | 22:59 | |
*** larainema has quit IRC | 23:00 | |
*** ccamposr__ has joined #openstack-qa | 23:17 | |
*** ccamposr has quit IRC | 23:20 | |
*** paras333 has joined #openstack-qa | 23:37 | |
*** paras333 has quit IRC | 23:38 | |
*** paras333 has joined #openstack-qa | 23:38 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!