*** tosky has quit IRC | 00:04 | |
*** lifeless_ has joined #refstack | 00:46 | |
*** lifeless has quit IRC | 00:46 | |
*** rmart04 has joined #refstack | 08:07 | |
*** kopecmartin has joined #refstack | 08:30 | |
*** openstackgerrit has joined #refstack | 08:42 | |
openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Decouple TempestConf class https://review.openstack.org/537355 | 08:42 |
---|---|---|
openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Split ClientManager https://review.openstack.org/533434 | 08:54 |
*** tosky has joined #refstack | 09:21 | |
openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use --non-admin for non admin users in zuul jobs https://review.openstack.org/554480 | 09:34 |
chandankumar | tosky: kopecmartin below is the stories i have created from ptg feedback for tempestconf | 10:42 |
chandankumar | https://storyboard.openstack.org/#!/story/2001693 | 10:42 |
chandankumar | https://storyboard.openstack.org/#!/story/2001695 | 10:42 |
chandankumar | https://storyboard.openstack.org/#!/story/2001696 | 10:42 |
chandankumar | Feel free to let me know if I missed anything | 10:42 |
chandankumar | arxcruz: ^^ | 10:42 |
arxcruz | chandankumar: ack | 10:43 |
tosky | noticed them, yes | 10:43 |
arxcruz | i'll take a look later | 10:43 |
tosky | I subscribed to all changes that happens to python-tempestconf in storyboard | 10:44 |
tosky | there is one for the refactoring, we may want to add more tasks and refer to each of them for each of the refactoring reviews | 10:44 |
chandankumar | tosky: yup, kopecmartin ^^ | 10:45 |
kopecmartin | tosky, chandankumar you want to create stories for each refactoring change? | 10:47 |
chandankumar | tosky: I need to add a task in tempestconf role to download all the tempest tests given in refstack https://refstack.openstack.org/api/v1/guidelines/2017.09/ in a file which will be used while running tempest tests, does it go in tempest or tempestconf role? | 10:48 |
chandankumar | kopecmartin: just include story: #<storyid> in the commit msg | 10:49 |
tosky | kopecmartin: up to you, but if the refactoring is connected, it could be a unique story with many tasks | 10:49 |
tosky | kopecmartin: then you can add Story: nnnnn and Task: zzzzz to your commit | 10:49 |
tosky | chandankumar: if it's about tempestconf *role*, I guess in tripleo-quickstart, how is it connected with tempestconf? | 10:50 |
chandankumar | tosky: https://review.openstack.org/#/c/541273/ -> about running refstack tests here, for that I need to add task to download test data somewhere where it will go i am not sure | 10:52 |
tosky | chandankumar: just to be sure: when you talk about tempestconf role, what do you mean exactly? | 10:56 |
chandankumar | tosky: i mean here https://github.com/openstack/python-tempestconf/tree/master/roles | 10:56 |
tosky | ok | 10:58 |
tosky | and you want to create a tempest job which will run all those tests, do I get it correctly? | 10:59 |
tosky | which project should this tempest test gate? | 10:59 |
tosky | chandankumar: because I think that the answer to your question depends on that | 11:01 |
chandankumar | tosky: refstack_client | 11:01 |
chandankumar | tosky: yes | 11:01 |
tosky | chandankumar: so a role that downloads the list of refstack tests maybe could go to refstack-client directly | 11:02 |
tosky | unless I'm missing something, that role should not need tempestconf | 11:02 |
chandankumar | tosky: kopecmartin How i can modify this one https://review.openstack.org/#/c/554480/ jinja templates will get picked up | 12:22 |
kopecmartin | chandankumar, I'm not sure if you can use jinja in a .yaml file .. let me check | 12:24 |
kopecmartin | chandankumar, you don't need braces to refer variables inside expression | 12:32 |
kopecmartin | chandankumar, https://stackoverflow.com/questions/40086613/ansible-jinja2-string-comparison | 12:32 |
kopecmartin | try that | 12:32 |
*** openstackgerrit has quit IRC | 12:33 | |
tosky | chandankumar, kopecmartin: or why not set a variable with a when in a separate ansible task, and use that variable? | 12:36 |
tosky | move the logic away from the shell | 12:36 |
tosky | technically the shell part should be minimal in ansible | 12:37 |
kopecmartin | tosky, yes, that's also an option | 12:38 |
*** openstackgerrit has joined #refstack | 12:53 | |
openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use --non-admin for non admin users in zuul jobs https://review.openstack.org/554480 | 12:53 |
chandankumar | tosky: kopecmartin will do | 12:53 |
openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use --non-admin for non admin users in zuul jobs https://review.openstack.org/554480 | 15:36 |
*** kopecmartin has quit IRC | 16:16 | |
openstackgerrit | Merged openstack/refstack-client master: Replace U+2013 EN DASH with a standard hypen for terminal https://review.openstack.org/554051 | 16:29 |
hogepodge | chandankumar: tosky: mguiney: Notes for upcoming meeting. Please add additional agenda items. https://etherpad.openstack.org/p/refstack-meeting-18-03-20 | 16:32 |
mguiney | can do! | 16:33 |
openstackgerrit | Merged openstack/refstack-client master: Ignore .eggs/ to be generated by ./setup_env https://review.openstack.org/554049 | 16:36 |
openstackgerrit | Merged openstack/refstack-client master: Update links in README https://review.openstack.org/550780 | 16:36 |
tosky | uhm, nothing to add on my side | 16:41 |
*** rmart04 has quit IRC | 16:42 | |
mguiney | looks like the general list of topics matches what i had to show | 16:42 |
*** arxcruz is now known as arxcruz|off | 18:53 | |
*** rmart04 has joined #refstack | 19:15 | |
*** rmart04 has quit IRC | 19:20 | |
*** openstackgerrit has quit IRC | 20:33 | |
*** tosky has quit IRC | 23:34 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!