Tuesday, 2017-06-06

« Monday, 2017-06-05 Index Wednesday, 2017-06-07 »
*** slaweq has quit IRC00:02
*** jamielennox is now known as jamielennox|away01:17
*** jamielennox|away is now known as jamielennox01:34
*** slaweq has joined #openstack-shade01:59
*** slaweq has quit IRC02:04
*** slaweq has joined #openstack-shade02:59
*** slaweq has quit IRC03:04
openstackgerritMonty Taylor proposed openstack-infra/shade master: Generalize version discovery for re-use  https://review.openstack.org/47077003:08
openstackgerritMonty Taylor proposed openstack-infra/shade master: Migrate dns to new discovery method  https://review.openstack.org/47099303:08
openstackgerritMonty Taylor proposed openstack-infra/shade master: First Pass: De-client-ify Users  https://review.openstack.org/46778403:08
*** gkadam has joined #openstack-shade03:34
openstackgerritMerged openstack-infra/shade master: Add pagination for the list_volumes call  https://review.openstack.org/46692703:45
openstackgerritMerged openstack-infra/shade master: Pass hints to Cinder scheduler in create_volume  https://review.openstack.org/47096803:55
*** slaweq has joined #openstack-shade04:00
*** slaweq has quit IRC04:05
*** gouthamr has quit IRC04:30
*** slaweq has joined #openstack-shade05:01
*** slaweq has quit IRC05:06
*** slaweq has joined #openstack-shade05:55
*** slaweq has quit IRC05:56
openstackgerritOpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/47044705:59
*** slaweq has joined #openstack-shade06:00
*** slaweq has quit IRC06:00
*** yfried has joined #openstack-shade07:00
*** ioggstream has joined #openstack-shade07:20
*** slaweq has joined #openstack-shade08:19
*** slaweq has quit IRC08:20
*** slaweq has joined #openstack-shade09:49
*** slaweq has quit IRC09:54
*** slaweq has joined #openstack-shade09:54
*** cdent has joined #openstack-shade09:57
*** slaweq has quit IRC09:59
*** slaweq has joined #openstack-shade10:07
*** slaweq has quit IRC10:07
*** jamielennox is now known as jamielennox|away10:10
*** jamielennox|away is now known as jamielennox10:16
*** purplerbot has quit IRC10:33
*** purplerbot has joined #openstack-shade10:33
*** yfried has quit IRC11:00
*** yfried has joined #openstack-shade11:05
*** gouthamr has joined #openstack-shade11:39
*** gouthamr has quit IRC12:39
*** gkadam has quit IRC12:49
*** gouthamr has joined #openstack-shade12:52
*** slaweq has joined #openstack-shade14:22
*** slaweq has quit IRC14:33
openstackgerritMerged openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/47044714:59
openstackgerritMerged openstack-infra/shade master: Handle ports with no 'created_at' attribute  https://review.openstack.org/46986914:59
*** yfried has quit IRC15:11
morganok.. back and looking at stuff15:41
morganshould have review eyes on the fix you did mordred  and prob another de-clientfy patch15:42
mordredmorgan: awesome! welcome back ...15:42
mordredmorgan: I have 3 patches I'm about to push up for you15:43
morganokie15:43
mordredmorgan: that should make the transition such way less15:43
mordredmorgan: (I maybe went on a vendetta to destroy the double-discovery calls that were happening, because they made doing the transition essentially impossible)15:44
morgannice15:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use shade discovery for keystone  https://review.openstack.org/47140415:45
openstackgerritMonty Taylor proposed openstack-infra/shade master: Avoid keystoneclient making yet another discovery call  https://review.openstack.org/47140515:45
morgannot impossible... just less fun.15:45
mordredmorgan: there ya go15:45
morgannot that any of this is "fun"15:45
mordredmorgan: with that, the discovery path for ksa vs. ksc usage should be identical for identity15:45
morgangood stuff, I'll look at it before I do further work15:45
mordredwhich means just translating a single call won't get borked15:45
* morgan drinks coffee after landing at 1am due to delays from SBA getting back to SEA15:46
morganalso a touch sunburnt from SoCal Beach weather :)15:47
mordredmorgan: yay sunburn!15:48
morganhey, vitimin deeee15:48
openstackgerritMonty Taylor proposed openstack-infra/shade master: First Pass: De-client-ify Users  https://review.openstack.org/46778415:48
morganit's important15:48
morganmy wife is a bit worse off (she napped on the beach for an hour or two)15:48
mordredmorgan: k. that's your patch rebased on the stack. there are a few more calls in it because I was poking at trying to get a set of calls done before fixing the discovery crap15:49
mordredmorgan: uhoh. napping on the beach can == pain15:49
morganmild burn15:49
mordredmorgan: I fell asleep on the beach when I was 15 and got VERY bad burn15:49
morganbut was nice out in santa Barbara15:49
morganmakes us wish for sunny socal weather and beaches and a pool after this weekend.15:50
morgananyway, ack on the extra calls15:51
mordredmorgan: it's ... some _lovely_ hackery I'm sure you're going to 'enjoy'15:54
morganso, more "fun" ahead...15:54
morganhttps://twitter.com/elonmusk/status/871915360819204097 <- Elon Musk joke of the day.15:55
morganoopse wrong channel.. doh15:55
morgansigh, copy/paste on phone not as reliable sometimes15:56
morganwith irccloud15:56
*** jamielennox is now known as jamielennox|away16:26
*** yolanda has left #openstack-shade16:52
morganmordred: +1 on all changed up to mine that you rebased16:58
morganmordred: all look solid16:58
mordredmorgan: woot! (also, there's so many places that make discovery calls)17:00
morganyes17:04
*** jamielennox|away is now known as jamielennox17:05
*** yfried has joined #openstack-shade17:08
*** yfried has quit IRC17:19
*** slaweq has joined #openstack-shade17:52
*** slaweq has quit IRC18:11
*** slaweq has joined #openstack-shade18:14
*** slaweq has quit IRC18:19
*** cdent has quit IRC18:31
openstackgerritMerged openstack-infra/shade master: Generalize version discovery for re-use  https://review.openstack.org/47077018:42
openstackgerritMerged openstack-infra/shade master: Migrate dns to new discovery method  https://review.openstack.org/47099318:42
openstackgerritMerged openstack-infra/shade master: Use shade discovery for keystone  https://review.openstack.org/47140418:42
openstackgerritMerged openstack-infra/shade master: Avoid keystoneclient making yet another discovery call  https://review.openstack.org/47140518:45
*** mordred has quit IRC18:47
*** mordred has joined #openstack-shade18:48
*** ioggstream has quit IRC19:02
*** gouthamr has quit IRC19:12
*** ioggstream has joined #openstack-shade19:45
morganmordred: nice.19:57
*** cdent has joined #openstack-shade19:58
mordredmorgan: I believe the remaining conversions should be MUCH easier to make19:58
mordredmorgan: I mean, other than the current state of it completey not working :)19:59
mordredmorgan: ah! neat - http://logs.openstack.org/84/467784/8/check/gate-shade-functional/f445e0a/console.html#_2017-06-06_16_39_57_18525920:03
mordredthe payload has a links section20:04
morganmordred: oh uhm20:14
mordredmorgan: patch coming ...20:14
morganoh i was looking at that20:14
mordredoh- or I can leave you to it too20:15
* mordred steps away from the patch20:15
morganseeingif you have a patch20:15
morgango for it20:15
morgani was just digging into it20:15
mordredmorgan: I think we should be able to add the links stuff to the test fixture first20:15
morganwell the .get is returning a unicode object we're trying to normalize?20:16
mordredit's actually (and rods is going to fix this for us)- if the adapter gets a dict with more than one key, it returns thewhole dict20:16
mordredso we're not getting a listof user dicts, we're getting a {'users':[], 'links':{}}20:17
morganoh20:17
morganooooooh20:17
mordredyah20:17
rodsyup working on it20:17
morgan*eye roll*20:17
morganokie20:17
morgancool20:17
morganotherwise this looks solid20:17
mordredmorgan: yah - early too-clever idea20:17
morganand yeah lets add the links into the fixture thing20:17
morgan*sigh*20:17
morganas much as i dislike that20:17
morganrods: i'll wait for your fix and we'll roll from there20:18
morganand i'll start building on that to make sure we don't hit the same *again* elsewhere20:18
mordredmorgan: yah - the general idea with rods patch is to just return the entire payload instead of stripping the top-level key20:18
morganyah wfm20:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: First Pass: De-client-ify Users  https://review.openstack.org/46778420:22
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add links to user list dict  https://review.openstack.org/47147420:22
mordredmorgan: ^^ thereyago20:22
morganah nice20:22
mordredmorgan: (I was mostly wanting to make sure I was handing you back a working patch after having mucked with it)20:23
morganhehe20:23
openstackgerritMorgan Fainberg proposed openstack-infra/shade master: De-client-ify User Ops  https://review.openstack.org/46778420:24
morgan^^20:24
morganfixed commit msg20:24
mordred\o/20:24
*** jamielennox has quit IRC20:34
*** jamielennox has joined #openstack-shade20:38
*** gouthamr has joined #openstack-shade20:40
*** ioggstream has quit IRC21:17
morganmordred: looks like we have a passing patchset now21:18
mordredmorgan: zomg21:34
mordredmorgan: btw - I'm working on a followup to my earlier patch to use auth.get_discovery instead of session.get('/') in the shade discovery call so that we'll use the ksa cache (that should remove one more excess discovery call for identity things)21:35
*** slaweq has joined #openstack-shade21:37
morgannice21:37
mordredmorgan: yah - I _think_ we'll actually be in fairly good shape overall so that when the ksa patches land and we update, we don't have to scramble to fix all of our tests because of mock differences21:38
mordredI mean, we'll have to do that a smidge maybe - but the logic as it applies to our tests should be close to consistent21:38
mordredwith the discovery spec21:39
*** gouthamr has quit IRC21:40
morgan++21:49
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use get_discovery from keystoneauth  https://review.openstack.org/47150122:05
mordredmorgan: ^^ voila!22:05
morgannoice22:15
*** slaweq has quit IRC22:31
*** gouthamr has joined #openstack-shade22:57
*** slaweq has joined #openstack-shade23:02
*** cdent has quit IRC23:03
*** slaweq has quit IRC23:07
*** gouthamr has quit IRC23:41
*** gouthamr has joined #openstack-shade23:44
*** gouthamr has quit IRC23:51
« Monday, 2017-06-05 Index Wednesday, 2017-06-07 »

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