| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-04-05 | |||
| 16:02:50 | ankur-gupta-f4 | Bout to head out to airport so will be on/off for the next couple hours but will log back in on the plane. | |
| 16:02:56 | mordred | cdent: the example iin https://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html#version-discovery says it was taken from nova, but it does not match what nova emits | |
| 16:03:19 | cdent | a) /me shakes tiny fist | |
| 16:03:24 | cdent | b) predates me | |
| 16:03:35 | mordred | k. I mena- I like what's in the spec better | |
| 16:03:36 | cdent | c) give me a few minutes and I'll look into it more closely and fix it | |
| 16:03:47 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Add --network and --port to server create https://review.openstack.org/444924 | |
| 16:03:50 | mordred | but I'm wondering if we should align the spec to nova or try to get nova to add an additional key to theirs? | |
| 16:03:55 | cdent | it is quite likely that what's in the spec is desirable | |
| 16:04:03 | mordred | cdent: (the tl;dr is nova uses "version" to mean max_version) | |
| 16:04:27 | cdent | oh yeah, that's an intentional change | |
| 16:04:30 | mordred | ok. cool | |
| 16:04:32 | cdent | the surrounding tesxt is wrong | |
| 16:04:42 | mordred | I'll make the project-navigator stuff align to the api-wg thing | |
| 16:05:00 | cdent | yes please | |
| 16:06:19 | mordred | cdent: also - for major versions without microversions, min_version and max_version can be omitted or contain empty strings? or only be omitted? or do you care? | |
| 16:08:24 | cdent | mordred: as far as I'm aware we haven't yet provided guidance for discovery in a non microversioned world, but my interpretation is that 'id' takes on the meaning of "the actual version you're getting, majorly" and min and max ought to be omitted | |
| 16:08:48 | reedip | sindhu : I updated the latest PS with the changes mentioned by dtroyer | |
| 16:09:09 | cdent | mordred: so, basically, keep id, put the version in there, leave min and max out | |
| 16:09:43 | sindhu | reedip: great! thanks :) | |
| 16:10:04 | dtroyer | is the presence/absense of min_version and max_version how we would detect microversion support? | |
| 16:10:31 | cdent | dtroyer: that seems to be the implication | |
| 16:10:38 | cdent | but I'm not sure if that's intentional or not | |
| 16:10:40 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 16:10:51 | dtroyer | I'm generalizing the client-side negotiation code to put into ksa… | |
| 16:12:52 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 16:13:21 | mordred | cdent: https://review.openstack.org/453361 | |
| 16:13:49 | mordred | dtroyer: that is my hunch, yes | |
| 16:14:27 | cdent | mordred: cool, will look properly soon | |
| 16:14:29 | mordred | dtroyer: and ooh - you adding version discovery and microversion-per-call suport? or just one or just theother? | |
| 16:14:50 | dtroyer | I've started with the negotiation, will be doing per-call since OSC needs it | |
| 16:15:04 | mordred | woot | |
| 16:15:08 | dtroyer | mordred: https://review.openstack.org/451618 | |
| 16:15:15 | mordred | I've been avoiding bothering with that so far | |
| 16:15:18 | dtroyer | it's all in OSC for development ATM | |
| 16:19:21 | mordred | dtroyer: ah - so cool - I _also_ want major version discovery and have a minor portion of that somewhere - but can add that in when you're happy with what you've got there for microversions | |
| 16:19:25 | mordred | (and I like what you've got there) | |
| 16:29:28 | openstackgerrit | Chris Dent proposed openstack/api-wg master: Remove reference to nova on version discovery https://review.openstack.org/453744 | |
| 16:35:11 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce L7Policy for Octavia (load balancing) https://review.openstack.org/451980 | |
| 16:38:09 | mordred | dtroyer: also, I finally got around to writing a patch to horizon to add suport for downloading clouds.yaml files | |
| 16:38:20 | mordred | (fixing tests right now) | |
| 16:38:49 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce L7Policy for Octavia (load balancing) https://review.openstack.org/451980 | |
| 16:39:04 | dtroyer | mordred: \o/ | |
| 16:44:04 | mordred | dtroyer: remote: https://review.openstack.org/453669 Add support for downloading clouds.yaml files | |
| 16:50:49 | dtroyer | mordred: Nice. Maybe OSC should grow this too, and maybe both need to have a way to pull the update for the vendors file? | |
| 16:52:03 | mordred | dtroyer: yah - I have some patches I'll push up later today about vendors files too | |
| 16:52:28 | dtroyer | coolio | |
| 16:52:29 | mordred | dtroyer: and I've been thinking of making a few osc commands like "add cloud" that could injest a clouds.yaml from a vendor into an existing one | |
| 16:53:20 | dtroyer | yeah, thats a natural place to manage that file... | |
| 16:53:34 | mordred | ++ | |
| 16:57:17 | reedip | stevemar: there ? | |
| 17:00:49 | openstackgerrit | Merged openstack/python-openstackclient master: Enable to create legacy router https://review.openstack.org/449176 | |
| 18:10:40 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce L7Policy for Octavia (load balancing) https://review.openstack.org/451980 | |
| 18:46:19 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Develop api.versions for microversion discovery https://review.openstack.org/451618 | |
| 18:51:39 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce L7Rule for Octavia (load balancing) https://review.openstack.org/452832 | |
| 18:54:46 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Clean up password prompt work-arounds https://review.openstack.org/453832 | |
| 19:06:04 | malloc83 | hi there :-) just a tad curious why this isnt covered or just not documented.. i just started using the openstack proposed sdk.. and realised after authenticating and using authorize to get a token.. it pretty much doesnt mean much,, because after the user browses around on my gui, i have zero way of using the token to authenticate again on the sdk ;-) | |
| 19:06:52 | briancurtin | You can auth again with that token | |
| 19:06:57 | malloc83 | is this just not possible at this time? I noticed there is a plugin ? password? but nothing stating i could use plugin token if it even exists.. | |
| 19:07:16 | malloc83 | bradjones: yup i can,, but not using the sdk.. i cant find any intercace or something via the connection class that allows that. | |
| 19:07:25 | briancurtin | huh? | |
| 19:07:54 | malloc83 | so openstack has a connection class that i am using to connect in python, i use authorise() method to get a token... | |
| 19:08:00 | malloc83 | after that i can use the token again against the api... | |
| 19:08:09 | malloc83 | but how do i use it again with the connection handler? | |
| 19:08:41 | malloc83 | since the connection handler has been re-initilized, i cant afford to store a username and password in memory to auth again :) it must have some way for me to provide the token to continue the session | |
| 19:09:03 | briancurtin | as in, you want to create another connection instance elsewhere, perhaps in another instance of your application, using the token that was already received? | |
| 19:09:20 | malloc83 | bradjones: spot on.. this is for wsgi app.. | |
| 19:09:46 | malloc83 | briancurtin: i am sorry spot on yes.. :) getting used to this new mac keyboard. | |
| 19:10:08 | briancurtin | malloc83: that has definitely worked in the past. let me write something up and send it to you in a min | |
| 19:10:11 | malloc83 | perhaps i missed this in the documentation, looked through all the examples too. | |
| 19:10:16 | openstackgerrit | Chris Dent proposed openstack/api-wg master: Remove reference to nova on version discovery https://review.openstack.org/453744 | |
| 19:11:52 | malloc83 | briancurtin: it works great, just no idea how to re-use the token with the connection handler.. other than that, everything seems to be as expected.. the documentation however has no mention of using token for auth, just v2,v3 keystone username and password, unscoped etc | |
| 19:15:39 | briancurtin | malloc83: is this what you’re looking for https://gist.github.com/briancurtin/a10f4ed985d51a64f9c5de4c63b685fc | |
| 19:15:54 | malloc83 | having a look quick | |
| 19:16:22 | briancurtin | malloc83: yeah, we use keystoneauth for all of the auth handling, so we need to do a better job either explaining some of what can be done, or link out to keystoneauth’s docs on it | |
| 19:16:28 | malloc83 | i guessed it exactly like that, didnt work.. :) but that would be worth putting in the docs. | |
| 19:16:50 | malloc83 | ill have a look now why, perhaps i broke it :) | |
| 19:16:59 | briancurtin | malloc83: what version are you using? | |
| 19:17:16 | malloc83 | v2 auth api ? or what you asking for? the sdk? | |
| 19:17:22 | briancurtin | SDK version | |
| 19:17:30 | malloc83 | checking quick... | |
| 19:17:51 | malloc83 | openstacksdk==0.9.14 | |
| 19:18:16 | malloc83 | its the well defaulto version via pip install. | |
| 19:19:19 | briancurtin | ok cool, yeah from the SDK side of things that should all work (it maybe wouldn’t work on something way old, though i can’t remember how old). if you have more info on what failed and how i can maybe help figure it out. if it’s something with keystone itself though, that i don’t really know about | |
| 19:20:14 | malloc83 | briancurtin: no problem, just checking my understanding.. if i have a unscoped token on v2, i should be able to scope a new one using that against a project/tenant right? getting a 401 | |
| 19:20:44 | briancurtin | i know even less about keystone than what you just asked about | |
| 19:20:55 | malloc83 | briancurtin: haha cool :) understood. | |
| 19:21:05 | malloc83 | thanks for your help so far :) | |
| 19:21:35 | sindhu | stevemar: hi, https://review.openstack.org/#/c/383025/ is ready for your review :) | |
| 19:22:15 | openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Allow override of distributed router flag https://review.openstack.org/433442 | |
| 19:34:11 | openstackgerrit | Merged openstack/api-wg master: Clarify the meaning of BODY https://review.openstack.org/451568 | |
| 19:52:41 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstacksdk master: Introduce L7Rule for Octavia (load balancing) https://review.openstack.org/452832 | |
| 20:06:43 | openstackgerrit | Chris Dent proposed openstack/api-wg master: Create a set of api interoperability guidelines https://review.openstack.org/421846 | |
| 20:51:42 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstacksdk master: Introduce Pool for Octavia (load balancing) https://review.openstack.org/449264 | |
| #openstack-sdks - 2017-04-06 | |||
| 00:49:41 | openstackgerrit | Merged openstack/python-openstacksdk master: Clean up some errant doc warnings/errors https://review.openstack.org/453661 | |
| 01:43:44 | openstackgerrit | Merged openstack/python-openstackclient master: Add "qos-policy" option to "port create" & "port set" https://review.openstack.org/419236 | |
| 01:55:24 | openstackgerrit | Merged openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 05:04:10 | openstackgerrit | Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585 | |
| 05:10:03 | openstackgerrit | Sean McCully proposed openstack/keystoneauth master: KeystoneAuth should default to system CAFile. https://review.openstack.org/452585 | |
| 05:39:03 | openstackgerrit | yangweiwei proposed openstack/python-openstackclient master: Update endpoint-delete, endpoint-show, endpoint-set method https://review.openstack.org/453429 | |