Tuesday, 2016-11-15

openstackgerritMerged openstack-infra/shade: Remove pointless and fragile unittest  https://review.openstack.org/39734500:54
openstackgerritMerged openstack-infra/shade: Be specific about protected being bool  https://review.openstack.org/39732301:31
*** abregman has joined #openstack-shade06:15
*** abregman has quit IRC08:17
*** abregman has joined #openstack-shade08:53
*** abregman has quit IRC09:04
*** abregman has joined #openstack-shade09:12
*** openstackgerrit has quit IRC09:18
*** openstackgerrit has joined #openstack-shade09:18
openstackgerritArie Bregman proposed openstack-infra/shade: Add compute usage support  https://review.openstack.org/39387312:50
*** abregman is now known as abregman|mtg13:06
*** abregman|mtg is now known as abregman13:23
*** yolanda has quit IRC13:39
openstackgerritMonty Taylor proposed openstack/oaktree: Expand on the nodepool API todo in the docs  https://review.openstack.org/39774713:44
openstackgerritMonty Taylor proposed openstack-infra/shade: Expose visibility on images  https://review.openstack.org/39775013:47
openstackgerritMonty Taylor proposed openstack-infra/shade: Collapse list/search for networks, users and keypairs  https://review.openstack.org/39720813:48
openstackgerritMonty Taylor proposed openstack-infra/shade: Normalize volumes  https://review.openstack.org/39695313:48
*** yolanda has joined #openstack-shade13:55
*** yolanda has quit IRC14:07
*** yolanda has joined #openstack-shade14:23
openstackgerritMerged openstack/oaktree: Expand on the nodepool API todo in the docs  https://review.openstack.org/39774714:48
*** gouthamr has joined #openstack-shade14:59
*** sigmavirus has joined #openstack-shade15:01
*** stevemar has joined #openstack-shade15:02
stevemarmordred: i'd like to help with oaktree, but realistically only after atlanta15:02
mordredstevemar: yay! happy to have you any time you've got time15:02
stevemarmordred: but mark me as interested -- "You Had My Curiosity But Now You Have My Attention"15:03
*** edtubill has joined #openstack-shade15:03
mordred\o/15:03
*** cdent has joined #openstack-shade15:04
*** sc68cal has joined #openstack-shade15:07
*** pabelanger has joined #openstack-shade15:12
*** abregman is now known as abregman|brb15:13
*** gouthamr has quit IRC15:16
openstackgerritMonty Taylor proposed openstack-infra/shade: Create and use a Adapter wrapper for REST in TaskManager  https://review.openstack.org/38420615:29
openstackgerritMonty Taylor proposed openstack-infra/shade: Have OpenStackHTTPError inherit from HTTPError  https://review.openstack.org/39013615:29
openstackgerritMonty Taylor proposed openstack-infra/shade: Use REST for listing images  https://review.openstack.org/38420815:29
Shrewsi wish i had time for oaktree15:33
cdentditto15:33
Shrewsugh, atlanta. when is that? feb?15:38
mordredyah - I think so15:38
pabelangerI joined to learn more about oaktree :)15:42
*** openstackgerrit has quit IRC15:48
*** openstackgerrit has joined #openstack-shade15:48
*** gouthamr has joined #openstack-shade15:51
*** jroll has joined #openstack-shade15:52
*** dtroyer has joined #openstack-shade15:53
* mordred waves to all the nice people15:57
Shrewswait, we have nice people here?15:58
*** gouthamr has quit IRC16:00
mordredShrews: ssssh16:01
* dtroyer looks around to see where they are hiding16:02
*** gouthamr has joined #openstack-shade16:07
*** gouthamr_ has joined #openstack-shade16:10
*** gouthamr_ has quit IRC16:11
*** gouthamr has quit IRC16:11
*** gouthamr_ has joined #openstack-shade16:12
openstackgerritSean M. Collins proposed openstack/oaktreemodel: [WIP] Add Neutron network resources  https://review.openstack.org/39780616:12
cdentso, if oaktree is all gRPC, that means that random curl user is unable to walk up and just use it, right? That's kind of a shame, but understandable.16:15
mordredcdent: yes - that is the tradeoff for sure16:16
* cdent weeps for the web that was16:17
openstackgerritSean M. Collins proposed openstack/oaktreemodel: [WIP] Add Neutron network resources  https://review.openstack.org/39780616:17
* cdent pours one out for his dead urls16:17
cdentetc16:17
mordredcdent: well, I mean, I say that - it's protobuf over http/2 - so in _theory_ one could still use curl16:17
mordredit would just be way more awkward16:17
cdentindeed16:17
cdentat this stage it seems a fair price to pay16:18
cdentbut it still makes me all maudlin16:18
mordredyah - and I think if we wanted ot accept json payloads instead of protobuf (or return them if requested) it wouldn't be the world's hardest thing to add later16:18
mordredcdent: in the go gRPC server impl there is a REST/gRPC gateway - we can't use it directly, but we could implement a similar construct16:19
mordred(I tried to come up with ways to use the go gateway with the python grpc service, but reality folded in on itself and I got lost in the nether reaches of time and space)16:19
cdentanybody mind if I add purplerbot in here? https://anticdent.org/purple-irc-bot.html16:20
cdent(mostly for the transclusion, not so much the loggin)16:20
mordredcdent: works for me!16:21
cdentrad16:21
*** purplerbot has joined #openstack-shade16:22
cdentboo16:22
cdent[t 1LuT]16:22
purplerbot<cdent> boo [2016-11-15 16:22:28] [n 1LuT]16:22
cdent\o/16:22
* mordred wave to purplerbot16:23
mordredsc68cal: added some comments inline on your WIP patch there16:27
*** abregman|brb is now known as abregman16:27
sc68calmordred: thanks.16:35
sc68calmordred: Do we want to paper over some of the oddities of the networking API? For example the whole subnet object, with the ipv6 attributes16:36
sc68caldo we want to break subnet into two subobjects, a v4 one and a v6 one?16:36
sc68caljust so we don't have this subnet object with tons of mutually exclusive fields16:36
mordredsc68cal: yah. that's a great question and where my brain had been leaning - the network/subnet split is TERRIBLE from an end user perspective16:37
mordredlike, you hand a network to create_server, but what you're really doing is attaching a vm to a subnet16:37
mordredand the qualities of what you can do with the vm networking-wise are all tied to the subnet - even though it's not the unit that's reported16:38
sc68cals/subnet/subnets/g16:38
mordred++16:38
mordredsc68cal: so - in short - yeah- same as we've invented our own api in shade for creating servers that totally hides much of the complexity in ips behind the scenes, doing that for networks seems like a good idea16:39
mordredsc68cal: I'll warn you - you may want to gouge your eyes out by the time you're done though16:39
*** abregman has quit IRC16:40
sc68calmordred: I'll start wearing a welding mask16:40
Shrewsthat's not sufficient enough16:41
*** calebb has quit IRC17:27
thingeemordred finally get around to https://review.openstack.org/#/c/384206/12 today ... catching up on the task manager code atm17:39
mordredthingee: it's some fun code!17:39
*** calebb has joined #openstack-shade17:46
* Shrews loves seeing more people in the shade codez17:50
*** edtubill has quit IRC17:59
*** edtubill has joined #openstack-shade19:16
*** cdent has quit IRC19:30
thingeemordred this seems to break the deprecated dreamhost profile https://review.openstack.org/#/c/384208/12 http://paste.openstack.org/show/589322/19:41
thingeedreamcompute is working though19:41
mordredthingee: oh! thanks! that's an excellent test case19:42
mordredthingee: there are evil evil things that happen in glanceclient to deal with urls with and without versions prepended19:42
mordredclearly we need to do more of them :)19:42
thingeeother clouds seem to be fine from my testing. I guess dreamhost's old cluster was already including a version?19:43
thingeein the endpoint19:43
thingeeI can't even see the endpoints. doing a cloud_config.get_endpoint('dreamhost') just gives None. I get None for all other clouds too.19:44
thingeeerrr19:44
thingeecloud_config.get_endpoint('image') ... where cloud_config was initialized with dreamhost profile.19:45
thingeemordred other than that though, everything else looks good to me19:46
mordredthingee: hrm. poo. my old dreamhost creds don't seem to work anymore. I should go check my accout19:50
mordredthingee: try doing "cloud=shade.openstack_cloud(cloud='dreamhost') ; print cloud.service_catalog" and it should print you the whole catalog for dreamhost (assuming that's the right cloud name)19:50
thingeemordred boom http://paste.openstack.org/show/589326/19:54
*** cdent has joined #openstack-shade19:55
*** edtubill has left #openstack-shade19:56
mordredthingee: cool. thanks! so - my comment in https://review.openstack.org/#/c/384208/12/shade/openstackcloud.py about figuring out discovery later ... it seems later may be now :)19:57
thingeemordred I'd be fine with a follow-up. Just as long as we have this fixed before a release.19:58
* thingee is not aware of who is actively using non-released builds though19:59
thingeejlk is it possible for us to get an openstack label in the github repo ansible/ansible-modules-core. Right now I've been filtering issues/pr with the cloud label and just vague 'openstack' search20:04
jlkThat could work. There is also tooling that sivel wrote to be made aware of things matching code paths.20:06
jlkone sec.20:06
jlkhttps://ansible.sivel.net/pr/byfile.html you can find open things there20:07
jlkby file20:07
jlkhttps://ansible.sivel.net has a link where you can sign up for notifications, by path I believe20:08
jlkI think the bot is what applies labels, https://github.com/ansible/ansibullbot  A PR to that might work.20:10
thingeethanks20:10
jlkperhaps https://github.com/ansible/ansibullbot/blob/master/lib/triagers/ansible_ansible_pulltriager.py#L5020:11
jlkthingee: confirmed, a PR to the bot is the best way to go about that.20:13
*** gouthamr_ has quit IRC20:14
mordredthingee: we can also add you in there to the list of people who should get pinged about openstack PRs20:26
mordredthingee: https://github.com/ansible/ansibullbot/blob/master/MAINTAINERS-EXTRAS.txt#L77 and https://github.com/ansible/ansibullbot/blob/master/MAINTAINERS-CORE.txt#L5320:27
thingeeShrews fyi https://github.com/ansible/ansible-modules-core/pull/5601#issuecomment-26074679420:48
mordredShrews, jlk, thingee: I wonder if we should try to get a GH PR bot here that announces openstack module PRs20:49
*** cdent has quit IRC21:06
mordredthingee: did dreamcompute fall off the internet?21:09
mordredthingee: nevermind - I seem to be having internet issues locally21:12
openstackgerritMonty Taylor proposed openstack-infra/shade: Use REST for listing images  https://review.openstack.org/38420821:30
mordredthingee: warning TERRIBLE HACK but may still be the best bet (/me cries) ^^21:31
jlkThere is a github to IRC integration available on github itself, I'm not sure if you can target it to specific paths though.21:56
*** gouthamr_ has joined #openstack-shade22:19
*** gouthamr_ has quit IRC22:26
openstackgerritMonty Taylor proposed openstack-infra/shade: Use REST for listing images  https://review.openstack.org/38420822:26
mordredjlk: looks like not :(23:02
mordredthingee: ^^ that version of the patch MAY actually do version discovery right and also fallback properly for glances configured with the version url23:02
mordredthingee: the version discovery is not tested in that patch ... I have another patch I need to ressurect to submit to do that23:18

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