Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-01-10
16:02:18 cdent oh hey
16:02:26 elmiko so, edleafe, dtantsur, i got a question from the sahara team yesterday about microversions =)
16:02:32 elmiko LOL
16:02:36 elmiko tosky: you around here?
16:02:40 tosky hi
16:02:42 elmiko hey
16:03:01 elmiko the sahara team is on the precipice of releasing their v2 api
16:03:10 elmiko it does not currently contain support for microversion
16:03:22 elmiko but they would like to know if it acceptable to add that support mid-cycle
16:03:35 elmiko tosky, check me if i've got that wrong
16:04:07 cdent since you have to opt in to use a later microversion it is safe to add them at any point
16:04:09 tosky that's correct, I'd just like to add that it may not be a problem in the end (the patch is in a better shape than what we originally though) but still, just to be sure
16:04:09 cdent "safe"
16:04:59 cdent and the "first" microversion is usually what's "now"
16:05:01 tosky the point is: if we merge the patch which declare APIv2 as stable today or tomorrow, would adding microversioning support before (say) M3 of this cycle break the API?
16:05:24 edleafe What cdent said. What you release now will always be the default unless otherwise requested
16:05:25 elmiko cdent: that was my position too
16:05:49 elmiko see tosky, no big deal =)
16:05:58 tosky oh, ok, thanks
16:06:18 elmiko yay, we're helping!
16:06:20 elmiko \o/
16:06:28 tosky out of curiosity, and a bit of history, why did cinder decide to bump to v3 instead of reusing v2, if their v3 is just v2+microversioning?
16:06:53 smcginnis tosky: The concern was that v2 did not have any awareness of microversions.
16:07:16 cdent and v2 had been around for a long time, yes?
16:07:24 smcginnis tosky: So conceivable someone could send a microversioned request to an older v2 deployment expencting a certain behavior, but not getting it.
16:07:35 smcginnis And having some unintended side effects that could be risky.
16:07:37 tosky oh, I see
16:08:10 smcginnis So it was decided with the addition of microversions we would also create a new v3 api that was identical, but guaranteed to be able to handle microversions correctly
16:08:12 tosky thanks, our case is a bit different because the first release of v2 in few months (two?) will have microversioning
16:08:24 smcginnis cdent: Yeah, v2 was around for quite a while.
16:09:06 edleafe smcginnis: so you didn't take advantage of the major version bump to get rid of any old cruftiness?
16:10:23 smcginnis edleafe: No, unfortunately. We weren't prepared to remove anything, and we did want to make the transition from v2 to v3 as easy as possible to make it more likely that we could drop v2 in the near future and be able to get to the point of only having one API.
16:10:49 edleafe smcginnis: fair enough
16:11:24 edleafe Although there will be clients looking to use v2 foreever...
16:11:34 edleafe forever, even
16:11:57 smcginnis It's at least marked as deprecated.
16:12:18 smcginnis I had planned on spending some time reaching out to different SDKs and helping them get over to v3, but no time.
16:13:03 elmiko edleafe: i think foreever was correct, at least the way it sounded in my head XD
16:13:27 cdent "but not time" (there it is again)
16:14:02 cdent I have a question about guidelines
16:14:12 cdent I have a couple of to do items for guidelines that need to be written
16:14:34 edleafe smcginnis: Well, even if every SDK updated to use v3 only, there are old copies in the wild being used.
16:14:38 mordred smcginnis: I really need to get a v3 layer in sdk
16:14:48 cdent But ennui has me doing a bit of a "does anyyone really read these things anymore?" so I'm not compelled
16:15:14 elmiko cdent: i can empathize
16:15:25 mordred smcginnis: the shade layer supports v3, but the sdk layer doesn't - and it's obviously stupid that this is the case
16:15:32 mordred but - yeah - time
16:16:01 mordred cdent: TAKE IT BACK!
16:16:09 cdent no backsies
16:16:32 smcginnis mordred: Ah, I wasn't aware of that.
16:16:34 gtema cdent: I would love to have additional 5-10 hours in each day
16:17:41 mordred smcginnis: there is a patch up that's in merge conflict that I need to revive
16:17:43 elmiko cdent: did you want to move the items from your todo list back into the free pool?
16:18:04 smcginnis mordred: Feel free to add me when you do and I can see if there's any way I can help.
16:18:27 mordred will do!
16:18:39 mordred initial support should be really easy - since it can just be a copy of the v2 tree
16:18:42 cdent elmiko: I don't they were ever really in the free pool and then assigned to me in the first place, just ideas I had or stuff that seemed like a natural fit. The question isn't "what should we do about these things to do". It's should _anyone_ bother to do these things?
16:19:01 elmiko cdent: ahh, gotcha
16:19:22 edleafe cdent: sounds like your TC ennui is spreading...
16:19:43 elmiko i think it would be nice to capture what you were thinking about, maybe in storyboard, but i don't see a pressing need to conscript folks into action
16:19:55 cdent edleafe: it's basically about making sure that any effort spent is effort that enough people require
16:20:03 elmiko i just like the idea of not losing the ideas you had
16:20:37 cdent and the difficult is that people require something, but not what is at hand
16:20:45 cdent elmiko: yeah, ack to that
17:01:49 elmiko later edleafe, cdent, dtantsur, if i don't see you later have a good weekend =)
17:02:08 edleafe C ya, elmiko!
17:12:04 dtantsur o/
22:57:05 mordred dtroyer, frickler, amotoki: https://review.openstack.org/#/c/629243 is green and unbreaks the OSC gate
#openstack-sdks - 2019-01-11
00:14:07 openstackgerrit Merged openstack/openstacksdk master: Restrict inventory test to devstack-admin https://review.openstack.org/629875
02:16:36 openstackgerrit Kailun Qin proposed openstack/openstacksdk master: Add network segment range resource https://review.openstack.org/625381
09:03:38 openstackgerrit 98k proposed openstack/os-service-types master: Updatae home-page https://review.openstack.org/630075
09:22:47 openstackgerrit Lajos Katona proposed openstack/openstacksdk master: Add port property: port-resource-request https://review.openstack.org/630092
09:49:57 openstackgerrit Andriy Shevchenko proposed openstack/microversion-parse master: Updatae home-page https://review.openstack.org/630103
10:59:06 openstackgerrit Luigi Toscano proposed openstack/os-api-ref master: Fix microversion test: handle different HTML renderings https://review.openstack.org/630146
10:59:28 openstackgerrit Luigi Toscano proposed openstack/os-api-ref master: Generate stable targets rather than random https://review.openstack.org/625654
11:12:12 openstackgerrit Bruno Haleblian proposed openstack/openstacksdk master: Bug : identity v3 Proxy role assignments only support instances. https://review.openstack.org/629690
12:33:48 openstackgerrit Merged openstack/os-api-ref master: Fix microversion test: handle different HTML renderings https://review.openstack.org/630146
12:33:49 openstackgerrit Merged openstack/os-api-ref master: Generate stable targets rather than random https://review.openstack.org/625654
16:31:51 openstackgerrit Eric Fried proposed openstack/os-service-types master: Update home-page https://review.openstack.org/630075
18:15:04 openstackgerrit Andriy Shevchenko proposed openstack/microversion-parse master: Update home-page https://review.openstack.org/630103
18:59:07 openstackgerrit Merged openstack/python-openstackclient master: Use os-cloud instead of OS env vars for functional tests https://review.openstack.org/629243
20:18:54 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Compute location properly in server https://review.openstack.org/626380
21:41:06 openstackgerrit Merged openstack/openstacksdk master: Add port property: port-resource-request https://review.openstack.org/630092
21:54:12 openstackgerrit Merged openstack/openstacksdk master: Add network segment range resource https://review.openstack.org/625381
21:54:13 openstackgerrit Merged openstack/openstacksdk master: Bug : identity v3 Proxy role assignments only support instances. https://review.openstack.org/629690
#openstack-sdks - 2019-01-12
01:16:37 openstackgerrit Bruno Haleblian proposed openstack/openstacksdk master: Fix/Add : Identity V3 validate user role https://review.openstack.org/630413
03:21:16 openstackgerrit Andriy Shevchenko proposed openstack/python-openstackclient master: fix tox python3 overrides https://review.openstack.org/607453
05:25:28 openstackgerrit Andriy Shevchenko proposed openstack-infra/shade master: Change openstack-dev to openstack-discuss https://review.openstack.org/622373
#openstack-sdks - 2019-01-13
14:48:47 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Restrict inventory test to devstack-admin https://review.openstack.org/630512
15:05:22 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add block_storage v3 API support https://review.openstack.org/566355
15:31:14 mordred smcginnis: ^^
15:59:10 openstackgerrit Merged openstack/openstacksdk master: Fix/Add : Identity V3 validate user role https://review.openstack.org/630413
16:39:24 smcginnis mordred: Thanks!
16:58:55 openstackgerrit Merged openstack-infra/shade master: Restrict inventory test to devstack-admin https://review.openstack.org/630512
17:01:06 openstackgerrit Bruno Haleblian proposed openstack/openstacksdk master: implement identity v3 Proxy group role management https://review.openstack.org/630515
19:49:00 openstackgerrit Yuval Shalev proposed openstack/openstacksdk master: Fix for not released thread in service_description https://review.openstack.org/630517
#openstack-sdks - 2019-01-14
09:27:21 openstackgerrit Johannes Kulik proposed openstack/python-openstackclient master: Add floating IP filter to floating IP list command https://review.openstack.org/628941
10:22:05 openstackgerrit Merged openstack/openstacksdk master: Add block_storage v3 API support https://review.openstack.org/566355
10:25:13 openstackgerrit Merged openstack/openstacksdk master: Fix for not released thread in service_description https://review.openstack.org/630517
10:25:14 openstackgerrit Merged openstack/openstacksdk master: implement identity v3 Proxy group role management https://review.openstack.org/630515

Earlier   Later