*** bobh has joined #openstack-sdks | 00:04 | |
*** bobh has quit IRC | 00:08 | |
*** bobh has joined #openstack-sdks | 00:09 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | 00:09 |
---|---|---|
mordred | Shrews: I found new terrible things I can't wait to tell you about | 00:09 |
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:10 |
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 |
mordred | adriant: kk. no rush | 00:11 |
adriant | it's just the changing of the Resource internals to make it dict based right? | 00:11 |
mordred | adriant: yah- and renaming two of the methods - but most things shoudl be touching those Resource bits via proxy methods | 00:11 |
adriant | Yeah, none of my code goes that deep into the Resource class | 00:12 |
mordred | woot. then you should be fine | 00:12 |
adriant | Most of the stuff that I always ran into was the session stuff you fixed :) | 00:12 |
*** bobh has quit IRC | 00: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:14 |
samueldmq | mordred: Shrews: I liked having openstack cloud and operator cloud as separate things. having that boundary made sense from user's perspective | 00:45 |
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) | 00:48 |
*** thrash is now known as thrash|g0ne | 01:37 | |
*** chenyb4 has joined #openstack-sdks | 01:45 | |
*** mriedem has quit IRC | 02:08 | |
*** bobh has joined #openstack-sdks | 02:53 | |
*** bobh has quit IRC | 02:55 | |
*** bobh has joined #openstack-sdks | 02:56 | |
*** bobh has quit IRC | 02:57 | |
openstackgerrit | Rui Chen proposed openstack/openstacksdk master: Add more options to enable ansible testing feature https://review.openstack.org/590078 | 03:02 |
*** bobh_ has joined #openstack-sdks | 03:08 | |
*** bobh_ has quit IRC | 03:11 | |
*** bobh_ has joined #openstack-sdks | 03:44 | |
*** bobh_ has quit IRC | 04:01 | |
*** e0ne has joined #openstack-sdks | 04:11 | |
*** e0ne has quit IRC | 04:24 | |
*** gkadam has joined #openstack-sdks | 04:28 | |
*** markvoelker has joined #openstack-sdks | 04:41 | |
*** olaph has quit IRC | 05:44 | |
*** gtema has quit IRC | 07:16 | |
*** jpena|off is now known as jpena | 07:50 | |
*** tosky has joined #openstack-sdks | 07:52 | |
*** gildub has quit IRC | 07:57 | |
*** gtema has joined #openstack-sdks | 08:16 | |
*** jpich has joined #openstack-sdks | 08:19 | |
*** dtantsur|afk is now known as dtantsur | 08:33 | |
openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | 08:56 |
*** cdent has joined #openstack-sdks | 09:08 | |
openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | 09:17 |
openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | 10:02 |
*** gkadam has quit IRC | 10:13 | |
*** gkadam has joined #openstack-sdks | 10:13 | |
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:28 |
openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | 10:36 |
openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | 10:40 |
*** gildub has joined #openstack-sdks | 10:43 | |
*** jpena is now known as jpena|lunch | 11:01 | |
*** chenyb4 has quit IRC | 11:21 | |
*** bobh_ has joined #openstack-sdks | 11:31 | |
*** thrash|g0ne is now known as thrash | 11:47 | |
*** jpena|lunch is now known as jpena | 11:58 | |
*** olaph has joined #openstack-sdks | 12:04 | |
openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | 12:13 |
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:25 |
gtema | dtroyer: If I do a fresh 'pip install python-openstackclient' I get cmd2-0.9.3 and cliff-2.13.0 | 12:27 |
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:29 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | 12:36 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | 12:36 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | 12:36 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | 12:36 |
Shrews | mordred: i'm giddy with anticipation | 12:42 |
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 |
gtema | dtroyer: I'm having this issue with python 3.6 | 12:43 |
mordred | Shrews: in https://review.openstack.org/530835 - please to enjoy the bits in image.py and test_image.py | 12:44 |
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:45 |
*** edmondsw has joined #openstack-sdks | 12:45 | |
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:51 |
mordred | Shrews: I thought you'd enjoy that | 12:53 |
Shrews | or maybe it's sad that that is what the api actually expects?? | 12:54 |
Shrews | i put nothing past openstack apis anymore | 12:54 |
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:54 |
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:55 |
Shrews | so glance broke their own api by removing properties? | 12:56 |
Shrews | i mean, i guess you can do that moving from v1 to v2... but that seems silly | 12:57 |
Shrews | meh | 12:57 |
gtema | dtroyer: even cmd2==0.9.0.1 has a problem already | 12:57 |
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 |
mordred | Shrews: but yeah | 12:58 |
Shrews | ah | 12:59 |
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:02 |
*** gildub has quit IRC | 13:11 | |
*** mriedem has joined #openstack-sdks | 13:24 | |
*** edmondsw has quit IRC | 13:29 | |
*** edmondsw has joined #openstack-sdks | 13:36 | |
*** edmondsw has quit IRC | 13:41 | |
*** bobh_ has quit IRC | 13:44 | |
*** edmondsw has joined #openstack-sdks | 13:51 | |
dtroyer | gtema: hmmm, ok, maybe py3 should be capped too… | 13:51 |
*** amotoki_ is now known as amotoki | 14:01 | |
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:25 |
*** gkadam_ has joined #openstack-sdks | 14:26 | |
*** gkadam has quit IRC | 14:30 | |
mordred | (we'll also need to cut a release with that in it) | 14:40 |
Shrews | +Ad | 14:40 |
mordred | yay | 14:42 |
*** dtantsur is now known as dtantsur|brb | 14:43 | |
elmiko | cdent edleafe dtantsur|brb heads up, i have a conflict at the start of the meeting, will most likely be late | 14:50 |
edleafe | elmiko: good, we'll have some time to gossip about you | 14:53 |
elmiko | + | 14:55 |
elmiko | ++ even | 14:55 |
elmiko | i might be signed up for a phone call after the conflicting meeting, no guarantees that i will make our meeting | 15:07 |
cdent | something something sparkle motion | 15:16 |
*** dtantsur|brb is now known as dtantsur | 15:29 | |
* elmiko chuckles | 15:31 | |
openstackgerrit | Merged openstack/openstacksdk master: Update config doc url to point to openstacksdk https://review.openstack.org/586726 | 15:52 |
openstackgerrit | Merged openstack/openstacksdk master: Handle image and object key metadata for shade transition https://review.openstack.org/586727 | 15:58 |
*** gtema has quit IRC | 16:02 | |
*** jpena is now known as jpena|off | 16:07 | |
*** jpich has quit IRC | 16:08 | |
cdent | edleafe: you wanna go ahead merge https://review.openstack.org/#/c/589130/ as it's just a naming thing? | 16:29 |
cdent | edleafe, dtantsur, elmiko : In fact I'm not sure _any_ of that stack warrants full guideline-style review | 16:30 |
cdent | it's basically reformatting and linking | 16:30 |
* dtantsur neither | 16:30 | |
cdent | the last one might be a guideline | 16:30 |
cdent | but not really | 16:31 |
edleafe | Agreed that they don't need the whole freeze process | 16:31 |
edleafe | But haven't looked them over yet in any detail. | 16:31 |
cdent | k, will not list them as new guidelines just refer to them in thebody of the message | 16:31 |
*** gkadam_ has quit IRC | 16:32 | |
edleafe | I +W'd https://review.openstack.org/#/c/589130/1 | 16:32 |
*** dtruong has joined #openstack-sdks | 16:32 | |
cdent | edleafe, dtantsur, elmiko : I don't know what else to say: https://etherpad.openstack.org/p/api-sig-newsletter | 16:38 |
*** tosky has quit IRC | 16:39 | |
openstackgerrit | Merged openstack/api-sig master: Update web frontpage to link to SIG not WG https://review.openstack.org/589130 | 16:40 |
edleafe | cdent: Dunno, you could have added some jokes or something | 16:41 |
edleafe | But if you want to be boring and dull, LGTM | 16:41 |
cdent | an api walks into a bar | 16:42 |
cdent | bartender says | 16:42 |
cdent | what the foo are you doing here | 16:42 |
* cdent takes his leave | 16:42 | |
dims | LOL | 16:46 |
elmiko | sorry gang, meeting is long | 16:56 |
cdent | elmiko: look what you made me do | 16:57 |
elmiko | i love that joke though | 16:58 |
edleafe | elmiko: you should read the minutes. We totally trashed you | 17:00 |
cdent | "love"? | 17:03 |
elmiko | haha | 17:10 |
*** cdent has quit IRC | 17:29 | |
*** dtantsur is now known as dtantsur|afk | 18:24 | |
openstackgerrit | Merged openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | 18:40 |
*** olaph has quit IRC | 18:54 | |
mordred | boom | 18:56 |
mordred | dtroyer: we're just about half done with the 2 items I wanted to get done before a 1.0 | 18:57 |
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 | 18:58 |
*** mriedem has quit IRC | 19:11 | |
*** mriedem has joined #openstack-sdks | 19:11 | |
dtroyer | mordred: I like these words that I read… | 19:51 |
mordred | dtroyer: \o/ | 19:56 |
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 | 19:57 |
samueldmq | heya, do https://storyboard.openstack.org/#!/story/2003401 and https://storyboard.openstack.org/#!/story/2003402 make sense at all to you? | 20:10 |
*** ssbarnea has quit IRC | 20:52 | |
*** slaweq has quit IRC | 21:26 | |
*** slaweq has joined #openstack-sdks | 22:11 | |
*** imacdonn has quit IRC | 22:12 | |
*** imacdonn has joined #openstack-sdks | 22:12 | |
*** tobasco is now known as tobias-urdin | 22:14 | |
*** slaweq has quit IRC | 22:15 | |
*** tosky has joined #openstack-sdks | 22:30 | |
openstackgerrit | Merged openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | 22:46 |
*** slaweq has joined #openstack-sdks | 23:11 | |
*** gildub has joined #openstack-sdks | 23:11 | |
*** slaweq has quit IRC | 23:16 | |
*** tosky has quit IRC | 23:48 | |
*** ssbarnea has joined #openstack-sdks | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!