| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-01-29 | |||
| 23:07:55 | dtroyer | mordred: sorry, endless-phone-syndrome | |
| 23:08:17 | dtroyer | I don't recall ever seeing that level of Cinder issue, usually it's races with resources. this is different | |
| 23:10:21 | dtroyer | FWIW, I just +W 538958 | |
| 23:11:02 | mordred | woot | |
| #openstack-sdks - 2018-01-30 | |||
| 00:09:31 | adriant | mordred: as part of the sdk stuff you were going to potentially make some nice constructor functions for people using it without an underlying clouds.yaml | |
| 00:10:57 | adriant | there is something that sort of lets you do that, but it... still looks for underlying cloud.yaml unless you explicitly tell it not to or something. | |
| 00:11:07 | adriant | am trying to find where that doc was | |
| 00:16:40 | adriant | here: https://github.com/openstack/python-openstacksdk/blob/master/doc/source/user/connection.rst#getting-a-connection-from-python-arguments-avoiding-cloudsyaml | |
| 00:17:39 | adriant | you were talking about making that into a nicer callable constructor that does that, without all the pointless boiler plate | |
| 00:20:56 | mordred | adriant: yes indeed. I'm just now AFKing - but thanks for the reminder I can hit that first thign in the morning | |
| 00:21:14 | adriant | mordred: sweet, thanks :) | |
| 00:21:38 | adriant | and are they any docks on doing that type of thing and saying what versions of stuff you want? | |
| 00:22:09 | adriant | since sometimes you do want to use a specific version of a given services API when you setup something like that in code | |
| 00:22:13 | adriant | ... | |
| 00:22:18 | adriant | docs* not docks | |
| 00:23:50 | adriant | I'll be around tomorrow and can test/review as needed! | |
| 00:30:08 | mordred | adriant: the docs need to be cleaned up - the tl;dr is that you can pass args like service_type_api_version to get_one_cloud (or to Connection itself) ... or via env vars .. | |
| 00:30:53 | mordred | adriant: so - for instance, OS_COMPUTE_API_VERSION='2' or compute_api_version='2' | |
| 00:31:01 | adriant | ideally, I would love a constructor function that is basically: explicitly ignore clouds.yaml, envvars, etc, and ONLY use what I'm giving it | |
| 00:31:23 | mordred | adriant: nod - can do | |
| 00:31:54 | adriant | a consumer of that is 100% certain of exactly where the config data it gets is coming from | |
| 00:32:13 | adriant | which is what I'd want for using it in services type stuff | |
| 00:32:40 | adriant | and cool, if you can easily enough supply what versions of a given service you need that makes it ideal | |
| 00:39:44 | mordred | adriant: woot. for now it's BASICALLY "config = openstack.config.OpenStackConfig(load_yaml_config=False, envvar_prefix='GARBAGE').get_one(region_name='my-region', auth=dict(auth_url='http://foo.example.com', username='', password=''), compute_api_version='2', compute_interface='public') ; conn = openstack.connect(config=config)" | |
| 00:40:05 | mordred | adriant: but that's obviously ugly - so when I awaken I'll get you a better option :) | |
| 00:40:12 | openstackgerrit | Merged openstack/python-openstackclient master: Use find_ip from openstacksdk https://review.openstack.org/538958 | |
| 00:40:15 | adriant | mordred: tyvm good dir! | |
| 00:40:18 | adriant | ... sir* | |
| 00:40:28 | mordred | I'm more like a dir :) | |
| 00:41:10 | mordred | adriant: I also have another version-selection related patch I've been hacking on today that should be ready for you to check out by tomorrow as well | |
| 00:41:51 | adriant | very cool. I'm going to play with the SDK as is today and see what it's like to use and mostly test out the old pain points I had with it. | |
| 00:42:03 | adriant | and tomorrow I'll follow up and see where you're at with that stuff | |
| 01:41:56 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Rename unit test cluster to clustering https://review.openstack.org/539071 | |
| 02:26:57 | openstackgerrit | OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902 | |
| 03:33:33 | openstackgerrit | Merged openstack/python-openstackclient master: Fix tox -e venv -- reno new |
|
| 05:01:50 | Qiming | modred | |
| 05:02:18 | Qiming | there? | |
| 05:19:12 | openstackgerrit | Merged openstack/python-openstacksdk master: Rename unit test cluster to clustering https://review.openstack.org/539071 | |
| 05:26:00 | adriant | mordred: the envvar_prefix doesn't work as intended. I'm hitting this when adding SDK support to my interpreter tool since I have a constructor function that takes elements from the cloud config as defaults, and lets you build a different connection object with a specified region and api versions. | |
| 05:26:24 | adriant | the error I'm getting is: OpenStackConfigException: Region RegionTwo is not a valid region name for cloud envvars. Valid choices are RegionOne. Please note that region names are case sensitive. | |
| 05:27:20 | adriant | when I do it in a clean env, no OS_* stuff set, it does work, but... when there are some set, I can actually build an entirely clean config/connection object entirely with values I supply. | |
| 05:28:30 | adriant | I can't* actually build | |
| 06:12:49 | adriant | mordred: code I was trying to get working that complains if OS_REGION_NAME is set: https://github.com/Adrian-Turjak/openstack-interpreter/pull/3/files#diff-f02c38fdd6ae5f99d7f313ce3277d81d | |
| 06:15:46 | adriant | also, if you know of how I can do something similar for the shade cloud object, that would be cool, because I would like to be able to build a shade cloud object with a constructor function that do pretty much the same thing I'm doing for my get_connection function with the SDK. | |
| 07:08:02 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:10:18 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:11:38 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:17:28 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: orchestration: fix typo in doc https://review.openstack.org/539116 | |
| 09:53:48 | openstackgerrit | Deepak Mourya proposed openstack/keystoneauth master: Override support message in AuthorizationFailure class https://review.openstack.org/539166 | |
| 10:54:14 | openstackgerrit | Kairat Kushaev proposed openstack/keystoneauth master: use defusedxml for XML parsing https://review.openstack.org/536761 | |
| 13:57:11 | openstackgerrit | Merged openstack/service-types-authority master: Add .zuul.yaml file with tox job https://review.openstack.org/538305 | |
| 14:19:01 | openstackgerrit | Merged openstack/service-types-authority master: Add masakari and service-type instance-ha https://review.openstack.org/534875 | |
| 15:03:04 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233 | |
| 15:03:04 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232 | |
| 15:03:05 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234 | |
| 15:03:22 | mordred | adriant: ok. that ^^ should take care of all of your questions and use cases (I hope) | |
| 15:36:16 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Update build to put content into sphinx location https://review.openstack.org/539239 | |
| 15:37:26 | mordred | dtroyer: jeez. it seems like devstack on pike is just flat broken for us there does't it :( | |
| 16:23:53 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Update build to put content into sphinx location https://review.openstack.org/539239 | |
| 16:30:16 | mordred | dtroyer: oh - those errors may have just been this morining's image issue | |
| 16:30:42 | dtroyer | mordred: I'm hoping so | |
| 16:32:39 | mordred | dtroyer: although the volume type create seems less happy | |
| 16:33:37 | mordred | locally, with stable/pike osc and master sdk I get: | |
| 16:33:39 | mordred | mordred@solace:~/src/git.openstack.org/openstack/python-openstackclient$ openstack --os-cloud=vexxhost volume type create monty | |
| 16:33:41 | mordred | Policy doesn't allow volume_extension:types_manage to be performed. (HTTP 403) (Request-ID: req-d9acb541-fa27-4983-9e07-65c5c0e42f6b) | |
| 16:33:52 | dtroyer | In the log I poked through that seemed to be the first osc invocation… I wonder if something got moved around? | |
| 16:33:56 | mordred | (which is what I'd expect, since I can't do that - but it's certainly getting past auth) | |
| 16:33:58 | dtroyer | oh, that's different | |
| 16:34:12 | mordred | so - my local error seems like correct behavior | |
| 16:34:29 | mordred | dtroyer: so - I might need to spin up a devstack and poke | |
| 16:36:51 | mordred | dtroyer: I put in a zuul hold so we can grab the node from that job next time it breaks and look | |
| 17:37:55 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270 | |
| 17:37:55 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234 | |
| 17:37:56 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271 | |
| 18:20:59 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232 | |
| 18:21:00 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234 | |
| 18:21:00 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233 | |
| 18:21:01 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271 | |
| 18:21:01 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270 | |
| 19:09:53 | openstackgerrit | Merged openstack/python-openstacksdk master: Provide compatibility for people passing raw sessions https://review.openstack.org/538968 | |
| 19:09:55 | openstackgerrit | Merged openstack/python-openstacksdk master: Update reno for stable/queens https://review.openstack.org/538693 | |
| 19:19:42 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: [WIP] Disallow setting default on internal network https://review.openstack.org/539303 | |
| 20:36:56 | adriant | mordred: looking at those constructor patches now | |
| 20:39:57 | mordred | adriant: awesome! thanks! | |
| 20:39:58 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: Disallow setting default on internal network https://review.openstack.org/539303 | |
| 20:50:17 | openstackgerrit | Colleen Murphy proposed openstack/python-openstackclient master: Add CRUD support for application credentials https://review.openstack.org/536163 | |
| 20:53:11 | adriant | mordred: with that code I think I can swap to just reusing the session, which actually makes things much simpler | |
| 20:53:33 | adriant | still reading, found a few typos on the comments/docs, but so far the code looks good | |
| 21:17:37 | mordred | adriant: awesome | |
| 22:33:41 | openstackgerrit | Merged openstack/python-openstacksdk master: orchestration: fix typo in doc https://review.openstack.org/539116 | |
| #openstack-sdks - 2018-01-31 | |||
| 00:11:10 | mordred | dtroyer: ok. I added a ton of debugging - and it stopped failing :( | |
| 00:12:22 | mordred | dtroyer: so I've rebased the real patch again and we'll see if it magically passes this time | |
| 01:48:33 | adriant | mordred: left you a note and a paste on one of those patches | |
| 01:49:43 | adriant | when I try initialising the connection directly with auth vars rather than a session, it still looks at the envvars | |
| 01:50:04 | adriant | but the sessions logic works! :P | |
| 01:51:13 | mordred | adriant: yay for the sessions! boo for the other thing ... thanks for test driving | |
| 01:51:22 | mordred | adriant: I've got another fun patch almost ready ... | |
| 01:52:04 | adriant | mordred: not a problem, and my current use case is a good one for testing this since it's within the context/scope of a running openstack client command, so there are envvars/clouds.yaml things to explicitly avoid :P | |
| 01:52:31 | adriant | it's rare someone will ever hit this same problem, but... better to ensure they don't because their context may be even worse | |
| 01:58:13 | mordred | the complex problems are the best :) | |