*** lifeless has quit IRC | 04:52 | |
*** kopecmartin has joined #refstack | 07:04 | |
openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Updating object-storage.operator_role https://review.openstack.org/575501 | 07:12 |
---|---|---|
*** tosky has joined #refstack | 07:36 | |
kopecmartin | tosky, chandankumar arxcruz any idea how to find out all attributes needed for user creation in older OpenStack releases? | 07:49 |
kopecmartin | https://github.com/openstack/python-tempestconf/blob/master/config_tempest/users.py#L111 | 07:49 |
kopecmartin | because tenantId doesn't seem to be right | 07:49 |
kopecmartin | and upstream developer.openstack documenation is absolutely useless, I don't know, why they don't keep the doc for older releases too | 07:50 |
arxcruz | kopecmartin: https://developer.openstack.org/api-ref/identity/v3/#create-user | 07:51 |
arxcruz | kopecmartin: users_client is v3 | 07:51 |
kopecmartin | arxcruz, yeah, i found that, so we know, the attribute is deprecated for the current release, how do i find out, when it was used last time? | 07:52 |
arxcruz | git history? | 07:53 |
kopecmartin | that could take forever :/ | 07:53 |
tosky | well, if it's just something from API v2, go back to a commit before the removal of API v2, regenerate its documentation locally with tox -e api-ref or so, and read it | 07:55 |
kopecmartin | tosky, ach, wouldn't it be great, if the documentation would be kept on the web and not removed? :D | 07:59 |
kopecmartin | but thanks, I'm gonna try it | 07:59 |
tosky | kopecmartin: previously the documentation for past releases was now kept, but this was changed during Denver | 08:37 |
tosky | that said, this is not the documentation of a past release, but an API reference for a past release | 08:37 |
tosky | and this was not exactly covered | 08:37 |
tosky | better ask on #openstack-doc | 08:37 |
*** lifeless has joined #refstack | 08:55 | |
openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: WIP: Replace tenant by project, part 2 https://review.openstack.org/575233 | 09:11 |
*** rmart04 has joined #refstack | 09:37 | |
arxcruz | shit man | 10:02 |
*** lifeless has quit IRC | 10:03 | |
*** lifeless has joined #refstack | 10:04 | |
openstackgerrit | Merged openstack/python-tempestconf master: Fix boto method https://review.openstack.org/575213 | 10:08 |
*** szaher has joined #refstack | 10:54 | |
openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Updating object-storage.operator_role https://review.openstack.org/575501 | 10:59 |
arxcruz | kopecmartin: http://logs.openstack.org/01/575501/3/check/python-tempestconf-tempest-devstack-demo/29ca66b/job-output.json.gz and http://logs.openstack.org/73/574773/7/check/python-tempestconf-tempest-devstack-demo/b3747cd/job-output.json.gz | 11:00 |
arxcruz | kopecmartin: two diferent patches, one the admin tests run, and fails | 11:00 |
arxcruz | the other is being skipped | 11:00 |
arxcruz | i don't know what's wrong | 11:00 |
arxcruz | on devstack-demo admin tests shouldn't run | 11:00 |
arxcruz | kopecmartin: is there a way to check if the user is admin or not in python-tempestconf ? | 11:00 |
arxcruz | brb, get my daughter at school | 11:00 |
chandankumar | hogepodge: mguiney tosky kopecmartin please have a look at refstack patch https://review.openstack.org/#/c/541273/ | 11:07 |
tosky | after kopecmartin verifies that his issues were addressed | 11:08 |
tosky | enough things to do at this point | 11:08 |
*** lifeless has quit IRC | 11:10 | |
arxcruz | back | 11:37 |
arxcruz | tosky: is there a way to identify if tempestconf is running as admin or demo ? | 12:58 |
* arxcruz don't know too much about identity stuff | 12:58 | |
tosky | *with* admin or with demo? Not sure either | 12:59 |
tosky | the roles are only a collection of capabilities iirc, so you should try to do something to see if you can really do it | 12:59 |
openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 14:04 |
arxcruz | tosky: chandankumar I took liberty to use chandankumar patch to set the operator_role because some logic was required that would be duplicated on chandankumar patch, and his patch was already on the way | 14:05 |
arxcruz | also, i add an except when creating the roles because was throwing exceptions.conflict | 14:05 |
tosky | no problem with fixing it | 14:06 |
arxcruz | i'll abandom my patch | 14:07 |
arxcruz | tosky: my change to ResellerAdmin was to test the demo test | 14:07 |
tosky | I see | 14:07 |
arxcruz | it seems if we set admin you must have admin credentials | 14:08 |
arxcruz | if we don't and try to use admin, it will fail | 14:08 |
arxcruz | so now, chandankumar is priority, because we are blocked in the production chain :) | 14:08 |
arxcruz | tosky: I also notice some exceptions.conflict so I took the liberty of add a try/except | 14:09 |
arxcruz | so if you guys have chance, please review | 14:09 |
tosky | but right now keystone is a bit borked anyway, and that's going to break more | 14:10 |
tosky | I mean, I'm busy with a bigger breakage | 14:10 |
arxcruz | tosky: no problem, let's wait the check jobs, if pass, please review when you have time | 14:11 |
arxcruz | kopecmartin: you too | 14:11 |
tosky | sure, check jobs are good to wait for anyway | 14:12 |
tosky | and also RDO CI jobs | 14:12 |
tosky | so that compatibility is not broken | 14:12 |
* kopecmartin is back, checking the chat | 14:40 | |
*** rmart04 has quit IRC | 15:02 | |
mguiney | o/will take a look | 15:36 |
openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 16:01 |
arxcruz | nice, just realize you can't run swift tests if you're not admin | 16:05 |
kopecmartin | arxcruz, what are those reviews you mentioned? I can review in the evening if gates are still running now | 16:06 |
tosky | arxcruz: about that patch: why set service_available false in the set_default_tempest_option? | 16:07 |
tosky | later, tempestconf will go through all services and set service_available explicitly | 16:07 |
tosky | so if you want to make sure that it's set to False, maybe setting it in set_default_tempest_options is not the best choice | 16:09 |
*** kopecmartin has quit IRC | 16:12 | |
arxcruz | tosky: if you don't have permission to create roles, you can't run swift tests | 16:17 |
arxcruz | it's failing because swift requires admin role | 16:17 |
arxcruz | tosky: and i just realize that now | 16:17 |
arxcruz | tosky: may i ping you later to review this patch? it's blocking promotion... | 16:21 |
tosky | arxcruz: -1 already | 16:21 |
arxcruz | lol | 16:21 |
tosky | arxcruz: you can't run *any* tests without admin permissions? I don't think so | 16:21 |
tosky | you just need the specific swift role, iirc | 16:21 |
arxcruz | tosky: nope, when you enable swift, the tempest.api.admin.object_storage start to run and fails | 16:22 |
tosky | I was able to run swift tests for sahara with a demo user, which could not run swift commands by default, just adding the proper swift role | 16:22 |
tosky | of course tempest.api._admin_ fails | 16:22 |
tosky | that should fail for everything if you are not admin, not just object-storage | 16:22 |
tosky | not unexpected | 16:23 |
tosky | admin tests should be run only by admin | 16:23 |
openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 16:31 |
arxcruz | tosky: i think you're right, update | 16:31 |
tosky | arxcruz: thanks; did you know that a recent change in keystone changed the defaults a bit, and member should be basically the same as Member (i.e. the lowercase version is the primary one)? | 16:49 |
tosky | or something like that | 16:49 |
tosky | it broke few things | 16:51 |
tosky | arxcruz: see comment 2 on https://bugs.launchpad.net/horizon/+bug/1777359 | 16:51 |
openstack | Launchpad bug 1777359 in OpenStack Dashboard (Horizon) "Unable to create a project from horizon on devstack" [Undecided,New] - Assigned to Lance Bragstad (lbragstad) | 16:51 |
tosky | but not sure if it should work with older versions too | 16:51 |
tosky | let's see the gates | 16:51 |
arxcruz | well, at least lance is a very nice guy to fix these problems :) | 17:39 |
arxcruz | tosky: https://review.openstack.org/#/c/572243/ | 17:40 |
tosky | arxcruz: yes, that's the function for which I proposed a revert (see last comment) | 17:40 |
tosky | :D | 17:40 |
tosky | if you want to read all the fun, check the logs of #openstack-keystone starting from this (our) morning | 17:41 |
arxcruz | tosky: no, all i'm thinking about now is my trip to brazil on friday :D | 17:41 |
tosky | oh, good too | 17:41 |
tosky | but you can read the logs on the plane | 17:41 |
tosky | instead of watching bad movies | 17:42 |
tosky | :D | 17:42 |
arxcruz | lol i already have a collection of tv shows to watch | 17:44 |
arxcruz | that i'm missing | 17:44 |
arxcruz | legion, colony, the expanse | 17:44 |
arxcruz | plus, skyrim on nintendo switch | 17:45 |
*** rmart04 has joined #refstack | 19:25 | |
*** lifeless has joined #refstack | 19:26 | |
*** rmart04 has quit IRC | 20:05 | |
*** lifeless has quit IRC | 21:21 | |
*** lifeless has joined #refstack | 21:22 | |
openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Replace tenant by project, part 2 https://review.openstack.org/575233 | 22:13 |
*** lifeless_ has joined #refstack | 22:32 | |
*** lifeless has quit IRC | 22:32 | |
*** ekhugen has quit IRC | 22:55 | |
*** ekhugen has joined #refstack | 22:59 | |
*** lifeless_ has quit IRC | 23:05 | |
*** lifeless has joined #refstack | 23:06 | |
*** tosky has quit IRC | 23:20 | |
*** lifeless has quit IRC | 23:43 | |
*** lifeless has joined #refstack | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!