Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-07-06
20:23:53 mordred efried: yah- sorry about that
20:44:31 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Improve validation & documentation https://review.openstack.org/478556
20:51:31 openstackgerrit Merged openstack/service-types-authority master: Improve validation & documentation https://review.openstack.org/478556
21:00:02 openstackgerrit Eric Fried proposed openstack/service-types-authority master: Remove validate.py main() https://review.openstack.org/478562
21:22:20 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add ./ to latest link https://review.openstack.org/481283
21:22:27 openstackgerrit Gage Hugo proposed openstack/python-openstackclient master: WIP - Add project tags functionality https://review.openstack.org/481284
21:31:34 openstackgerrit Merged openstack/service-types-authority master: Add ./ to latest link https://review.openstack.org/481283
21:50:04 efried mordred The return from discover.get_version_data - that's allowed to say max_version='latest', right?
21:52:47 efried kinda looks like not.
21:52:49 mordred efried: the top-level get_version_data?
21:52:54 efried yeah
21:53:07 mordred nope - that'll only have values that show up in version discovery documents
21:58:35 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Stop publishing yaml file https://review.openstack.org/481305
22:19:01 openstackgerrit Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
22:19:08 efried mordred ^ it's a start.
22:24:16 efried mordred Is there some reason you didn't want Discover.versioned_data_for() to return an EndpointData?
22:24:22 mordred efried: great patch! I left a few comments - mostly centered around one thing that will likely make you punch yourself in the face
22:26:46 mordred efried: I'm not coming up with a good reason right off the bat - it's probably a result of how that stack was developed
22:27:31 efried mordred Possibly you were trying to keep it consistent with data_for(), which it seems to supersede. But it seems like since you're superseding anyway, you might as well make it prettier. I haven't dug all the way in yet, though.
22:27:34 mordred efried: originally versioned_data_for was just an update to data_for
22:27:38 efried right
22:27:44 mordred efried: but then we realized that it was changing data_for's behavior
22:27:53 mordred which we can't do
22:27:58 efried So might as well shoot the whole hog.
22:28:10 mordred so the solution was to just split it out - which we could do without reworking the whole stack
22:28:39 mordred HOWEVER - we're not released yet -so reworking it to return an EndpointData potentially makes sense
22:28:52 mordred (like you say - might was well shoot the whole hog)
22:30:29 mordred efried: might be a smidge tricky given how get_versioned_data works ... but if you see a path - go for it
22:30:35 efried k
22:32:32 openstackgerrit Dean Troyer proposed openstack/cliff master: add tests for display command classes and hooks https://review.openstack.org/481192
22:40:04 openstackgerrit Rajath Agasthya proposed openstack/cliff master: Run hooks for DisplayCommandBase https://review.openstack.org/479157
22:40:05 openstackgerrit Rajath Agasthya proposed openstack/cliff master: add tests for display command classes and hooks https://review.openstack.org/481192
#openstack-sdks - 2017-07-07
01:49:26 dtroyer mordred: I am queueing up some lib releases (will be out of town for a few days), https://review.openstack.org/481355 is for o-c-c. +/-1 it as needed, I don't think there is a rush (yet).
02:58:41 openstackgerrit Merged openstack/cliff master: Run hooks for DisplayCommandBase https://review.openstack.org/479157
03:06:43 openstackgerrit Merged openstack/cliff master: add tests for display command classes and hooks https://review.openstack.org/481192
04:02:37 openstackgerrit lihaijing proposed openstack/python-openstackclient master: Replace six.iteritems() with .items() https://review.openstack.org/481407
06:07:46 openstackgerrit lihaijing proposed openstack/python-openstackclient master: Replace six.iteritems() with .items() https://review.openstack.org/481407
12:40:46 openstackgerrit M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
13:51:37 openstackgerrit Merged openstack/python-openstackclient master: Use openstackdocstheme in release note https://review.openstack.org/481173
15:00:08 GreenBlood Somebody using the Shade lib/client able to confirm that listing the images omits the shared ones ?
15:04:59 GreenBlood Listing images with glanceclient.images.list() and with the Shade wrapper does not give me the same results it seems
15:10:28 GreenBlood (Altho judging by the API calls they look very much the same so I don't understand). There is only "is_public=None" and "limit=20" in the glanceclient call, and no argument on the Shade call ([endpoint]/v2/images)
15:24:09 mordred GreenBlood: there is a potential behavioral difference depending on how new the glance service is
15:24:55 mordred GreenBlood: gimme a few minutes and I'll go dig up what the deal is
15:25:10 mordred actually - that's a silly thing to say
15:26:45 GreenBlood Yeah but the service is the same wether I use the glanceclient or shade/openstackclient nah ?
15:27:40 GreenBlood Oh wait I think I noticed something
15:27:54 GreenBlood I think our glanceclient uses v1 and shade uses v2
15:28:08 GreenBlood that would make sense
15:29:02 mordred ah - that'll definitely do it - however, it's possible also that we're not sending somehting we need to to get shared images returned
15:29:58 GreenBlood Judging by the HTTP debug, shade sends "GET /v2/images " and our code using glanceclient sends "GET /v1/images/detail?limit=20&is_public=None"
15:31:30 mordred kk. so - you can force shade to use v1 by setting image_api_version=1 in your clouds.yaml file - also - for v2, there is this note:
15:31:44 mordred "If you omit this parameter (visibility ), the response shows public, private, and those shared images with a member status of accepted."
15:32:11 mordred so images in v2 default to "shared" if you don't set any other status, and 'shared' images will only show up for a user in v2 by default if that user has "accepted" the image
15:33:21 mordred GreenBlood: so - list_images in v2 with no other parameters omitting shared by not accepted images is 'correct' behavior - although it might be surprising
15:33:35 GreenBlood Yeah well the user has accepted, I can see it in Horizon and stuff.
15:33:47 GreenBlood I'll try using v1 api but yeah its a bit wierd imho
15:33:50 mordred ah! well, that seems like a whole different bug
15:34:11 mordred in fact, that is a glance behavior that does not match the docs
15:34:21 GreenBlood Oh
15:35:34 mordred so - fwiw, the way to get shade to show you shared-but-not-accepted images is to do search_images(filters=dict(member_status='all')) ... that's not super friendly - so I think we should add a flag to the api to make it easy to opt-in to seeing _ALL_ images including unaccepted shared images
15:41:36 GreenBlood Good to know
15:42:01 GreenBlood But it's really bugging me this listing issue thingy
15:42:32 GreenBlood that would be really wierd to have shared images on v1 and not v2 no ?
15:48:41 GreenBlood Using "image_api_version: 1" in the cloud config file solves the issue
15:53:22 GreenBlood btw this tool, Shade, is really awesome
15:53:39 GreenBlood I discovered it the other day and I'm amazed
15:56:15 mordred GreenBlood: yay! I'm glad you like it!
15:56:39 mordred GreenBlood: yah - it was a behavior change in the v2 api - v1 doesn't understand shared at all, so it doesn't filter anything
15:57:06 GreenBlood Yeah but search with member_status='all' doesnt change the result I think
15:57:14 GreenBlood or maybe i'm just blind
16:07:14 GreenBlood which would be sad, really
16:21:00 GreenBlood Even with the CLI, doing "glance --os-image-api-version 1 image list" and without specifying the version, the shared images appears or disappears. I'll send a mail to my cloud provider :)
16:21:07 GreenBlood Thank you -again- for your help
16:38:29 openstackgerrit Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
16:38:49 fried_rice mordred ^
16:59:03 openstackgerrit Merged openstack/keystoneauth master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479191
17:12:16 mordred GreenBlood: https://review.openstack.org/481702 made a patch to add a show_all flag
17:14:21 GreenBlood oooh
17:14:22 GreenBlood nice
17:21:48 GreenBlood ok i tested it and the behaviour does not changes, so it must be an issue on the provider's side
17:22:34 GreenBlood wait no i'm just dumb
17:22:36 GreenBlood Gah
17:22:44 GreenBlood I'll test it monday
18:04:39 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Stop publishing yaml file https://review.openstack.org/481305
18:05:20 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add api_reference_project field https://review.openstack.org/480721
18:17:45 openstackgerrit Eric Fried proposed openstack/service-types-authority master: add types for freezer and monasca-log-api https://review.openstack.org/480718
20:23:12 openstackgerrit Eric Fried proposed openstack/keystoneauth master: normalize_version_number([1]) => (1, 0) and docs https://review.openstack.org/481309
20:25:07 fried_rice mordred ^ after much frustration with docstring formats. Where can I find out wtf it's expecting? Cause it doesn't seem to be standard RST.
20:39:58 openstackgerrit Eric Fried proposed openstack/keystoneauth master: Fix _run_discovery caching; misc cleanup https://review.openstack.org/481754
20:43:29 fried_rice mordred ^ needs to go in before we cut the next release, else it's bogus (changes sig of at least one public method, but that method hasn't been released yet AFAICT).
20:44:58 fried_rice mordred Also, forget what I said yesterday about Discover.versioned_data_for() returning an EndpointData. The contents seem to be quite different - though I'm still trying to understand why and where each applies.
#openstack-sdks - 2017-07-08
00:44:43 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/480277
08:03:07 openstackgerrit Sean McGinnis proposed openstack/python-openstackclient master: Explicitly use volume API v2 https://review.openstack.org/481799
21:18:07 openstackgerrit Akihiro Motoki proposed openstack/osc-lib master: Emit warnings to encourage cliff FormattableColumn https://review.openstack.org/462150
#openstack-sdks - 2017-07-10
10:42:44 openstackgerrit M V P Nitesh proposed openstack/python-openstackclient master: List images with --tags for 'openstack image list' https://review.openstack.org/481615
12:05:27 openstackgerrit NidhiMittalHada proposed openstack/python-openstackclient master: image-list should support filters 'name','status' https://review.openstack.org/479670
12:21:01 openstackgerrit NidhiMittalHada proposed openstack/python-openstackclient master: image-list should support filters 'name','status' https://review.openstack.org/479670
12:57:06 openstackgerrit Merged openstack/service-types-authority master: Stop publishing yaml file https://review.openstack.org/481305
13:01:27 openstackgerrit Chris Dent proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140

Earlier   Later