| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-09-17 | |||
| 09:25:03 | tobiash | mordred: it looks like all recent (devstack related?) jobs of openstacksdk are broken | |
| 09:42:18 | tosky | tobiash: something related with a BadZipfile exception during pip installation? | |
| 10:28:36 | tobiash | tosky: looks like something with keystone and groups: http://logs.openstack.org/18/602618/1/check/openstacksdk-functional-devstack-tips/02b87a9/testr_results.html.gz | |
| 10:29:00 | tobiash | tosky: but I have no knowledge about these things | |
| 10:29:20 | tosky | oh, oki | |
| 10:35:45 | dtantsur | http://logs.openstack.org/18/602618/1/check/openstacksdk-functional-devstack-tips/02b87a9/controller/logs/screen-keystone.txt.gz?level=WARNING#_Sep_16_15_37_39_019932 | |
| 10:36:05 | dtantsur | tosky: keystone looks quite broken ^^^ | |
| 10:37:30 | dtantsur | tobiash: is it the same error for all failed jobs? sometimes they Just Fail | |
| 10:38:18 | tobiash | dtantsur: at least for the last 5 or 6 changes the same set of jobs fails deterministically | |
| 10:38:24 | tobiash | and I think also the same test cases | |
| 10:38:25 | dtantsur | meh | |
| 10:40:21 | tobiash | these started to fail somewhere between friday and saturday last week | |
| 10:40:41 | dtantsur | tobiash: oh, they did like a full rewrite of the API O___O https://github.com/openstack/keystone/commit/27f226554a21667eab62cfc61c13abfc2e780518 | |
| 10:40:48 | dtantsur | gotta have some fun | |
| 10:42:47 | tobiash | dtantsur: thanks for checking, I think I'll just wait and try a recheck tomorrow ;) | |
| 10:48:21 | dtantsur | tobiash: mind filing a bug against keystone? I'm pretty sure that code is broken | |
| 10:50:37 | tobiash | dtantsur: can do that in an hour | |
| 11:03:03 | dtantsur | tobiash: never mind, did it https://bugs.launchpad.net/keystone/+bug/1792913 | |
| 11:03:03 | openstack | Launchpad bug 1792913 in OpenStack Identity (keystone) "Keystone HTTP 500: AttributeError: type object 'GroupsResource' has no attribute '_get_domain_id_from_token'" [Undecided,New] | |
| 11:13:40 | tobiash | thanks :) | |
| 11:17:00 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DO NOT MERGE] Testing the gate fix https://review.openstack.org/603098 | |
| 11:17:34 | AJaeger | dtantsur, mordred, shade is also broken... | |
| 11:17:38 | dtantsur | tobiash: you can watch https://review.openstack.org/#/c/602808/ | |
| 11:17:42 | dtantsur | AJaeger: ^^^ | |
| 11:18:09 | AJaeger | dtantsur: thanks | |
| 12:58:41 | openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 13:51:32 | ml | what's the status of openstackcloud.py? Will this still be used in future releases or will this be deprecated in favor of proxy implementations through the Resource interface? | |
| 13:53:09 | ml | or is it still ok to implement convenience methods to query/update/delete resources by name_or_id ? | |
| 15:41:31 | mordred | ml: openstackcloud.py is still totally a supported interface and will continue to be - so it's great to implement convenience methods there | |
| 15:41:59 | mordred | ml: we will hopefully start to implement some of those methods using the underlying proxy methods, but that should be transparent to the user | |
| 16:00:03 | dtantsur | mordred: hey, welcome back! mind checking a few baremetal patches? https://review.openstack.org/599420 https://review.openstack.org/599533 https://review.openstack.org/600452 | |
| 16:00:18 | dtantsur | we're getting really close, probably one patch away, from switching metalsmith away from ironicclient :) | |
| 17:27:01 | ml | mordred: alright. thanks for the information | |
| 18:09:07 | mordred | dtantsur|afk: woot! reviewing now | |
| 18:30:26 | Shrews | mnaser: regarding https://github.com/ansible/ansible/pull/45701 , we have a policy of not having the service name (e.g., "nova") in the name of the module | |
| 18:30:52 | Shrews | mnaser: i think mordred wrote the naming rules down somewhere, but i forget where | |
| 18:31:48 | Shrews | mordred: https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/cloud/openstack/README.md | |
| 18:32:09 | Shrews | mnaser: oh, i forgot the "admin" module rule | |
| 18:32:19 | Shrews | heh | |
| 18:33:04 | mnaser | Shrews: i guess i indirectly followed the rules :D | |
| 18:33:23 | mnaser | the reason is it is managing nova services, and other services have their own 'services' sooo | |
| 18:34:27 | Shrews | mnaser: yeah | |
| 18:34:35 | Shrews | my bad. strange test failure though | |
| 18:45:34 | openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Fix some spaces in help messages https://review.openstack.org/603206 | |
| #openstack-sdks - 2018-09-18 | |||
| 00:16:27 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | |
| 03:26:35 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 05:07:22 | openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | |
| 06:39:12 | openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | |
| 07:38:06 | openstackgerrit | Andreas Jaeger proposed openstack/openstacksdk master: [DO NOT MERGE] Testing the gate fix https://review.openstack.org/603098 | |
| 11:30:48 | openstackgerrit | Dmitry Tantsur proposed openstack/api-sig master: Add document describing consuming version discovery https://review.openstack.org/459405 | |
| 13:16:58 | mordred | dtantsur|brb: I think we still don't quite have version discovery working right with sdk and keystoneauth and noauth endpoint_override in ironic | |
| 13:19:43 | mordred | I think it's hitting lines 105/106 in openstack/service_description.py | |
| 13:33:20 | openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 14:10:10 | dtantsur | mordred: :( | |
| 14:18:59 | dtantsur | mordred: why do we even need that fallback to base Proxy? | |
| 14:26:37 | mordred | dtantsur: I *think* it's useful for people developing against things that aren't in os-service-types yet? but I think if we detect things properly we shouldn't hit that fallback | |
| 14:26:42 | mordred | I'll try to figure that out in a little bit | |
| 14:31:59 | dtantsur | mordred: I feel like this fallback may be addressing a non-existing case.. | |
| 14:32:41 | mordred | dtantsur: it probably is | |
| 14:50:32 | dtantsur | mordred: oh, could you check another baremetal patch? https://review.openstack.org/#/c/594242/ I forgot about ti | |
| 15:40:24 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Wire in retries for all baremetal actions https://review.openstack.org/603427 | |
| 15:40:26 | dtantsur | comments on the direction are welcome ^^^ | |
| 16:50:10 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 17:03:01 | AJaeger | mordred , could you look at remaining python3-first changes, please? https://review.openstack.org/#/q/is:open+topic:python3-first+(project:openstack/os-client-config+OR+project:openstack-infra/requestsexceptions+OR+project:openstack/openstacksdk+OR+project:openstack-infra/shade) | |
| 17:03:09 | AJaeger | Who else reviews these and can help? | |
| 17:21:44 | mordred | AJaeger: done. I see you hit a bunch of rechecks already - those should work this time, the issues before were due to a keystone bug which has since been fixed | |
| 17:25:11 | AJaeger | mordred: yeah, waited for those ;) | |
| 17:25:13 | AJaeger | mordred: thanks | |
| 17:29:20 | openstackgerrit | Merged openstack-infra/requestsexceptions master: import zuul job settings from project-config https://review.openstack.org/594367 | |
| 17:57:11 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Update baremetal objects with fields added up to Rocky https://review.openstack.org/599420 | |
| 18:05:20 | openstackgerrit | Merged openstack/openstacksdk master: Consolidate cloud/base.py into functional/base.py https://review.openstack.org/591031 | |
| 19:00:09 | openstackgerrit | Merged openstack/openstacksdk master: Add sjc1 to vexxhost profile https://review.openstack.org/602985 | |
| 20:41:59 | openstackgerrit | Merged openstack-infra/shade master: import zuul job settings from project-config https://review.openstack.org/594368 | |
| 20:44:37 | openstackgerrit | Merged openstack-infra/shade master: switch documentation job to new PTI https://review.openstack.org/594369 | |
| 20:44:39 | openstackgerrit | Merged openstack-infra/shade master: add python 3.6 unit test job https://review.openstack.org/594370 | |
| 20:57:34 | openstackgerrit | Merged openstack-infra/shade master: Cleanup .zuul.yaml https://review.openstack.org/598585 | |
| 21:26:51 | openstackgerrit | Merged openstack/openstacksdk master: switch documentation job to new PTI https://review.openstack.org/594374 | |
| #openstack-sdks - 2018-09-19 | |||
| 00:34:03 | openstackgerrit | Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528 | |
| 02:08:35 | openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | |
| 02:12:16 | openstackgerrit | Wenran Xiao proposed openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | |
| 03:34:12 | openstackgerrit | Ian Wienand proposed openstack/osc-lib master: Mask password when specified as an argument with a space https://review.openstack.org/603528 | |
| 05:34:40 | openstackgerrit | Merged openstack/python-openstackclient master: Fix help message for subnetpool default-quota value https://review.openstack.org/603279 | |
| 06:03:14 | AJaeger | dtantsur|afk, mordred, https://review.openstack.org/#/c/594393/ fails on queens - any ideas how to fix the test suite? That's the final python3-first change to import for SDK... | |
| 06:20:58 | openstackgerrit | Andreas Jaeger proposed openstack/openstacksdk master: Update .zuul.yaml https://review.openstack.org/603563 | |
| 08:22:13 | openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Handle empty values in regions https://review.openstack.org/601485 | |
| 08:59:29 | dtantsur | AJaeger: ouch. mordred, shade CI seems to use new ansible with old shade/openstacksdk | |
| 08:59:46 | dtantsur | so it actually does not exercise shade. need to cap ansible? | |
| 09:01:07 | dtantsur | AJaeger: I'd wait for mordred, I'm not overly familiar with shade. But I suspect we may need to upper-cap ansible on all its ansible jobs. | |
| 09:18:28 | AJaeger | dtantsur: thanks for digging into it, hope mordred has an idea. | |
| 09:57:25 | mordred | AJaeger, dtantsur: I just pushed up another rev on 594393 | |
| 09:57:36 | dtantsur | wow, an early mordred | |
| 09:57:45 | dtantsur | (or is it late?) | |
| 09:57:59 | mordred | dtantsur: it's early - and for no reason I can determine :( | |
| 09:58:16 | dtantsur | happens :( | |
| 09:58:21 | mordred | but - maybe that means I'm going to be productive today | |
| 09:58:53 | dtantsur | early wake up - and productive? gotta need a few litres of coffee :) | |
| 10:03:00 | mordred | oh. yes. very much so | |
| 10:03:21 | ml | I got some issues with the CI too, maybe you can help | |
| 10:03:34 | AJaeger | mordred: if that job passes, we might need to merge the project-config change first to remove the "broken" ansible job to merge 594393. But since 594393 is the last one, this is fine... | |
| 10:03:54 | ml | want to enable neutron-fwaas for my implementation over here: https://review.openstack.org/#/c/592303/ but didn't have any success yet | |