Friday, 2017-05-12

*** gouthamr has quit IRC00:50
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add "Multi Cloud with Shade" presentation  https://review.openstack.org/46333001:22
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add super basic machine normalization  https://review.openstack.org/46410101:22
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add 'public' as a default interface for get_mock_url  https://review.openstack.org/46410201:22
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert first ironic_client test to REST  https://review.openstack.org/46410301:22
*** larainema has quit IRC02:01
*** gouthamr has joined #openstack-shade02:58
*** gouthamr has quit IRC03:22
*** gouthamr has joined #openstack-shade03:27
*** gouthamr has quit IRC04:16
*** larainema has joined #openstack-shade04:22
*** yfried has joined #openstack-shade04:47
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add super basic machine normalization  https://review.openstack.org/46410104:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert first ironic_client test to REST  https://review.openstack.org/46410304:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add 'public' as a default interface for get_mock_url  https://review.openstack.org/46410204:54
*** yfried has quit IRC04:58
*** yfried has joined #openstack-shade05:38
*** yfried has quit IRC05:43
*** yfried has joined #openstack-shade06:10
*** yfried has quit IRC06:15
*** yfried has joined #openstack-shade06:18
*** yfried has quit IRC06:25
*** jamielennox is now known as jamielennox|away06:53
*** rods has quit IRC09:43
*** ioggstream has joined #openstack-shade09:45
*** jamielennox|away is now known as jamielennox12:30
*** gkadam has joined #openstack-shade13:24
*** pabelanger has quit IRC13:36
*** pabelanger has joined #openstack-shade13:36
-openstackstatus- NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks.13:38
*** rods_ has joined #openstack-shade13:44
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Add a deprecation warning for the cinderclient  https://review.openstack.org/46379513:44
*** rods_ has quit IRC14:05
*** gkadam has quit IRC14:15
*** rods_ has joined #openstack-shade14:20
*** yfried has joined #openstack-shade14:21
*** yfried has quit IRC14:26
mordredrods_: gah. I totally gave you bad review yesterday. sorry (I was very tired)14:29
rods_mordred np :)14:32
rods_mordred what's next step? which other client needs to be updated?14:33
rods_nova?14:33
mordredrods_: actually - I just looked at it and made a littlehelper function - one sec ...14:44
* mordred should have done this a while ago - whoops14:46
openstackgerritMonty Taylor proposed openstack-infra/shade master: Make deprecated client helper method  https://review.openstack.org/46421314:48
mordredrods_: next step past cinderclient - ironicclient and novaclient are the last two - morgan and samueldmq are working on keystone_client (and I've gone down a very long rabbit hole related to endpoint/version discovery related to that)14:49
mordredrods_: there's the patch to add a little helper method to do the deprecation properly (I also reorganized the constructors a little bit)14:49
rods_mordred I can take the nova client14:49
mordredwoot! I'm starting to bang on the ironic_client a bit - I found some very gross things in that one last night14:50
mordredbut I've been writing specs/documents for the last 2 weeks and I think doing a little hacking on removing ironic_client will be nice :)14:51
rods_:)14:53
rods_mordred I think I can abandon my changeset, right?14:54
*** rods_ has quit IRC14:56
*** gouthamr has joined #openstack-shade15:19
*** rods has joined #openstack-shade15:46
*** cdent has joined #openstack-shade15:49
*** gouthamr has quit IRC15:57
*** cdent has quit IRC15:58
mordredrods: I'm rebasing your patch on top of mine - patch coming16:00
*** gouthamr has joined #openstack-shade16:15
*** slaweq has joined #openstack-shade16:43
*** rods_ has joined #openstack-shade16:49
*** rods_ has quit IRC16:50
mordredShrews: I just made a patch to do a slight cleanup on legacy clients - and in so doing discovered that something had been causing pep8 to fail open (wat?) - so there is also a pep8 cleanup patch :)16:56
mordredShrews: also - did you enjoy the quiet with folks at the summit?17:00
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add helper method to fetch service catalog  https://review.openstack.org/46423817:06
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add ironicclient to constructors list  https://review.openstack.org/46423917:06
*** slaweq has quit IRC17:07
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove cinder client  https://review.openstack.org/46379517:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Make deprecated client helper method  https://review.openstack.org/46421317:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix pep8 errors that were lurking  https://review.openstack.org/46424117:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Move legacy client constructors to mixin  https://review.openstack.org/46424217:18
mordredrods: ^^ I updated your patch in the middle of that17:19
*** slaweq has joined #openstack-shade17:21
mordredjlk: had an idea while writing slides for talks this week - which I think might 'fix' some annoyances either you or other people have had in the past about consuming clouds.yaml info in os_ ansible modules17:22
* jlk listens with his brain meat17:23
mordredjlk: the idea basically is - make the "cloud" parameter able to take a dictionary that would be the same as the contents of the entry in clouds.yaml17:23
mordredjlk: so - if you prefer to put that info into your ansible variables, you could just say "cloud: {{ cloud_dict }}" and it would do the right thing17:23
mordredjlk: do you think that would be more better or more worser?17:24
jlkit'd be different. Right now you can fill all those things into auth: block, right? You just couldn't say auth: <here's a dict>17:24
mordredjlk: you can't - there's a ton of settings that don't go into dict that you currently _must_ put into clouds.yaml if you want to use them17:25
mordredjlk: for instance, image_api_verison or compute_endpoint_override17:25
mordredso if you have a complex config, you must put it in clouds.yaml and refer to it by name with the cloud: param. but if you have a simple one - you can just put it in auth: and not provide a cloud: param at all17:26
jlknod17:27
jlkbeing able to feed in a pre-built dict could be useful.17:27
mordredbut the concern I have is that explaining things might be too complex and confuse people (even though it's just an advanced feature that most people likely should ignore)17:27
mordredkk17:27
mordredI'll take a stab at writing that patch and see what it looks like17:27
rodsmordred ok, thx17:33
*** slaweq has quit IRC17:44
*** slaweq has joined #openstack-shade17:44
*** slaweq has quit IRC17:49
*** ioggstream has quit IRC17:50
*** slaweq has joined #openstack-shade19:35
*** gouthamr has quit IRC19:50
*** morgan_ has joined #openstack-shade20:19
*** morgan_ has quit IRC20:20
*** yolanda has quit IRC20:35
*** yolanda has joined #openstack-shade20:43
*** slaweq has quit IRC20:50
*** slaweq has joined #openstack-shade20:50
*** slaweq has quit IRC22:46
*** slaweq has joined #openstack-shade22:46
*** slaweq has quit IRC22:51
*** slaweq has joined #openstack-shade23:58

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