Thursday, 2016-11-10

Shrewsmordred: i prefer the list over search, fwiw01:45
Shrewswill look at things in the morn01:45
*** noama has quit IRC05:24
*** noama has joined #openstack-shade05:25
*** abregman has joined #openstack-shade06:05
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-shade07:48
openstackgerritArie Bregman proposed openstack-infra/shade: Add support for limits  https://review.openstack.org/39523507:59
*** kamsz has quit IRC08:11
*** abregman_ has joined #openstack-shade08:26
*** abregman_ has quit IRC08:26
*** abregman has quit IRC08:29
*** abregman has joined #openstack-shade08:32
openstackgerritArie Bregman proposed openstack-infra/shade: Add compute usage support  https://review.openstack.org/39387308:44
*** abregman has quit IRC08:52
*** abregman has joined #openstack-shade09:08
openstackgerritArie Bregman proposed openstack-infra/shade: Add compute usage support  https://review.openstack.org/39387309:30
*** yolanda has quit IRC09:31
*** yolanda has joined #openstack-shade09:31
openstackgerritArie Bregman proposed openstack-infra/shade: Add support for limits  https://review.openstack.org/39523509:33
openstackgerritArie Bregman proposed openstack-infra/shade: Add support for limits  https://review.openstack.org/39523509:34
*** abregman is now known as abregman|afk10:47
*** abregman|afk is now known as abregman11:34
*** edtubill has joined #openstack-shade15:05
*** yolanda has quit IRC15:11
mordredShrews: 15:16:37 openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make search_projects a special case of list_projects  https://review.openstack.org/39586815:16
mordredShrews: this version should be good now - the previous version had some holes around jmsepath15:17
Shrewsai'ight15:18
*** yolanda has joined #openstack-shade15:26
*** abregman has quit IRC15:33
openstackgerritMonty Taylor proposed openstack-infra/shade: Normalize projects  https://review.openstack.org/39630916:13
mordredShrews: there's that ^16:14
Shrews*nod*16:14
thingeejlk I'm testing https://github.com/ansible/ansible-modules-core/pull/5503 today16:34
*** edtubill has quit IRC16:47
openstackgerritMonty Taylor proposed openstack-infra/shade: Only generate checksums if neither is given  https://review.openstack.org/39632017:14
*** edtubill has joined #openstack-shade17:16
openstackgerritMonty Taylor proposed openstack-infra/shade: Cache file checksums by filename and mtime  https://review.openstack.org/39632217:20
openstackgerritMonty Taylor proposed openstack-infra/shade: Fail up to date check on one out of sync value  https://review.openstack.org/39632817:28
openstackgerritMonty Taylor proposed openstack-infra/shade: Fail up to date check on one out of sync value  https://review.openstack.org/39632817:29
openstackgerritMonty Taylor proposed openstack-infra/shade: Fail up to date check on one out of sync value  https://review.openstack.org/39632817:34
openstackgerritMonty Taylor proposed openstack-infra/shade: Only generate checksums if neither is given  https://review.openstack.org/39632017:34
openstackgerritMonty Taylor proposed openstack-infra/shade: Cache file checksums by filename and mtime  https://review.openstack.org/39632217:34
Shrewsmordred: any idea for a plan to mark APIs as deprecated? (e.g., search*)17:44
mordredShrews: not really - I mean, the docstrings will fall away - but I'm not sure actual deprecation marking gets us anywhere?17:45
clarkbyou could use warning?17:45
mordredclarkb: we could - but what would the warning be warning against?17:45
clarkbthough that gets irritating when its libs generating them17:46
mordredyah17:46
clarkbI think if you wanted to remove the deprecated things in shade 2.0 you'd want warnigns17:46
clarkbif you want ot just make them passthrough then maybe docstring is good enough17:46
mordredI think passthrough sounds more like the right way forward17:47
clarkbmordred: lets also plan for nodepool restart later today after image uploads to pick up the shade release17:48
mordredclarkb: agree17:48
mordredclarkb: do we want to land those patches above and release again before that?17:48
clarkbmordred: I don't think its necessary since it won't actually speed anything up for our use of it (since we are doing the dib side hash generation of both hashes and we'd have to change that instead)17:50
mordredclarkb: good point17:50
clarkbbut it also won't hurt I don't think17:50
mordrednah - no need to rush17:50
Shrewsmordred: comments on 39630917:50
mordredShrews: thanks on create/update- for human/name - those are nova-specific attributes - I do not believe keystone returns the17:51
mordredthem17:51
Shrewsmordred: but they do!17:51
Shrewsmordred: i tested :)17:51
mordredreally? SIGH17:52
Shrewswould i lie?17:52
mordredShrews: yup17:52
Shrewsmordred: http://paste.openstack.org/show/588788/17:53
Shrewsi also like that they're HUMAN_ID is a boolean. that makes total sense17:53
Shrewss/they're/their/17:54
mordredShrews: whatcha wanna bet HUMAN_ID=False means there is no value in human_id17:54
Shrewsmordred: ah, yeah, maybe17:55
clarkbwould match up with the None there17:55
Shrewsmordred: we need another field to tell us if HUMAN_ID is True or False17:55
openstackgerritMonty Taylor proposed openstack-infra/shade: Normalize projects  https://review.openstack.org/39630917:55
mordredShrews: or we just need to pop it and be done with it17:55
Shrewsok, done shade reviewing for the day. time to test my own crappy code17:59
jlkthingee: thanks!17:59
mordredShrews: woot! thank you18:00
Shrewsmordred: oh, btw, not sure why https://review.openstack.org/395867 failed for you. but didn't +A the list b/c of it18:01
mordredShrews: it looks like a devstack timeout18:03
mordredShrews: so just a slow node, best I can tell18:03
openstackgerritMonty Taylor proposed openstack-infra/shade: Fail up to date check on one out of sync value  https://review.openstack.org/39632818:33
openstackgerritMerged openstack-infra/shade: Make a private method more privater  https://review.openstack.org/39586719:44
openstackgerritMerged openstack-infra/shade: Make search_projects a special case of list_projects  https://review.openstack.org/39586819:44
openstackgerritMerged openstack-infra/shade: Normalize projects  https://review.openstack.org/39630919:44
*** openstackgerrit has quit IRC19:48
*** openstackgerrit has joined #openstack-shade19:48
openstackgerritMerged openstack-infra/shade: Only generate checksums if neither is given  https://review.openstack.org/39632020:27
*** edtubill has quit IRC20:41
thingeercarrillocruz how can I test out https://github.com/ansible/ansible-modules-core/pull/4261 ... aside from the issue Shrews brought up22:41

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