Wednesday, 2018-06-06

*** xinliang has quit IRC01:12
*** lifeless has quit IRC07:02
*** lifeless has joined #refstack07:02
*** kopecmartin has joined #refstack07:13
*** openstackgerrit has quit IRC07:19
*** lifeless has quit IRC07:22
*** lifeless has joined #refstack07:28
*** tosky has joined #refstack07:28
*** openstackgerrit has joined #refstack07:49
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Allow non admin users to upload image to glance  https://review.openstack.org/56825207:49
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Allow non admin users to upload image to glance  https://review.openstack.org/56825207:53
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-test-accounts-path argument  https://review.openstack.org/57269508:25
* tosky wonders if --create-test-accounts or just --create-accounts-file or08:28
kopecmartintosky, i don't know, whatever you think is better :)08:32
toskyI know it sounds weird, but the more I think about create-test-accounts-path, the more it sounds about creating a path08:34
tosky:D08:35
kopecmartinright, --create-accounts-file sounds better, I'll update it08:39
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Generate accounts.yaml  https://review.openstack.org/57078108:44
toskythanks08:47
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument  https://review.openstack.org/57269508:47
toskykopecmartin: about the duplicate task: I think that there should be just one08:49
toskyif they express the exact same action item, no need for a duplicate08:49
kopecmartintosky, they express the same .. so we need to delete one of them08:50
arxcruz|ruckkopecmartin: https://review.openstack.org/#/c/570781/ i'll wait the dnm patch and check results before +1 ;)08:51
kopecmartinarxcruz|ruck, ok, thanks08:51
*** lifeless has quit IRC09:05
*** lifeless has joined #refstack09:42
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service  https://review.openstack.org/56886910:23
chandankumartosky: kopecmartin ^^ need help on one of the unit tests, I am not sure how to mock client.skip_path() method10:23
*** lifeless has quit IRC10:26
*** lifeless has joined #refstack10:27
*** lifeless_ has joined #refstack10:57
*** lifeless has quit IRC10:57
kopecmartinchandankumar, tosky how is it with creation of roles and their assignments?11:34
kopecmartinhttps://review.openstack.org/#/c/571981/11:34
kopecmartinthe tests are passing, but no swift tests are there :(11:34
kopecmartinit's strange there is no output of tempest tests, but in the DNM path is the output as expected, a Member role was created https://review.openstack.org/#/c/571985/11:38
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument  https://review.openstack.org/57269511:51
toskyehm, not sure what is the question11:59
kopecmartinthe question is, if the role needs to be assigned to a project/user whatever after it's created or not12:00
kopecmartini have no idea how these things work :/12:00
kopecmartinbrb12:00
toskyit's not magically assigned12:02
toskyin fact, how would you assign a role to a user automatically?12:02
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service  https://review.openstack.org/56886912:41
chandankumarkopecmartin: arxcruz|ruck tosky ^^ please have a look at this review12:41
arxcruz|ruckchandankumar: -112:42
arxcruz|rucklol, just kidding, i'll review once we get +1 from zuul12:42
chandankumarkopecmartin: please have a look at the test12:42
arxcruz|ruckchandankumar: https://review.openstack.org/#/c/572697/3 i'll not -1, but still have unecessary check12:46
chandankumararxcruz|ruck: so you mean to remove conditional of v2 and v3 and reverse the condition for v1, sinc tempest plugin is branchless, we donot know who is using v112:48
toskydoes tempest support v1?12:51
arxcruz|ruckthere's v1 code there, but don't know if it still supported12:59
*** lifeless_ has quit IRC13:16
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument  https://review.openstack.org/57269513:20
chandankumartosky: arxcruz|ruck v1 is deprecated long time ago https://specs.openstack.org/openstack/cinder-specs/specs/juno/deprecate_v1_api.html13:21
arxcruz|ruckchandankumar: so, no need the if13:21
arxcruz|ruckimho13:21
arxcruz|ruck*_latest will be v3 if v3 is enabled, or v213:21
arxcruz|ruckand it's already bein checked on tempest side13:22
arxcruz|ruckbut...13:22
arxcruz|ruck¯\_(ツ)_/¯13:22
arxcruz|ruckmy job is red, need this patch, so i'll not -113:22
chandankumartosky: arxcruz|ruck since barbican tempest plugin is branchless, we donot know where it is used, I am keeping v1 just reversing the condition13:22
arxcruz|ruckwell, you just told v1 is deprecated...13:23
arxcruz|ruckagain... ¯\_(ツ)_/¯13:23
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service  https://review.openstack.org/56886913:25
chandankumararxcruz|ruck: tosky http://codesearch.openstack.org/?q=CONF.volume_feature_enabled.api_v2&i=nope&files=&repos=13:26
chandankumarmost of them have condition for v113:27
arxcruz|ruckwell, if all of them throw themselves from the bridge will you throw yourself too? :P13:29
chandankumararxcruz|ruck: I have updated the patch13:38
arxcruz|ruckchandankumar: :*13:38
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: [DNM] remove image.http_image from playbook  https://review.openstack.org/57277513:48
kopecmartinopenstack, tosky do you know if it's possible to see all available options in tempest.conf for at least several tempest releases?14:59
kopecmartinI can see an example of tempest.conf for current release14:59
kopecmartinhttps://docs.openstack.org/tempest/latest/sampleconf.html14:59
kopecmartinbut there is no note about if there are all options and i don't see any trigger on the site how i could see the example in previous versions15:00
kopecmartinchandankumar ^^15:01
toskykopecmartin: probably the only way is regenerating the list; switch to the tag that you want to check and run tox -e genconfig15:01
kopecmartintosky, nice, it could work15:02
chandankumartosky: kopecmartin I think we can generate it using oslo config15:39
toskychandankumar: yes, and that's what tox -e genconfig does15:39
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: Remove image.http_image from playbook  https://review.openstack.org/57277516:09
chandankumarkopecmartin: tosky ^^16:09
*** kopecmartin has quit IRC17:02
openstackgerritChandan Kumar proposed openstack/python-tempestconf master: [DNM] patch to test backward compatibilty/ container check  https://review.openstack.org/57052418:23
*** lifeless has joined #refstack19:35
*** lifeless_ has joined #refstack21:22
*** lifeless has quit IRC21:23
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Replace tenant by project, part 1  https://review.openstack.org/57301722:00
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Remove admin_*  from identity section  https://review.openstack.org/57302222:24
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: Generate accounts.yaml  https://review.openstack.org/57078122:32
openstackgerritMartin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument  https://review.openstack.org/57269522:33
*** lifeless_ has quit IRC22:34
*** lifeless has joined #refstack22:34
*** tosky has quit IRC23:31
*** lifeless has quit IRC23:46
*** lifeless has joined #refstack23:46

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