| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-25 | |||
| 14:41:35 | mordred | corvus: from openstack.config import loader ; config = loader.OpenStackConfig().get_one(**args) ; conn = openstack.connection.Connection(config=config) | |
| 14:42:48 | corvus | mordred: still wants auth_url | |
| 14:43:32 | corvus | File \"/home/corvus/git/openstack-infra/zuul-jobs/.tox/py35/lib/python3.5/site-packages/keystoneauth1/loading/base.py\", line 162, in load_from_options\n raise exceptions.MissingRequiredOptions(missing_required)\nkeystoneauth1.exceptions.auth_plugins.MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url\n", | |
| 14:44:09 | corvus | that's from the get_one line | |
| 14:45:18 | mordred | poo. lemme poke further 0 sorry about that | |
| 14:48:39 | mordred | corvus: ok. I jave reproduced that lcaly | |
| 14:49:39 | dtantsur | folks, how do I deal with "The service catalog is empty" from session.get_endpoint of the service catalog is NOT empty? | |
| 14:49:50 | dtantsur | in what other cases can this happen? | |
| 14:50:02 | mordred | dtantsur: I've never seen that | |
| 14:50:08 | dtantsur | :( | |
| 14:50:19 | mordred | dtantsur: are you sure you have a token/catalog? | |
| 14:50:48 | dtantsur | mordred: the session is created with keystoneauth1.identity.v3.Token, the token comes from Mistral, no idea how it creates it | |
| 14:51:01 | dtantsur | it's tripleo, there is nothing I can be sure about :D | |
| 14:51:35 | mordred | dtantsur: it's possible the catalog got stripped from the token if it came from middleware | |
| 14:52:00 | dtantsur | mordred: the token - is it a string or an object? I think I get a string | |
| 14:52:01 | mordred | people like doing that to reduce size of the payload for reasons I don't really understand | |
| 14:52:20 | dtantsur | and now a funny thing: if I wrap it in an Adapter, it seems to work Oo | |
| 14:52:29 | mordred | weird | |
| 14:53:54 | dtantsur | extremely weird | |
| 14:57:02 | dtantsur | so, this apparently blows up https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L275 | |
| 15:02:16 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support passing profile to get_one https://review.openstack.org/585780 | |
| 15:02:24 | mordred | corvus: ^^ | |
| 15:04:06 | dtantsur | I guess I short report it against keystoneauth. it should not report EmptyCatalog if it's not empty in reality. | |
| 15:04:24 | corvus | mordred: that works, thanks! | |
| 15:04:38 | dtantsur | s/short/should/ (WUT) | |
| 15:05:13 | mordred | corvus: yay! | |
| 15:13:37 | dtantsur | filed https://bugs.launchpad.net/keystoneauth/+bug/1783590 | |
| 15:13:38 | openstack | Launchpad bug 1783590 in keystoneauth "EmptyCatalog raised when the catalog is not empty" [Undecided,New] | |
| 16:10:59 | dtantsur | mordred: do you know if I can pass the catalog to keystoneauth somehow? given that Token auth is, well, broken apparently in this regard | |
| 16:12:52 | dtantsur | ok, never mind, it just always treats the provided token as unscoped.. | |
| 17:04:26 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| 17:06:19 | dtroyer | mordred: if you're happy with the conversation, https://review.openstack.org/#/c/582334/ I think is ready | |
| 17:07:13 | dtroyer | that and https://review.openstack.org/#/c/558824/ are the two I think we want to wait on for the release…I'm doing a last pass through release notes yet | |
| 17:49:57 | mordred | dtroyer: +A on that one | |
| 17:51:37 | dtroyer | mordred: Thanks. did you intend https://review.openstack.org/585030 to go in now too? I just rechecked it... | |
| 17:52:14 | mordred | dtroyer: no - I mean, I wish - but we need a keystoneauth prelease to pick uit up | |
| 17:52:26 | dtroyer | ah, right. ok | |
| 17:52:32 | mordred | silly freezes | |
| 17:52:35 | mordred | ;) | |
| 17:52:57 | mordred | tbh - I've been meaning to write the ksa patch for weeks, so it's totally my fault | |
| 17:57:45 | openstackgerrit | Merged openstack/python-openstackclient master: Add command to show all service versions https://review.openstack.org/584905 | |
| 19:20:27 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Release note cleanup for 3.16.0 release https://review.openstack.org/585853 | |
| 19:31:35 | openstackgerrit | Merged openstack/python-openstackclient master: Don't sent disk_over_commit if nova api > 2.24 https://review.openstack.org/582334 | |
| 19:58:50 | dtroyer | mordred: https://review.openstack.org/#/c/585853/ is the relnote cleanup and likely 3.16.0 release commit, ready to go… | |
| 20:07:57 | mordred | love it | |
| 20:36:34 | openstackgerrit | Merged openstack/python-openstackclient master: Release note cleanup for 3.16.0 release https://review.openstack.org/585853 | |
| 20:56:38 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| 21:04:59 | mordred | dtroyer: woot. now I guess for next cycle there's no excuse I can give for not figuring out the remaining osc/sdk bits | |
| #openstack-sdks - 2018-07-26 | |||
| 01:51:08 | openstackgerrit | Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 02:58:41 | openstackgerrit | Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661 | |
| 05:02:39 | openstackgerrit | Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 05:03:45 | openstackgerrit | Nguyen Hai proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 07:11:51 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Add error message when network resources use "set" "unset" command https://review.openstack.org/579515 | |
| 07:24:24 | usr2033 | hi, does official openstack-sdk support quota operations? | |
| 08:58:15 | openstackgerrit | OpenStack Release Bot proposed openstack/python-openstackclient master: Update reno for stable/rocky https://review.openstack.org/586006 | |
| 09:01:02 | openstackgerrit | Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086 | |
| 09:02:57 | openstackgerrit | Josephine Seifert proposed openstack/python-openstackclient master: osc-included image signing (using openstacksdk) https://review.openstack.org/580086 | |
| 11:54:44 | openstackgerrit | Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661 | |
| 12:00:01 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532 | |
| 12:34:05 | openstackgerrit | OpenStack Release Bot proposed openstack/keystoneauth master: Update reno for stable/rocky https://review.openstack.org/586083 | |
| 12:35:07 | openstackgerrit | OpenStack Release Bot proposed openstack/openstacksdk master: Update reno for stable/rocky https://review.openstack.org/586094 | |
| 12:35:37 | openstackgerrit | OpenStack Release Bot proposed openstack/os-client-config master: Update reno for stable/rocky https://review.openstack.org/586100 | |
| 12:36:32 | openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Add flag for returning unofficial types https://review.openstack.org/585411 | |
| 12:36:33 | openstackgerrit | OpenStack Release Bot proposed openstack/osc-lib master: Update reno for stable/rocky https://review.openstack.org/586112 | |
| 12:48:32 | odyssey4me | mordred: I thought you might like to know a new and exciting and totally useful error I got the other day when I tried to do a compute thing with the openstack client. :) http://logs.openstack.org/15/582315/9/check/openstack-ansible-functional-ubuntu-xenial/911cdf7/job-output.txt.gz#_2018-07-21_09_14_00_043030 | |
| 12:49:09 | odyssey4me | It turns out, when you do not have an endpoint for the service, you get the rather oscure "'list' object has no attribute 'get'" error. | |
| 13:18:10 | mordred | odyssey4me: well! that certainly is exciting | |
| 13:24:33 | mordred | odyssey4me: I'm not 100% sure if that's coming from osc or novaclient- how hard would it be to recreate that with --debug ? | |
| 13:25:31 | odyssey4me | mordred: hmm, let me do that and get back to you - I might already have one up that I can break | |
| 13:25:55 | odyssey4me | oh bother, I don't - ok, let me kick off a build | |
| 13:36:14 | mordred | odyssey4me: woot. thanks! | |
| 16:07:40 | odyssey4me | mordred: it turns out that it's far more fun than anticipated - to replicate it you have to setup the compute endpoint with the wrong service backing it | |
| 16:08:09 | odyssey4me | in my test I put the placement service endpoints into the catalog, saying it was the compute service | |
| 16:08:11 | odyssey4me | https://gist.github.com/odyssey4me/fdfb2474bdd6fde4d1368af2af4044c6 | |
| 16:08:45 | odyssey4me | while of course this is not someone one would expect someone to do, it is perhaps better to handle the error somewhat more elegantly | |
| 16:12:18 | dtroyer | odyssey4me: this sounds like it might be a good integration test for OSC. Those drop commands in the top and use requests-mock so we let the entire stack have a chance to get it wrong, but still wouldn't require a specially-configured could to test against... | |
| 16:27:58 | cdent | thanks elmiko, was rather distracted during all that | |
| 16:28:36 | elmiko | cdent: no worries =) | |
| 16:28:51 | openstackgerrit | Merged openstack/api-wg master: Add links to errors-example.json https://review.openstack.org/578369 | |
| 16:28:52 | openstackgerrit | Merged openstack/api-wg master: Expand schema for error.codes to reflect reality https://review.openstack.org/580703 | |
| 16:33:32 | edleafe | cdent, elmiko, dtantsur: https://etherpad.openstack.org/p/api-sig-newsletter is ready for your review | |
| 16:34:17 | edleafe | Oops - hold on a minute | |
| 16:34:56 | edleafe | ok, it's ready now. Forgot to paste the links | |
| 16:35:07 | cdent | lgtm | |
| 16:35:50 | elmiko | edleafe: maybe reference that notmyname suggested we look at the draft? | |
| 16:36:12 | edleafe | elmiko: sure. You wanna add that? | |
| 16:36:18 | elmiko | k | |
| 16:37:23 | elmiko | edleafe: how's that look? | |
| 16:37:37 | edleafe | lgtm | |
| 16:37:39 | edleafe | ship it? | |
| 16:37:45 | elmiko | +1 | |
| 16:38:03 | elmiko | thanks again edleafe ! | |
| 16:39:03 | edleafe | sent | |
| 16:39:18 | elmiko | \o/ | |
| 19:10:20 | mordred | dtroyer, odyssey4me: yes to integration test! | |
| 19:11:20 | mordred | odyssey4me: and thanks for making that traceback ... it also looks like the base issue is in novaclient with an assumption on its part on how error payloads look. when we shift stuff over to sdk the error handler there is more permissive because it has to handle all of the services- so it's coded defensively to check and see what exists | |
| 19:11:35 | mordred | in any case, yes - I do think that error could be handled more elegantly | |
| 20:02:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add method for returning a raw response for an object https://review.openstack.org/586262 | |
| 20:02:55 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Update create_object to handled chunked data https://review.openstack.org/585532 | |
| 20:02:56 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for streaming object responses https://review.openstack.org/586250 | |