Posted	Nick	Remark
#openstack-sdks - 2018-01-30
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 https://review.openstack.org/538536
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: Add a descriptor object for each service proxy https://review.openstack.org/539232
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: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: Add OpenStackCloud object to Connection https://review.openstack.org/539234
17:37:55	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
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: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233
18:21:00	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
18:21:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
18:21:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
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 :)
02:12:54	mordred	adriant: BWAHAHAHAHA. found it and it's silly
02:13:17	mordred	adriant: http://paste.openstack.org/show/658082
02:13:59	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232
02:14:00	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233
02:14:00	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234
02:14:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
02:14:01	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271

Earlier   Later