Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-09-25
12:46:45 mordred oh - you are so right!
12:46:46 samueldmq mordred: https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/openstackcloud.py#L4831
12:47:43 mordred yup
12:48:06 samueldmq mordred: that was a good catch
12:48:20 mordred ++ ... testing against real things is important :)
12:48:29 mordred samueldmq: did you find any clouds with glance v1 still?
12:48:40 samueldmq mordred: ah, besides all that, I have patches for normalizing network/subnet container and objects
12:48:59 samueldmq so they have a location and properties fields in it
12:49:16 mordred yes! I saw these and am excited by them (haven't fully looked at them yet, but will try to today)
12:49:18 samueldmq and look the same regarless what cloud is underneath
12:49:39 samueldmq mordred: they're not passing local tests yet, because unitt tests were not expecting too many fields
12:50:06 samueldmq normalize is important to everyhting... but for now I focused on these in my scope :(
12:50:29 samueldmq mordred: glance v1 was interesting.. nobody runs it by default, but some still have, e.g catalyst
12:50:44 samueldmq I had to set image_endpoint explicitly i nthe clouds.yml
12:50:53 samueldmq so it uses endpoint_override (iirc)
12:54:35 mordred well - good to hear nobody is using it by default :)
12:54:49 samueldmq hehe no ...
12:54:56 mordred samueldmq: so - when you're ready to really poke yourself in the eye ...
12:55:00 samueldmq btw I'll update vendor _support later on
12:56:27 mordred samueldmq: one of the things on the todo list, which is very similar to your normalize work
12:57:14 mordred samueldmq: is to get the shade layer to start using the underlying sdk layer - the sdk Resource objects are the equiv of the _normalize methods in the shade layer
12:57:39 openstackgerrit Merged openstack/openstacksdk master: Update the URL in doc https://review.openstack.org/604738
12:57:40 mordred samueldmq: it's a very hand wavey project - tons of questions about what it wants to look like and whatnot
12:58:36 samueldmq mordred: well, yes but I think there might be a few differences between shade normalize and the resource objs
12:58:55 mordred and like - taking image for example - I think we might want to move the _v1 methods into openstack/image/v1/_proxy and the _v2 methods into openstack/image/v2/_proxy.py - and then make sure both proxy classes have a create_image method that takes the same arguments - and then we can just have the shade layer create_image method call self.image.create_image(*args, **kwargs)
12:58:56 samueldmq I don't remember an exemp out of my head but I think I've taken a look into that
12:59:02 samueldmq but definitely something to do
12:59:16 mordred samueldmq: yes - there definitely are
12:59:39 samueldmq mordred: nice (image eg.)
12:59:39 mordred samueldmq: it's also possible that the upload_image example I just wrote about might be a TERRIBLE way to go about doing things :)
12:59:52 mordred so - there's certainly some exploration to do :)
12:59:59 samueldmq hehe but I got the idea
13:00:05 mordred \o/
13:00:11 mordred man, I need more coffee already this morning
13:00:30 samueldmq mordred: I want to have functional tests running against real clouds
13:00:40 samueldmq like in a period job running once a day or somehting
13:01:11 samueldmq I do'nt need to say only testing against devstack is very biased :-)
13:02:40 samueldmq mordred: ++ more coffee is always a good idea
13:28:08 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303
13:28:23 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811
13:59:03 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Fix location region field in docs https://review.openstack.org/605081
14:00:44 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Fix location region field in docs https://review.openstack.org/605081
14:27:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for per-service rate limits https://review.openstack.org/604521
14:27:25 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a semaphore and lock https://review.openstack.org/604926
14:41:12 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Import rate limiting TaskManager from nodepool https://review.openstack.org/574285
14:41:13 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add support for per-service rate limits https://review.openstack.org/604521
14:41:13 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Make RateLimitingTaskManager the TaskManager https://review.openstack.org/604520
14:41:14 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a semaphore and lock https://review.openstack.org/604926
16:04:24 openstackgerrit Merged openstack/openstacksdk master: Allow search on containers https://review.openstack.org/600680
16:17:29 openstackgerrit Merged openstack/openstacksdk master: Normalize image when using PUT on Glance v2 https://review.openstack.org/602031
16:17:31 openstackgerrit Merged openstack/openstacksdk master: Run all tasks through the threadpool https://review.openstack.org/603739
16:40:44 samueldmq fwiw https://review.openstack.org/#/c/605081/ is a really quick review/merge
16:40:54 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Normalize network resources https://review.openstack.org/602218
16:50:31 mordred samueldmq: why yes it is!
16:50:56 samueldmq mordred: \o/
16:52:38 timburke mordred: samueldmq: fyi on the container headers sample -- looks like that's coming from radosgw; i wouldn't rely on there being a X-Container-Bytes-Used-Actual header
16:53:09 timburke up to you whether to expose it to users anyway, but it's not something swift would respond with
16:53:14 samueldmq timburke: hmm but X-Container-Bytes-Used should be there right?
16:53:19 timburke yup!
16:53:52 mordred timburke: thanks! yeah - I'd like the sdk interface be things swift sends
16:53:58 samueldmq kk we wont rely on radosgw. it is not even 100% compatible
16:54:13 samueldmq it returns me 200 OK when I do get on an object that does not exist
16:54:26 timburke hmm... :-/
16:54:32 mordred that seems like an unfortunate life choice for it
16:54:59 samueldmq I will open a bug on that project
16:55:07 timburke i've got my own beef with some of our choices of response code, but that's pretty bad
16:55:29 samueldmq but it's hard to say you're compatible with an api that you don't run that api's funcional tests
16:55:39 samueldmq especially in a big and dynamic project like opesntack :-)
16:55:48 mordred that said - some amount of compat code in sdk to work around such things on behalf of users is in-bounds for sdk I think - we work around some weird life choices in rax publiccloud, for instance - but we should definitely keep the interface to be what's in swift
16:56:20 samueldmq mordred: agreed. we should try to keep it within openstack world
16:56:28 mordred ++
16:56:37 samueldmq in a few days someone runs openstack api's with amazon underneath
16:56:46 samueldmq and we don't need to make shade work with that all
16:57:38 mordred yeah ... I don't have much personal desire for that thing to have success
16:58:17 samueldmq :-)
16:59:42 openstackgerrit Merged openstack/openstacksdk master: Fix location region field in docs https://review.openstack.org/605081
17:30:24 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Normalize subnet resources https://review.openstack.org/602228
17:42:17 samueldmq mordred: on the get container convo ...
17:42:37 samueldmq if I look up the headers and create an object including storage_policy
17:43:00 samueldmq an entity returned by get_container would be different than one returned by list_containers
17:43:19 samueldmq because the one from get would have more attrs (e.g storage_policy)
17:43:45 samueldmq which is a bit weird
17:50:46 samueldmq or is it okay if a GET returns more details than a list do?
18:16:54 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Allow returning munch on create and get container https://review.openstack.org/596546
18:18:14 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Normalize container resources https://review.openstack.org/602233
18:31:22 Shrews mordred: comment on https://review.openstack.org/#/c/604521/6/openstack/connection.py
18:34:59 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Normalize object resources https://review.openstack.org/602234
18:36:43 mordred Shrews: your comment is correct
18:37:45 Shrews well that's a first
18:41:41 openstackgerrit Merged openstack/keystoneauth master: Reformat Adapter docstring https://review.openstack.org/605042
18:49:16 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Trim away the cover and py35 jobs https://review.openstack.org/605168
18:52:46 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Clean up python3 test and remove duplicate jobs https://review.openstack.org/604517
19:01:19 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Clean up python3 test and remove duplicate jobs https://review.openstack.org/604517
19:05:40 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Restore timeout_scaling_factor https://review.openstack.org/604628
19:34:35 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303
19:34:42 openstackgerrit Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811
22:05:38 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Update vendor support info for catalyst https://review.openstack.org/605245
22:11:22 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Update vendor support info for ecs https://review.openstack.org/605248
22:16:05 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Update vendor support info for switchengines https://review.openstack.org/605249
22:22:26 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Update vendor support info for vexxhost https://review.openstack.org/605252

Earlier   Later