| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-09 | |||
| 12:53:46 | mordred | Shrews: I thought you'd enjoy that | |
| 12:54:18 | Shrews | or maybe it's sad that that is what the api actually expects?? | |
| 12:54:37 | Shrews | i put nothing past openstack apis anymore | |
| 12:54:52 | mordred | nope. the api expects extra key-value pairs to be sent in the main image dict (this is one of the differences with v1 of the glance api which has a properties field) | |
| 12:55:13 | mordred | Shrews: so the sdk in this case is actually not doing that - and is not actually supporting dealing with the extra key-value pairs | |
| 12:56:15 | Shrews | so glance broke their own api by removing properties? | |
| 12:57:12 | Shrews | i mean, i guess you can do that moving from v1 to v2... but that seems silly | |
| 12:57:24 | Shrews | meh | |
| 12:57:57 | gtema | dtroyer: even cmd2==0.9.0.1 has a problem already | |
| 12:58:35 | mordred | Shrews: no - glance didn't break their own api - the sdk code implementing it never implemented it - and the test which theoretically showed it worked wasn't actually testing it | |
| 12:58:39 | mordred | Shrews: but yeah | |
| 12:59:33 | Shrews | ah | |
| 13:02:22 | mordred | Shrews: I think the whole stack should be green now - I had to add a property back to the https://review.openstack.org/589193 to make osc happy | |
| 13:51:18 | dtroyer | gtema: hmmm, ok, maybe py3 should be capped too… | |
| 14:25:18 | mordred | Shrews: if you get a sec, https://review.openstack.org/#/c/586727 could use eyes - it's needed for https://review.openstack.org/#/c/586728/ to work | |
| 14:40:10 | mordred | (we'll also need to cut a release with that in it) | |
| 14:40:51 | Shrews | +Ad | |
| 14:42:26 | mordred | yay | |
| 14:50:54 | elmiko | cdent edleafe dtantsur|brb heads up, i have a conflict at the start of the meeting, will most likely be late | |
| 14:53:32 | edleafe | elmiko: good, we'll have some time to gossip about you | |
| 14:55:10 | elmiko | + | |
| 14:55:13 | elmiko | ++ even | |
| 15:07:28 | elmiko | i might be signed up for a phone call after the conflicting meeting, no guarantees that i will make our meeting | |
| 15:16:45 | cdent | something something sparkle motion | |
| 15:52:30 | openstackgerrit | Merged openstack/openstacksdk master: Update config doc url to point to openstacksdk https://review.openstack.org/586726 | |
| 15:58:26 | openstackgerrit | Merged openstack/openstacksdk master: Handle image and object key metadata for shade transition https://review.openstack.org/586727 | |
| 16:29:10 | cdent | edleafe: you wanna go ahead merge https://review.openstack.org/#/c/589130/ as it's just a naming thing? | |
| 16:30:25 | cdent | edleafe, dtantsur, elmiko : In fact I'm not sure _any_ of that stack warrants full guideline-style review | |
| 16:30:38 | cdent | it's basically reformatting and linking | |
| 16:30:55 | cdent | the last one might be a guideline | |
| 16:31:02 | cdent | but not really | |
| 16:31:09 | edleafe | Agreed that they don't need the whole freeze process | |
| 16:31:20 | edleafe | But haven't looked them over yet in any detail. | |
| 16:31:57 | cdent | k, will not list them as new guidelines just refer to them in thebody of the message | |
| 16:32:09 | edleafe | I +W'd https://review.openstack.org/#/c/589130/1 | |
| 16:38:12 | cdent | edleafe, dtantsur, elmiko : I don't know what else to say: https://etherpad.openstack.org/p/api-sig-newsletter | |
| 16:40:02 | openstackgerrit | Merged openstack/api-sig master: Update web frontpage to link to SIG not WG https://review.openstack.org/589130 | |
| 16:41:28 | edleafe | cdent: Dunno, you could have added some jokes or something | |
| 16:41:40 | edleafe | But if you want to be boring and dull, LGTM | |
| 16:42:06 | cdent | an api walks into a bar | |
| 16:42:11 | cdent | bartender says | |
| 16:42:16 | cdent | what the foo are you doing here | |
| 16:46:59 | dims | LOL | |
| 16:56:58 | elmiko | sorry gang, meeting is long | |
| 16:57:30 | cdent | elmiko: look what you made me do | |
| 16:58:01 | elmiko | i love that joke though | |
| 17:00:00 | edleafe | elmiko: you should read the minutes. We totally trashed you | |
| 17:03:28 | cdent | "love"? | |
| 17:10:56 | elmiko | haha | |
| 18:40:06 | openstackgerrit | Merged openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 18:56:25 | mordred | boom | |
| 18:57:04 | mordred | dtroyer: we're just about half done with the 2 items I wanted to get done before a 1.0 | |
| 18:58:07 | mordred | dtroyer: I'm gonna try to get the other bit written next week - but hopefully will be on target for an early-cycle 1.0 so that we can actually start work in earnest on using it | |
| 19:51:04 | dtroyer | mordred: I like these words that I read… | |
| 19:56:06 | mordred | dtroyer: \o/ | |
| 19:57:00 | mordred | dtroyer: I'm probably going to start with glance support in osc - since I've got to do some surgery on that for sdk anyway so it'll all be paged in | |
| 20:10:18 | samueldmq | heya, do https://storyboard.openstack.org/#!/story/2003401 and https://storyboard.openstack.org/#!/story/2003402 make sense at all to you? | |
| 22:46:30 | openstackgerrit | Merged openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| #openstack-sdks - 2018-08-10 | |||
| 01:35:38 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 05:38:30 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 05:44:45 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 06:36:21 | usr2033 | hi, i have a simple code like this. https://paste.ofcode.org/t6eXJ2gx9MA9QX7UAGQrTF i have a fundamental problem i guess. Openstack calls takes random time for return and i couldn't handle this. Execution of this code returns four different value for same input. What is the best practice for this problem? | |
| 06:38:08 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Fix inconsistency (nit) https://review.openstack.org/572771 | |
| 06:44:23 | usr2033 | i couldn't different response even one line "os_connection.list_projects()" | |
| 07:59:55 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 09:27:04 | usr2033 | i think i found the problem. list_servers function can handle or work with undefined identity service version but list_projects function call does not. | |
| 12:06:08 | mordred | usr2033: ah- sorry about that - the very next thing on my todo list is to finish plumbing in version discovery so that _everything_ can work without a defined service version | |
| 12:59:45 | dtantsur | wait, does Image API really allow you to set random string attributes on an Image object? O___o | |
| 13:25:41 | mordred | dtantsur: yup | |
| 13:25:49 | mordred | dtantsur: that was one of the changes in image api v2 | |
| 13:25:58 | mordred | dtantsur: it got rid of the "properties" attribute | |
| 13:26:08 | dtantsur | wownotwow | |
| 13:26:16 | mordred | and instead put user-defined key/value metadata into the main image object itself | |
| 13:26:18 | mordred | right? | |
| 13:26:46 | mordred | in shade we chose to ignore that design choice and continue to present it as properties even if the service is running v2 | |
| 13:27:16 | mordred | which is what I expect to do to fix the brokenness around image in sdk currently | |
| 13:27:50 | mordred | dtantsur: what's even better is that operators can define top-level keys to be "reserved" in their config | |
| 13:28:23 | dtantsur | O.M.G. | |
| 13:28:34 | mordred | dtantsur: you can't un-know that | |
| 13:30:03 | dtantsur | unfortunately | |
| 13:30:12 | dtantsur | I'm looking into JSON patch support.. and it's a mess | |
| 13:31:07 | dtantsur | I'll throw a half-finished patch soon just to collect opinions | |
| 13:34:31 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Correct update operations for baremetal https://review.openstack.org/590770 | |
| 13:34:32 | dtantsur | mordred: wdyt about the overall approach ^^^ ? | |
| 13:40:55 | mordred | dtantsur: I suppose we can't just assume jsonpatch should be used if commit_message = 'PATCH' can we? | |
| 13:42:46 | usr2033 | mordred: thanks for response. i had a problem with get_project function. Normally get_project function in cloud/openstackcloud.py accept domain_id but in use like in get_compute_quota i can't pass domain_id to function. I had patched for that file for myself. I opened story in https://storyboard.openstack.org/#!/story/2003285 | |
| 13:43:58 | mordred | dtantsur: overall approach looks good | |
| 13:44:57 | dtantsur | mordred: I assumed HTTP PATCH can also be used with an incomplete resource representation | |
| 13:45:13 | dtantsur | though openstack services seem to use PUT for that | |
| 13:47:05 | mordred | yah - they do seem to - but I think you're right- better safe than sorry | |
| 13:49:38 | mordred | maybe we could default it to None, then have a little method like def _use_json_patch(self): if self.commit_method is None: return self.commit_method == 'PATCH' ; else return self.commit_method | |
| 13:49:52 | mordred | but maybe that'stoo much - dunno, just thinking out loud | |
| 14:30:08 | openstackgerrit | Alan Bishop proposed openstack/python-openstackclient master: Deprecate volume create --project and --user options https://review.openstack.org/575804 | |
| 14:53:38 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix the heat template resource more cleaner https://review.openstack.org/590786 | |
| 14:53:42 | mordred | Shrews: ^^ | |
| 15:21:01 | cdent | mordred: I wish to register my informal +1 on dtantsur as sdk core | |
| 15:32:25 | mordred | cdent: so noted | |
| 15:32:41 | cdent | he's OK | |
| 15:32:52 | cdent | someday might even get a huzzah | |
| 15:39:27 | mordred | cdent: wow. that's pretty strong | |