| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-08 | |||
| 18:33:16 | mordred | I am pretty sure they are unrelated, since that patch doesn't touch that job at all | |
| 18:33:38 | frickler | ack | |
| 18:34:20 | frickler | dtroyer: with the dependency merged, I hope https://review.openstack.org/558824 at last is now ready for some final reviews | |
| 18:34:27 | Shrews | are we comfortable approving it then? | |
| 18:35:29 | mordred | Shrews: yeah - I think it should be fine | |
| 20:03:59 | openstackgerrit | Merged openstack/openstacksdk master: Stop using the -consumer devstack jobs https://review.openstack.org/589675 | |
| 20:15:08 | openstackgerrit | Merged openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| #openstack-sdks - 2018-08-09 | |||
| 00:09:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 00:09:53 | mordred | Shrews: I found new terrible things I can't wait to tell you about | |
| 00:10:21 | mordred | adriant: no worries re: time - you've been busy! I mostly want to make sure that once this stack goes green, we make sure it doesn't break you before we land it | |
| 00:11:12 | adriant | mordred: it mostly looks like stuff that shouldn't affect me, but I won't get a chance to play with it until next week at the earliest | |
| 00:11:25 | mordred | adriant: kk. no rush | |
| 00:11:30 | adriant | it's just the changing of the Resource internals to make it dict based right? | |
| 00:11:56 | mordred | adriant: yah- and renaming two of the methods - but most things shoudl be touching those Resource bits via proxy methods | |
| 00:12:18 | adriant | Yeah, none of my code goes that deep into the Resource class | |
| 00:12:27 | mordred | woot. then you should be fine | |
| 00:12:44 | adriant | Most of the stuff that I always ran into was the session stuff you fixed :) | |
| 00:14:13 | adriant | mordred: oh and https://review.openstack.org/#/c/544544/ totally forgot about that patch, but the person who proposed it never responded | |
| 00:45:56 | samueldmq | mordred: Shrews: I liked having openstack cloud and operator cloud as separate things. having that boundary made sense from user's perspective | |
| 00:48:43 | samueldmq | for instance, say I want to validate the abstraction layer against all public clouds. that means I would only consider calls for users, excluding operator's calls (which were in a separate thing) | |
| 03:02:29 | openstackgerrit | Rui Chen proposed openstack/openstacksdk master: Add more options to enable ansible testing feature https://review.openstack.org/590078 | |
| 08:56:10 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 09:17:35 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 10:02:59 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 10:28:41 | gtema | does anyone experience broken osc interactive mode? I do always get "unrecognized arguments" for each command. Unfortunately same happend even with 3.14.2, so likely problem is elsewhere | |
| 10:36:01 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 10:40:49 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 12:13:17 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 12:25:56 | dtroyer | gtema: check your version of cmd2 (0.8.8) and cliff (2.13.0). Those are the versions I have in my development venv | |
| 12:27:17 | gtema | dtroyer: If I do a fresh 'pip install python-openstackclient' I get cmd2-0.9.3 and cliff-2.13.0 | |
| 12:29:43 | gtema | dtroyer, if I manually downgrade cmd2 to 0.8.8 it works, so probably freeze the version if the fix is not fast? | |
| 12:36:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| 12:36:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 12:36:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 12:36:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | |
| 12:42:22 | Shrews | mordred: i'm giddy with anticipation | |
| 12:43:20 | dtroyer | gtema: cmd2 has been a barrel of fun lately… we do have it capped at <0.9.0 for python 2, python3 is fine above that | |
| 12:43:50 | gtema | dtroyer: I'm having this issue with python 3.6 | |
| 12:44:22 | mordred | Shrews: in https://review.openstack.org/530835 - please to enjoy the bits in image.py and test_image.py | |
| 12:45:12 | mordred | Shrews: where the test was testing setting a property on the remote image called "properties" with the value '{"description": "This is not an image"}' | |
| 12:51:54 | Shrews | mordred: i don't yet know what the fundamental issue is, but i find it humorous that someone thought making the dict a string was the proper way to send the value | |
| 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 | |