Thursday, 2018-08-09

*** bobh has joined #openstack-sdks00:04
*** bobh has quit IRC00:08
*** bobh has joined #openstack-sdks00:09
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer  https://review.openstack.org/53083500:09
mordredShrews: I found new terrible things I can't wait to tell you about00:09
mordredadriant: 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 it00:10
adriantmordred: 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 earliest00:11
mordredadriant: kk. no rush00:11
adriantit's just the changing of the Resource internals to make it dict based right?00:11
mordredadriant: yah- and renaming two of the methods - but most things shoudl be touching those Resource bits via proxy methods00:11
adriantYeah, none of my code goes that deep into the Resource class00:12
mordredwoot. then you should be fine00:12
adriantMost of the stuff that I always ran into was the session stuff you fixed :)00:12
*** bobh has quit IRC00:13
adriantmordred: oh and https://review.openstack.org/#/c/544544/ totally forgot about that patch, but the person who proposed it never responded00:14
samueldmqmordred: Shrews: I liked having openstack cloud and operator cloud as separate things. having that boundary made sense from user's perspective00:45
samueldmqfor 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|g0ne01:37
*** chenyb4 has joined #openstack-sdks01:45
*** mriedem has quit IRC02:08
*** bobh has joined #openstack-sdks02:53
*** bobh has quit IRC02:55
*** bobh has joined #openstack-sdks02:56
*** bobh has quit IRC02:57
openstackgerritRui Chen proposed openstack/openstacksdk master: Add more options to enable ansible testing feature  https://review.openstack.org/59007803:02
*** bobh_ has joined #openstack-sdks03:08
*** bobh_ has quit IRC03:11
*** bobh_ has joined #openstack-sdks03:44
*** bobh_ has quit IRC04:01
*** e0ne has joined #openstack-sdks04:11
*** e0ne has quit IRC04:24
*** gkadam has joined #openstack-sdks04:28
*** markvoelker has joined #openstack-sdks04:41
*** olaph has quit IRC05:44
*** gtema has quit IRC07:16
*** jpena|off is now known as jpena07:50
*** tosky has joined #openstack-sdks07:52
*** gildub has quit IRC07:57
*** gtema has joined #openstack-sdks08:16
*** jpich has joined #openstack-sdks08:19
*** dtantsur|afk is now known as dtantsur08:33
openstackgerritYang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota  https://review.openstack.org/59019308:56
*** cdent has joined #openstack-sdks09:08
openstackgerritLajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value  https://review.openstack.org/58816809:17
openstackgerritLajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value  https://review.openstack.org/58816810:02
*** gkadam has quit IRC10:13
*** gkadam has joined #openstack-sdks10:13
gtemadoes 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 elsewhere10:28
openstackgerritYang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota  https://review.openstack.org/59019310:36
openstackgerritYang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota  https://review.openstack.org/59019310:40
*** gildub has joined #openstack-sdks10:43
*** jpena is now known as jpena|lunch11:01
*** chenyb4 has quit IRC11:21
*** bobh_ has joined #openstack-sdks11:31
*** thrash|g0ne is now known as thrash11:47
*** jpena|lunch is now known as jpena11:58
*** olaph has joined #openstack-sdks12:04
openstackgerritLajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value  https://review.openstack.org/58816812:13
dtroyergtema: check your version of cmd2 (0.8.8) and cliff (2.13.0).  Those are the versions I have in my development venv12:25
gtemadtroyer: If I do a fresh 'pip install python-openstackclient' I get cmd2-0.9.3 and cliff-2.13.012:27
gtemadtroyer, if I manually downgrade cmd2 to 0.8.8 it works, so probably freeze the version if the fix is not fast?12:29
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict  https://review.openstack.org/58919312:36
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource  https://review.openstack.org/58919412:36
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks  https://review.openstack.org/58919512:36
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer  https://review.openstack.org/53083512:36
Shrewsmordred: i'm giddy with anticipation12:42
dtroyergtema: cmd2 has been a barrel of fun lately…  we do have it capped at <0.9.0 for python 2, python3 is fine above that12:43
gtemadtroyer: I'm having this issue with python 3.612:43
mordredShrews: in https://review.openstack.org/530835 - please to enjoy the bits in image.py and test_image.py12:44
mordredShrews: 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-sdks12:45
Shrewsmordred: 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 value12:51
mordredShrews: I thought you'd enjoy that12:53
Shrewsor maybe it's sad that that is what the api actually expects??12:54
Shrewsi put nothing past openstack apis anymore12:54
mordrednope. 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
mordredShrews: so the sdk in this case is actually not doing that - and is not actually supporting dealing with the extra key-value pairs12:55
Shrewsso glance broke their own api by removing properties?12:56
Shrewsi mean, i guess you can do that moving from v1 to v2... but that seems silly12:57
Shrewsmeh12:57
gtemadtroyer: even cmd2==0.9.0.1 has a problem already12:57
mordredShrews: 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 it12:58
mordredShrews: but yeah12:58
Shrewsah12:59
mordredShrews: 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 happy13:02
*** gildub has quit IRC13:11
*** mriedem has joined #openstack-sdks13:24
*** edmondsw has quit IRC13:29
*** edmondsw has joined #openstack-sdks13:36
*** edmondsw has quit IRC13:41
*** bobh_ has quit IRC13:44
*** edmondsw has joined #openstack-sdks13:51
dtroyergtema: hmmm, ok, maybe py3 should be capped too…13:51
*** amotoki_ is now known as amotoki14:01
mordredShrews: 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 work14:25
*** gkadam_ has joined #openstack-sdks14:26
*** gkadam has quit IRC14:30
mordred(we'll also need to cut a release with that in it)14:40
Shrews+Ad14:40
mordredyay14:42
*** dtantsur is now known as dtantsur|brb14:43
elmikocdent edleafe dtantsur|brb heads up, i have a conflict at the start of the meeting, will most likely be late14:50
edleafeelmiko: good, we'll have some time to gossip about you14:53
elmiko+14:55
elmiko++ even14:55
elmikoi might be signed up for a phone call after the conflicting meeting, no guarantees that i will make our meeting15:07
cdentsomething something sparkle motion15:16
*** dtantsur|brb is now known as dtantsur15:29
* elmiko chuckles15:31
openstackgerritMerged openstack/openstacksdk master: Update config doc url to point to openstacksdk  https://review.openstack.org/58672615:52
openstackgerritMerged openstack/openstacksdk master: Handle image and object key metadata for shade transition  https://review.openstack.org/58672715:58
*** gtema has quit IRC16:02
*** jpena is now known as jpena|off16:07
*** jpich has quit IRC16:08
cdentedleafe: you wanna go ahead merge https://review.openstack.org/#/c/589130/ as it's just a naming thing?16:29
cdentedleafe, dtantsur, elmiko : In fact I'm not sure _any_ of that stack warrants full guideline-style review16:30
cdentit's basically reformatting and linking16:30
* dtantsur neither16:30
cdentthe last one might be a guideline16:30
cdentbut not really16:31
edleafeAgreed that they don't need the whole freeze process16:31
edleafeBut haven't looked them over yet in any detail.16:31
cdentk, will not list them as new guidelines just refer to them in thebody of the message16:31
*** gkadam_ has quit IRC16:32
edleafeI +W'd https://review.openstack.org/#/c/589130/116:32
*** dtruong has joined #openstack-sdks16:32
cdentedleafe, dtantsur, elmiko : I don't know what else to say: https://etherpad.openstack.org/p/api-sig-newsletter16:38
*** tosky has quit IRC16:39
openstackgerritMerged openstack/api-sig master: Update web frontpage to link to SIG not WG  https://review.openstack.org/58913016:40
edleafecdent: Dunno, you could have added some jokes or something16:41
edleafeBut if you want to be boring and dull, LGTM16:41
cdentan api walks into a bar16:42
cdentbartender says16:42
cdentwhat the foo are you doing here16:42
* cdent takes his leave16:42
dimsLOL16:46
elmikosorry gang, meeting is long16:56
cdentelmiko: look what you made me do16:57
elmikoi love that joke though16:58
edleafeelmiko: you should read the minutes. We totally trashed you17:00
cdent"love"?17:03
elmikohaha17:10
*** cdent has quit IRC17:29
*** dtantsur is now known as dtantsur|afk18:24
openstackgerritMerged openstack/openstacksdk master: Rename Resource get and update to not clash with dict  https://review.openstack.org/58919318:40
*** olaph has quit IRC18:54
mordredboom18:56
mordreddtroyer: we're just about half done with the 2 items I wanted to get done before a 1.018:57
mordreddtroyer: 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 it18:58
*** mriedem has quit IRC19:11
*** mriedem has joined #openstack-sdks19:11
dtroyermordred: I like these words that I read…19:51
mordreddtroyer: \o/19:56
mordreddtroyer: 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 in19:57
samueldmqheya, 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 IRC20:52
*** slaweq has quit IRC21:26
*** slaweq has joined #openstack-sdks22:11
*** imacdonn has quit IRC22:12
*** imacdonn has joined #openstack-sdks22:12
*** tobasco is now known as tobias-urdin22:14
*** slaweq has quit IRC22:15
*** tosky has joined #openstack-sdks22:30
openstackgerritMerged openstack/openstacksdk master: Add computed attribute type and location to base resource  https://review.openstack.org/58919422:46
*** slaweq has joined #openstack-sdks23:11
*** gildub has joined #openstack-sdks23:11
*** slaweq has quit IRC23:16
*** tosky has quit IRC23:48
*** ssbarnea has joined #openstack-sdks23:54

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!