| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-06-28 | |||
| 19:26:14 | mriedem | oh wait we passed a law on nit picking didn't we? | |
| 19:38:29 | mriedem | mordred: supports_microversion is <= right? | |
| 19:44:35 | mriedem | mordred: comments inline | |
| 19:45:30 | mordred | mriedem: yes - <= ... and thanks! | |
| 19:48:48 | mriedem | and a late comment | |
| 19:48:55 | mriedem | 'host' is required in the body always, even if host=None | |
| 19:48:56 | mriedem | it's dumb | |
| 19:49:06 | mriedem | "The host to which to migrate the server. If this parameter is None, the scheduler chooses a host." | |
| 19:56:16 | rm_work | dtroyer: thanks for pushing that tag change through :) | |
| 19:59:21 | rm_work | mordred: was it you i was talking to about fixing some of the neutron version discovery issues that were introduced in 0.10? | |
| 19:59:36 | rm_work | I don't remember where we left that | |
| 20:01:16 | rm_work | looks like it's still not working in 0.14.x for me, though the message is a bit clearer :P | |
| 20:23:15 | rm_work | Failed to contact the endpoint at https://openstack:9696 for discovery. Fallback to using that endpoint as the base url. | |
| 20:23:21 | rm_work | NotFoundException: 404: Client Error for url: https://openstack:9696/networks, Not Found | |
| 20:23:29 | rm_work | ^^ on Liberty | |
| 20:56:01 | mordred | rm_work: I *think* we have a fix for that in master and I'm a bad person who has not cut a release for you yet | |
| 20:56:12 | rm_work | lol np, i can test it | |
| 20:56:22 | rm_work | you have a CR? | |
| 20:56:24 | rm_work | or, SHA | |
| 20:56:29 | rm_work | i can apply | |
| 20:56:36 | rm_work | or i guess i could just checkout master | |
| 20:56:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Only send force parameter to live migration if supported https://review.openstack.org/578960 | |
| 20:56:57 | mordred | mriedem: ^^ updates. thanks for the feedback btw - super helpful! | |
| 20:56:59 | mordred | rm_work: looking | |
| 20:57:00 | rm_work | yeah sec i'll just switch to master | |
| 20:57:03 | rm_work | don't worry about it | |
| 20:57:04 | mordred | ok. cool | |
| 20:57:31 | rm_work | hmmmmmmm | |
| 20:57:48 | mordred | it's also possible that we looked at this, I said "oh, I see the issue and will write a patch" and then didn't | |
| 20:57:56 | rm_work | yeah i think it's the latter | |
| 20:58:03 | rm_work | master seems to be the same | |
| 20:58:26 | mordred | yes! | |
| 20:58:30 | mordred | I remember the thing now | |
| 20:58:54 | rm_work | yeah everyone using this cloud is stuck on 0.9.x which is just sad T_T | |
| 20:58:59 | mordred | kmalloc: ^^ on older clouds, neutron's version discovery document is auth protected (because of course it is) | |
| 20:59:06 | rm_work | lolol yes | |
| 20:59:26 | mordred | kmalloc: I kind of think we should just update ksa to send a token if it has one when doing discovery | |
| 20:59:42 | mordred | kmalloc: (but not to get a token if it doesn't already have one perhaps) | |
| 21:00:05 | mordred | kmalloc: or else I can probably work around it in SDK - but it's definitely a weird gotcha for folks | |
| 21:00:08 | rm_work | wouldn't that just make the error inconsistent then? :( | |
| 21:00:39 | kmalloc | Hmm | |
| 21:00:44 | mordred | rm_work: hrm. good point | |
| 21:01:05 | kmalloc | Yeah it is weird | |
| 21:01:53 | rm_work | can you assume that if it is auth protected, that it's one of the old versions pre-discovery? and that tells you what the endpoint is? :P or is it still variable? | |
| 21:01:56 | mordred | kmalloc: the sdk answer will basically be "if network_endpoint_override is None and not catalog_url.endswith('/v2.0'): network_endpoint_override = catalog_url + '/v2.0' | |
| 21:02:20 | mordred | rm_work: well - *generaly* no - but for neutron it turns out there is one and only one version if it's old | |
| 21:02:26 | rm_work | yes | |
| 21:02:30 | rm_work | that is what i was thinking | |
| 21:08:45 | mordred | AHHHHH | |
| 21:12:31 | rm_work | :P | |
| 21:14:07 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Hardcode v2.0 onto end of neutron endpoints https://review.openstack.org/579015 | |
| 21:14:17 | mordred | rm_work: ^^ untested yet - but maybe that'll fix it for you | |
| 21:14:31 | rm_work | will try it | |
| 21:15:24 | rm_work | mordred: A++ :) | |
| 21:15:31 | rm_work | thanks | |
| 21:15:43 | mordred | rm_work: it worked? | |
| 21:15:45 | rm_work | yep | |
| 21:15:48 | mordred | yay! | |
| 21:15:53 | mordred | I will update that patch with some tests | |
| 21:16:09 | rm_work | review: A++, would bother mordred about patches again | |
| 21:16:14 | mordred | :) | |
| 21:16:28 | rm_work | thanks muchly | |
| 21:16:35 | mordred | kmalloc: ^^ in case you wanna look at the evil hacky thing | |
| 21:16:37 | rm_work | starting to have stuff we'd like to have past 0.9.x :P | |
| 21:27:16 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Hardcode v2.0 onto end of neutron endpoints https://review.openstack.org/579015 | |
| 21:33:45 | kmalloc | K | |
| 21:57:32 | openstackgerrit | Sorin Sbarnea proposed openstack/cliff master: Assure executable name is kept when app is called as module https://review.openstack.org/579025 | |
| 22:11:51 | openstackgerrit | Lance Bragstad proposed openstack/python-openstackclient master: Implement support for project limits https://review.openstack.org/575211 | |
| 22:11:51 | openstackgerrit | Lance Bragstad proposed openstack/python-openstackclient master: Implement support for registered limits https://review.openstack.org/574475 | |
| 23:08:59 | kmalloc | mordred: looking at evil hacky-thing now that i am not on mobile | |
| 23:09:41 | kmalloc | mordred: gross, but hey, if it fixes it | |
| #openstack-sdks - 2018-06-29 | |||
| 01:48:13 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Fix a typo https://review.openstack.org/572771 | |
| 02:47:42 | openstackgerrit | Merged openstack/python-openstackclient master: Add ability to filter image list by tag https://review.openstack.org/575256 | |
| 03:25:26 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 06:22:47 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| 08:52:27 | openstackgerrit | wu.chunyang proposed openstack/openstacksdk master: Add release note link in README https://review.openstack.org/578672 | |
| 08:59:08 | openstackgerrit | wu.chunyang proposed openstack/openstacksdk master: Add release note link in README https://review.openstack.org/578672 | |
| 09:49:14 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 11:43:07 | dtantsur | mordred: morning! dumped more thoughts in https://etherpad.openstack.org/p/microversions-better-than-beer | |
| 11:59:47 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 12:01:22 | mordred | dtantsur: morning! awesome | |
| 12:09:57 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 12:10:36 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 12:31:12 | dtroyer | dhellmann: if you have a minute and need a break today to think about something a bit different than py3 and TC-stuff, I'd love to hear your thoughts on https://etherpad.openstack.org/p/osc-included_image_signing. tl;dr: signing code that wants to be used in both OSC and server-side (nova so far), and an appropriate place for it to live. | |
| 12:35:11 | mordred | dtroyer: seems like functionality we'd want in sdk too | |
| 12:36:05 | dtroyer | mordred: I didn't think about that long, I suppose I had oslo on my brain a bit much | |
| 12:36:29 | dtroyer | is nova ready to use sdk? have they already added it as a dep? | |
| 12:36:51 | mordred | they haven't - I wasn't going to push on it until we cut the 1.0 | |
| 12:37:27 | dtroyer | agreed there, I wonder if this is enough of a reason to start | |
| 12:37:27 | mordred | but it shouldn't be an issue for it to. that said - looking at cursive real quick, I don't think sdk would want to pick it up as a depend | |
| 12:37:45 | mordred | dtroyer: I can make some initial patches to check it out | |
| 12:37:56 | dtroyer | I don't think you need to, the signing bits don't use the rest of cursive as far as I saw last night | |
| 12:38:04 | mordred | cool | |
| 12:38:05 | dtroyer | it was just a home | |
| 12:38:30 | dtroyer | pile on to that ML thread and see what Josephine thinks | |
| 12:38:34 | mordred | well - image signing code _definitely_ would be welcome in sdk | |
| 12:38:36 | mordred | will do | |
| 12:43:35 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Only send force parameter to live migration if supported https://review.openstack.org/578960 | |
| 12:43:51 | mordred | dtantsur: speaking of microversions ^^ | |
| 12:47:58 | openstackgerrit | Monty Taylor proposed openstack/osc-lib master: Use openstack.config for config and Session objects https://review.openstack.org/579139 | |