| opendevreview | Ghanshyam proposed openstack/grenade master: WIP: testing https://review.opendev.org/c/openstack/grenade/+/803397 | 00:54 |
|---|---|---|
| opendevreview | Dr. Jens Harbott proposed openstack/grenade master: WIP: testing https://review.opendev.org/c/openstack/grenade/+/803397 | 05:05 |
| *** ykarel|away is now known as ykarel | 05:08 | |
| opendevreview | yatin proposed openstack/grenade stable/wallaby: Unblock Gate: create encryption type with right params https://review.opendev.org/c/openstack/grenade/+/803431 | 05:43 |
| *** jpena|off is now known as jpena | 07:01 | |
| *** rpittau|afk is now known as rpittau | 07:13 | |
| *** ykarel is now known as ykarel|lunch | 08:43 | |
| *** ykarel|lunch is now known as ykarel | 09:23 | |
| opendevreview | Merged openstack/devstack stable/victoria: Fix Usage of rdo-release rpm https://review.opendev.org/c/openstack/devstack/+/803072 | 10:41 |
| *** dviroel|out is now known as dviroel | 11:16 | |
| *** jpena is now known as jpena|lunch | 11:26 | |
| *** jpena|lunch is now known as jpena | 12:32 | |
| opendevreview | Slawek Kaplonski proposed openstack/devstack master: Deploy Neutron with enforced new RBAC rules https://review.opendev.org/c/openstack/devstack/+/797450 | 13:20 |
| opendevreview | Ananya proposed opendev/elastic-recheck master: Run elastic-recheck container https://review.opendev.org/c/opendev/elastic-recheck/+/729623 | 13:22 |
| opendevreview | Ghanshyam proposed openstack/grenade master: Test created encrypted volume type https://review.opendev.org/c/openstack/grenade/+/803397 | 13:54 |
| *** jpena is now known as jpena|off | 15:05 | |
| opendevreview | yatin proposed openstack/devstack stable/ussuri: Fix Usage of rdo-release rpm https://review.opendev.org/c/openstack/devstack/+/803441 | 15:14 |
| ykarel | gmann, reported https://bugs.launchpad.net/devstack/+bug/1938914 for the issue i mentioned yesterday | 15:18 |
| ykarel | whoami-rajat__, recheck on https://review.opendev.org/c/openstack/devstack/+/803039 intentional ? | 15:20 |
| ykarel | shouldn't pass until stable/wallaby grenade merges or depends-on that | 15:21 |
| whoami-rajat__ | ykarel, i think we've a misunderstanding regarding the grenade used, i think grenade master is used there, the grenade patch merge and your patch CI results were at the same time | 15:21 |
| whoami-rajat__ | as other patches pass grenade and also glance gate is unblocked | 15:22 |
| whoami-rajat__ | i think we don't need the wallaby patch | 15:22 |
| whoami-rajat__ | ykarel, to check that observation i rechecked | 15:22 |
| ykarel | whoami-rajat__, as i see grenade patch merged at 12:08 AM , and recheck on devstack patch was at 12:21 AM | 15:24 |
| ykarel | but ok to check if other projects are unblocked, | 15:24 |
| ykarel | seeing lot of green https://zuul.opendev.org/t/openstack/builds?job_name=grenade, so should be correct | 15:26 |
| whoami-rajat__ | ykarel, true but see this patch is passing grenade https://review.opendev.org/c/openstack/devstack/+/797450 | 15:26 |
| ykarel | whoami-rajat__, okk than should be good, need to check how it works | 15:27 |
| *** ykarel is now known as ykarel|away | 15:33 | |
| whoami-rajat__ | yeah let's see | 15:36 |
| *** whoami-rajat__ is now known as whoami-rajat | 15:37 | |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add vgpu resize tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/790304 | 15:43 |
| gmann | whoami-rajat: ykarel|away in wallaby grenade test vicrotia->wallaby, so if cinder s going to backport the change to victoria we can backport grenade fix too | 15:58 |
| gmann | whoami-rajat: ykarel|away also I am adding volume creation with created volume type to test it https://review.opendev.org/c/openstack/grenade/+/803397 | 15:59 |
| gmann | frickler: yoctozepto created volume type usage in grenade is ready https://review.opendev.org/c/openstack/grenade/+/803397 | 15:59 |
| yoctozepto | gmann: another issue? my, these are growing faster than I can wrap my head around them | 16:00 |
| gmann | yoctozepto: not issue :) but extending the test for the issue we faced for cinder change | 16:01 |
| gmann | yoctozepto: using the created voluem type in volume creation so that we know volume type is created successfully and usable | 16:02 |
| yoctozepto | makes sense | 16:03 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Consider tempest constraints set as env variable https://review.opendev.org/c/openstack/tempest/+/802629 | 16:13 |
| *** rpittau is now known as rpittau|afk | 16:14 | |
| bluesonic | gmann: Thanks for the suggestions on the required changes. Testing seems to be going well with the changes. Will update the PS for final review and approval if everything goes well :) | 16:59 |
| abhishekk | gmann, if you have time I need small help | 17:31 |
| abhishekk | when I clone glance-tempest-plugin manually and run tests from it using tempest it does not work, fails with oslo_config.cfg.NoSuchOptError: no such option enforce_scope in group [image-feature-enabled] | 17:32 |
| abhishekk | I do have tempest.conf under glance-tempest-plugin/etc directory | 17:33 |
| gmann | bluesonic: thanks, yeah i did not realize they were cleaned up at test level instead of class level | 17:47 |
| gmann | abhishekk: did you install glance-tempest-plugin ? and run tempest list-plugins it should show glance-tempest-plugin | 17:47 |
| abhishekk | gmann, after cloning when I try to install it, gives error, | 17:48 |
| abhishekk | ERROR: glance-tempest-plugin is not a valid editable requirement. It should either be a path to a local project or a VCS URL (beginning with svn+, git+, hg+, or bzr+). | 17:48 |
| abhishekk | I think i need to use setup.py ? | 17:49 |
| abhishekk | gmann, problem solved, thank you | 17:51 |
| abhishekk | another question is I don't needed to install when I cloned cinder-tempest-plugin and ran | 17:51 |
| gmann | abhishekk: yeah, python setup.py install | 17:56 |
| abhishekk | gmann, ack, thank you | 17:56 |
| gmann | abhishekk: until tempest detect the plugin it would not run test from plugins. tempest list-plugins command can give you list of all detected plugins by tempest | 17:57 |
| abhishekk | gmann, ok, when I run above command it does not shows cinder plugin in the output, but I am able to run cinder plugin tests | 17:58 |
| *** ricolin_ is now known as ricolin | 18:02 | |
| gmann | abhishekk: humm, via tempest run ? | 18:04 |
| abhishekk | gmann, yes | 18:04 |
| gmann | without installation it would be possible. list-plugins should list it if installed | 18:04 |
| gmann | *would not | 18:05 |
| abhishekk | gmann, I think the reason is I have copied tempest.conf in /etc/tempest folder, let me delete it and try again | 18:05 |
| abhishekk | nope, still it is running | 18:06 |
| gmann | tempest.conf have no role in running test | 18:06 |
| gmann | if test runner detect the test then it can run | 18:06 |
| gmann | but once you install plugins you are not able to see it in list-plugins ? | 18:07 |
| abhishekk | gmann, https://paste.opendev.org/show/807893/ | 18:07 |
| abhishekk | gmann, I have installed glance-tempest-plugin and I am able to see it | 18:07 |
| gmann | ok, and run test too right ? | 18:08 |
| abhishekk | I haven't installed cinder-tempest-plugin and it is not listed output but still I am able to run those tests from cinder plugin | 18:08 |
| abhishekk | yes, I am able to run glance-tempest-plugin tests | 18:08 |
| gmann | ko | 18:08 |
| gmann | ok | 18:08 |
| gmann | I think cinder_tempest_plugin is there somewhere in system but list plugins not detecting | 18:09 |
| abhishekk | also I have deleted tempest.conf file from /etc/tempest | 18:10 |
| abhishekk | but still cinder plugin is saying; | 18:10 |
| abhishekk | 2021-08-04 18:04:17.140 2761 INFO tempest [-] Using tempest config file /etc/tempest/tempest.conf | 18:10 |
| opendevreview | Huy Tran proposed openstack/tempest master: Changes to ensure delete_containers is done at class level tear down https://review.opendev.org/c/openstack/tempest/+/802388 | 18:11 |
| gmann | I am sure cinder plugin is running from other place as list-plugins also not detecting it | 18:12 |
| abhishekk | one major issue is | 18:12 |
| abhishekk | 2021-08-04 18:04:17.151 2761 INFO tempest.test_discover.plugins [-] Register additional config options from Tempest plugin: glance_tests | 18:12 |
| abhishekk | it is refering to glance_tests which is plugin name of glance_tests | 18:13 |
| abhishekk | look carefully at that paste I have shared | 18:13 |
| abhishekk | it is running cinder tests, but referring to glance tempest plugin :/ | 18:14 |
| abhishekk | totally confusing | 18:16 |
| gmann | abhishekk: that is fine as tempest load from all plugins. glance plugin is installed to Tempest will register all the config option from plugins that is why you see glance_test loading lg or so | 18:18 |
| gmann | abhishekk: you are running cinder plugin test so it them from somewhere it is installed tempest run --regex cinder_tempest_plugin.rbac.v3 --concurrency=4 | 18:18 |
| gmann | try to run glance tests and see if any issue? | 18:18 |
| abhishekk | no issues with glance tests | 18:19 |
| gmann | ok | 18:19 |
| bluesonic | gmann: five (5) consecutive runs of object_store test suite with 100% passed . https://review.opendev.org/c/openstack/tempest/+/802388 | 18:19 |
| gmann | cinder plugin is installed somewhere in your system somehow and running test | 18:19 |
| gmann | otherwise list-plugins will give you all the plugin list you can run test from | 18:19 |
| gmann | bluesonic: great, and cleanup is in order now? | 18:20 |
| abhishekk | means either issue is tempest list-plugins is not picking it or it is not installed and still I am able to run those tests | 18:20 |
| gmann | abhishekk: you said you have not installed it right? | 18:20 |
| abhishekk | I will try it on new setup | 18:20 |
| gmann | abhishekk: yeah, or try to run some other plugin test you will get error | 18:21 |
| abhishekk | gmann, I just cloned cinder-tempest-plugin and not installed it | 18:21 |
| abhishekk | gmann, ack | 18:21 |
| gmann | for example, watcher-tempest-plugin | 18:22 |
| bluesonic | gmann: yes...so far so good with the on-demand runs... | 18:22 |
| gmann | without clone and with clone but no install | 18:22 |
| gmann | bluesonic: nice, I will review once we have gate result | 18:22 |
| bluesonic | gmann: cool! | 18:22 |
| abhishekk | gmann, I cloned neutron_tempest_plugin and ran using, tempest run --regex neutron_tempest_plugin.api --concurrency=4 | 18:27 |
| abhishekk | no error | 18:27 |
| gmann | so test run successfully or just skip ? | 18:28 |
| gmann | becasue at least they would not run successfully as it is not installed and their config options will not be available | 18:29 |
| abhishekk | gmann, test skipped | 18:29 |
| abhishekk | for watcher I am getting error | 18:29 |
| abhishekk | but not for cinder and neutron | 18:30 |
| gmann | ok, basically they might be installed somewhere earlier | 18:30 |
| gmann | but for running tests from plugins you need to install them and tempest should detect them in list-plugins | 18:31 |
| abhishekk | may be worth trying on new setup | 18:31 |
| abhishekk | AFAIK I never used/cloned/installed neutron and cinder tempest plugins | 18:32 |
| abhishekk | before today | 18:32 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Allow for ethernet type in rx queue test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/803511 | 19:47 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Allow for ethernet type in rx queue test https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/803511 | 19:49 |
| *** dviroel is now known as dviroel|out | 20:38 | |
| bluesonic | gmann: thanks for the +2 and wf :) | 22:00 |
| gmann | bluesonic: np!, thanks for fixing it. | 22:00 |
| opendevreview | Steve Baker proposed openstack/devstack master: Use scram-sha-256 hashing for postgresql auth https://review.opendev.org/c/openstack/devstack/+/803519 | 22:10 |
| opendevreview | Brian Rosmaita proposed openstack/devstack master: Allow cinder default quotas configuration https://review.opendev.org/c/openstack/devstack/+/803521 | 22:29 |
| opendevreview | Steve Baker proposed openstack/devstack master: Ensure 02-proxy-uwsgi.conf exists before writing https://review.opendev.org/c/openstack/devstack/+/803530 | 23:45 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!