| *** lifeless_ has quit IRC | 01:13 | |
| *** lifeless has joined #refstack | 01:13 | |
| *** lifeless_ has joined #refstack | 01:24 | |
| *** lifeless has quit IRC | 01:25 | |
| *** lifeless has joined #refstack | 01:40 | |
| *** lifeless_ has quit IRC | 01:41 | |
| *** markvoelker has joined #refstack | 01:47 | |
| openstackgerrit | Merged openstack/python-tempestconf master: Fix username is not defined warning https://review.openstack.org/575204 | 01:48 |
|---|---|---|
| *** lifeless has quit IRC | 01:54 | |
| *** lifeless has joined #refstack | 02:00 | |
| *** markvoelker has quit IRC | 02:21 | |
| *** markvoelker has joined #refstack | 03:18 | |
| *** markvoelker has quit IRC | 03:51 | |
| *** markvoelker has joined #refstack | 04:48 | |
| *** markvoelker has quit IRC | 05:21 | |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 05:37 |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 05:43 |
| *** kopecmartin has joined #refstack | 06:42 | |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 06:47 |
| *** markvoelker has joined #refstack | 07:18 | |
| *** tosky has joined #refstack | 07:28 | |
| *** markvoelker has quit IRC | 07:50 | |
| *** rmart04 has joined #refstack | 07:59 | |
| kopecmartin | tosky, chandankumar so what about this? :) https://review.openstack.org/#/c/570781/ | 08:08 |
| arxcruz | tosky: heh, regarding the patch, i'll add a DNM on tqe to test with a depends on, sounds good for you? | 08:12 |
| tosky | arxcruz: the question is: why didn't we enable swift so far? | 08:13 |
| tosky | otherwise this will regress | 08:13 |
| tosky | the reason for this not being catched in fact it's that swift is not enabled | 08:14 |
| tosky | maybe there was a reason but I forgot | 08:14 |
| arxcruz | tosky: we will add a scenario002 on p-tc\ | 08:15 |
| arxcruz | from now i'll say p-tc when i refer to python-tempestconf | 08:16 |
| arxcruz | cuz it's too long to type | 08:16 |
| tosky | or just tempestconf | 08:17 |
| tosky | but that's for the tripleo jobs | 08:17 |
| tosky | and on packstack and devstack? | 08:17 |
| tosky | even in the current review, the swift tests are skipped | 08:17 |
| kopecmartin | we disable swift because of saving resources, or was it something else? I don't remember either | 08:24 |
| arxcruz | tosky: i don't know how to enable swift | 08:30 |
| arxcruz | on packstack job | 08:30 |
| tosky | arxcruz: I think kopecmartin knows - changing the profile 0 | 08:31 |
| kopecmartin | do I? maybe that was another reason, why we disabled swift :D | 08:32 |
| tosky | chandankumar: do you remember? I can ping dmellado too | 08:34 |
| tosky | or maybe we just used the default devstack settings, where swift is not enabled | 08:34 |
| tosky | and that was it | 08:34 |
| tosky | I'm surprised by packstack, though; isn't swift deployed by default? | 08:35 |
| kopecmartin | tosky, oh, yes it is, i have it enabled in my pacstack machine and i didn't do any extra settings | 08:38 |
| kopecmartin | we just disabled it in tempest.conf because of the number of tests i guess | 08:38 |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 08:39 |
| tosky | we can enable just one or two tests, to see if we can talk to it | 08:39 |
| tosky | I think it's important; as arxcruz wrote, this issue went unnoticed also because we didn't have swift on the gates | 08:40 |
| arxcruz | indeed | 08:41 |
| chandankumar | tosky: in devstack-minimal swift is not enabled https://github.com/openstack-dev/devstack/blob/master/.zuul.yaml#L208 | 08:41 |
| chandankumar | tosky: https://review.openstack.org/#/c/568869/ swift is enabled in this one | 08:42 |
| tosky | can we move the removal of service_available.swift False \ to arxcruz's patch? | 08:43 |
| tosky | it seems that the direction is to enable it anyway | 08:43 |
| *** markvoelker has joined #refstack | 08:48 | |
| chandankumar | tosky: yup | 08:54 |
| chandankumar | arxcruz: can you flip the switch in swift patch? | 08:54 |
| *** lifeless has quit IRC | 08:57 | |
| arxcruz | yes i can | 09:07 |
| kopecmartin | tosky, is this only about adding the new job, or also about fixing the code so that it passes the unit tests? https://review.openstack.org/#/c/574773/ | 09:07 |
| tosky | kopecmartin: for now it is about fixing the basic porting issues so that python3 environment can at least start | 09:09 |
| tosky | if you try to run tox -e cover with python 3, or simply tox -e py35 locally, running fails before starting | 09:09 |
| tosky | I will follow-up with the fix for the remaining issues | 09:09 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Updating object-storage.operator_role https://review.openstack.org/575501 | 09:09 |
| arxcruz | done | 09:09 |
| kopecmartin | tosky, can we create a story with tasks for that? | 09:10 |
| tosky | kopecmartin: a python 3 porting story? | 09:10 |
| *** lifeless has joined #refstack | 09:10 | |
| kopecmartin | tosky, yes | 09:10 |
| tosky | oki | 09:11 |
| kopecmartin | to track it, so that we know, what's done and what's needed to be done :) | 09:11 |
| tosky | sure | 09:11 |
| tosky | it totally makes sense | 09:11 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More compatibility with Python 3 https://review.openstack.org/574773 | 09:20 |
| tosky | here you are, https://storyboard.openstack.org/#!/story/2002574 | 09:20 |
| *** markvoelker has quit IRC | 09:22 | |
| kopecmartin | tosky, great, thanks | 09:24 |
| kopecmartin | tosky, chandankumar arxcruz last question , what boolean value should decide, what credentials (admin/demo) will be printed to the accounts.yaml file? https://review.openstack.org/#/c/570781/14/config_tempest/main.py@407 | 09:35 |
| kopecmartin | now there's create boolean value, what does make more sense, non-admin or create? | 09:36 |
| *** lifeless has quit IRC | 09:38 | |
| tosky | uhm, let's start from the behavior | 09:40 |
| tosky | when you said "demo credentials", do you mean the user credentials when the user is non-admin (which may not be "demo", but some custom username), or... | 09:41 |
| tosky | do you mean specifically the credentials of the demo user created when admin credentials are specified? | 09:41 |
| *** lifeless has joined #refstack | 09:45 | |
| *** markvoelker has joined #refstack | 10:18 | |
| *** lifeless_ has joined #refstack | 10:22 | |
| *** lifeless has quit IRC | 10:23 | |
| *** lifeless_ has quit IRC | 10:39 | |
| *** lifeless has joined #refstack | 10:41 | |
| *** markvoelker has quit IRC | 10:51 | |
| *** lifeless_ has joined #refstack | 10:55 | |
| *** lifeless has quit IRC | 10:56 | |
| *** lifeless has joined #refstack | 11:00 | |
| *** lifeless_ has quit IRC | 11:01 | |
| chandankumar | arxcruz: please rebase this one https://review.openstack.org/#/c/570158/ | 11:03 |
| *** lifeless has quit IRC | 11:05 | |
| *** lifeless has joined #refstack | 11:10 | |
| kopecmartin | tosky, sorry, i went for a lunch .. yes, demo credentials = credentials without privileges to create resources | 11:15 |
| *** lifeless_ has joined #refstack | 11:15 | |
| *** lifeless has quit IRC | 11:15 | |
| *** markvoelker has joined #refstack | 11:48 | |
| *** markvoelker has quit IRC | 12:03 | |
| *** markvoelker has joined #refstack | 12:03 | |
| tosky | uhm | 12:17 |
| tosky | what else do non-admin and create control? | 12:17 |
| kopecmartin | tosky, create controls the fact if python-tempestconf is allowed to create resources or not, non-admin controls the credentials | 12:18 |
| kopecmartin | tosky, i discussed this with chandankumar , and it looks fine as it is | 12:18 |
| kopecmartin | because if want to use the accounts.yaml you need to have the resources created | 12:18 |
| tosky | okidoki | 12:19 |
| kopecmartin | i was just a little bit confused, because those two variables are for different things, so wasn't sure, how is the best to build the condition there | 12:20 |
| kopecmartin | great, than the change is ready for review | 12:20 |
| tosky | so the duck methodology is the best one in such cases | 12:20 |
| kopecmartin | true | 12:20 |
| chandankumar | tosky: kopecmartin except image upload non-admin cannot do anything on resource creation side | 12:35 |
| kopecmartin | yes, sure | 12:36 |
| *** lifeless_ has quit IRC | 12:46 | |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 12:48 |
| chandankumar | arxcruz: tosky kopecmartin http://logs.openstack.org/01/575501/2/check/python-tempestconf-tempest-packstack-demo/c4502fd/job-output.txt.gz#_2018-06-15_11_10_17_371053 | 12:54 |
| chandankumar | the same error is coming in my patch also | 12:54 |
| arxcruz | chandankumar: would be nice to have in the collect logs the etc/tempest.conf and the tempest.html | 12:55 |
| arxcruz | freaking hard to roll down all the output to see the error | 12:55 |
| arxcruz | yes, i am lazy | 12:55 |
| kopecmartin | arxcruz, if knew how, i would make the jobs to push the files to logs somewhere, so that we don't have to scroll | 12:57 |
| arxcruz | kopecmartin: i can take a look, but not today | 12:58 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Remove default-overrides.conf https://review.openstack.org/572418 | 13:13 |
| chandankumar | arxcruz: 2018-06-15 11:10:17.371053 | allinone | Details: Insufficient number of users provided. AccountTest,AccountTest have allocated all the credentials for this allocation request | 13:53 |
| arxcruz | yeah, but why is this started to happen ? | 13:54 |
| tosky | some breakage in keystone due to the changes for flask? | 13:58 |
| tosky | I would track the history of changes in keystone | 13:58 |
| kopecmartin | tosky, arxcruz chandankumar please, if you all agree, let's merge this: https://review.openstack.org/#/c/570781 | 14:17 |
| arxcruz | kopecmartin: no because i'll need to rebase my patch :P | 14:18 |
| arxcruz | just kidding, +2 now | 14:18 |
| kopecmartin | :D | 14:18 |
| kopecmartin | thanks | 14:18 |
| arxcruz | kopecmartin: where's the dnm ? | 14:18 |
| kopecmartin | arxcruz, https://review.openstack.org/#/c/572695 | 14:19 |
| kopecmartin | demo jobs failed, but for other reason, accounts.yamls were generated with the right credentials | 14:19 |
| arxcruz | kopecmartin: ok | 14:19 |
| tosky | kopecmartin: which other reasons? Because the demo job working is a good showcase of accounts.yaml | 14:20 |
| tosky | and a slightly different codepath than admin | 14:20 |
| kopecmartin | tosky, because accounts.yaml generated by demo user is useless as he doesn't have right to create resources | 14:21 |
| tosky | kopecmartin: ok, but wasn't one of the original point the need for a fully demo-based job? | 14:22 |
| tosky | or are we going to use the future refstack-client job as cross-gating? | 14:23 |
| tosky | I think I may have already asked this | 14:23 |
| tosky | the point is that this use case (accounts.yaml for non-admin user) is important enough that we should not risk regression, which means an automated job | 14:23 |
| kopecmartin | no, we don't need that for demo job .. demo jobs work because we are generating accounts.yaml files there with admin credentials | 14:24 |
| kopecmartin | I don't know what regression we could have here, it's just printing the credentials to a file | 14:25 |
| tosky | are we printing the correct credentials? :) | 14:26 |
| kopecmartin | yes, you can check the log, check what option we passed (--non-admin or not) and after the generation part, accounts.yaml is printed there :) | 14:28 |
| tosky | uhm, I'm not totally sure, but let's see | 14:41 |
| tosky | in the worst case we can add a cross-gating job from refstack-client, after that the integration is merged | 14:42 |
| tosky | anyway: we need more documentation before 2.0.0 | 14:42 |
| *** rmart04 has quit IRC | 15:10 | |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 17:02 |
| *** kopecmartin has quit IRC | 17:14 | |
| openstackgerrit | Merged openstack/python-tempestconf master: Generate accounts.yaml https://review.openstack.org/570781 | 17:39 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 17:56 |
| *** chandankumar is now known as chkumar|pto | 18:01 | |
| *** lifeless has joined #refstack | 19:34 | |
| *** lifeless has quit IRC | 20:02 | |
| *** lifeless has joined #refstack | 20:03 | |
| *** lifeless has quit IRC | 20:49 | |
| *** lifeless has joined #refstack | 20:50 | |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 21:35 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 21:35 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Remove default-overrides.conf https://review.openstack.org/572418 | 21:46 |
| *** lifeless has quit IRC | 22:01 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Fix boto method https://review.openstack.org/575213 | 22:04 |
| *** lifeless has joined #refstack | 22:49 | |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 22:50 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More compatibility with Python 3 https://review.openstack.org/574773 | 23:25 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More python 3 porting: filter and map https://review.openstack.org/575874 | 23:25 |
| openstackgerrit | Luigi Toscano proposed openstack/python-tempestconf master: More Python 3 porting: filter and map https://review.openstack.org/575874 | 23:26 |
| *** tosky has quit IRC | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!