Thursday, 2017-06-08

*** jamielennox is now known as jamielennox|away02:04
*** jamielennox|away is now known as jamielennox02:15
*** slaweq has quit IRC02:53
*** gkadam has joined #openstack-shade03:42
*** slaweq has joined #openstack-shade04:25
*** slaweq has quit IRC04:38
*** jamielennox is now known as jamielennox|away05:12
*** jamielennox|away is now known as jamielennox05:19
*** jamielennox is now known as jamielennox|away05:56
*** yfried has joined #openstack-shade05:57
*** jamielennox|away is now known as jamielennox06:03
*** ioggstream has joined #openstack-shade06:10
*** gkadam is now known as gkadam-afk07:30
*** GheRivero has joined #openstack-shade08:09
*** GheRivero has left #openstack-shade08:15
*** gkadam-afk is now known as gkadam08:16
*** frickler_ is now known as frickler08:36
*** ioggstream has quit IRC09:02
*** ioggstream has joined #openstack-shade09:03
*** cdent has joined #openstack-shade09:54
noshankus@mordred - thanks - first thing it does it call ProjectList, which returns valid projects as expected, then runs task "running task NovaLimitsGet" and completes "ran task NovaLimitsGet" - there's no other significant output, bar "GET call to identity for http://XXX.XXX.XXX.XXX:5000/v2.0/tenants used request id..." The NovaLimitsGet doesn't seem to use http, as there is no REQ, RESP or RESP BODY for the call... just a Munch object as11:42
noshankusexpected.11:42
noshankusThe ProjectList does use curl http request to get the list of projects however...11:43
mordrednoshankus: oh, piddle. I've got a fix in master for getting the http calls for novaclient things - I should cut a release with that11:49
noshankus@mordred Haha! You're going to start dreading my messages :) No worries, I'll keep an eye out for the release...11:50
noshankusIt looks like nova has deprecated the proxy API calls and you'll have to get those values from netowrk API instead: https://developer.openstack.org/api-ref/compute/?expanded=show-the-detail-of-quota-detail11:54
noshankus@mordred Ooops! Looks like get_network_quotas() is the new way to access those!11:55
mordredoh yay!11:57
noshankusWhile floating ip does return in get_compute_quotas() and get_compute_limits(), they are only populated in get_network_quotas()....11:57
mordrednoshankus: at some point we talked about adding a get_quotas() call that would just get all of them11:57
noshankus@mordred that would be amazing. One stop shop for get_quotas(). I like it...11:58
mordredI'll put it on the list for sure11:59
noshankusDeadly, thanks mordred. Now I just have to find out where to find the number of _used_ floating ips :D11:59
*** ioggstream has quit IRC12:37
*** iogg has joined #openstack-shade12:37
*** jroll has left #openstack-shade12:49
*** gkadam has quit IRC12:50
*** iogg has quit IRC12:57
*** iogg has joined #openstack-shade12:57
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Don't remove top-container element for volume REST API calls  https://review.openstack.org/47226813:47
rodsmordred ^^ the patch was getting to big for just one commit, I'm splitting it to make it easier to review and manage13:53
rods*too13:53
*** yolanda has joined #openstack-shade14:28
mordredrods: ++14:33
*** GheRivero has joined #openstack-shade14:36
*** slaweq has joined #openstack-shade15:11
rodsmordred is this patch https://review.openstack.org/#/c/467784/10 that is waiting for the work on the Adapter to be unblocked?15:25
rodssince I'm splitting the work in several commits, I'll do my best to unblock it as soon as I can15:27
mordredrods: oh - no - actually, I can land that already and you can work on top of it - lemme go do that15:30
*** yfried has quit IRC15:37
mordredrods: your patch looks great - thanks!15:40
*** slaweq has quit IRC15:43
rodsmordred ty15:53
*** noshankus has quit IRC15:54
openstackgerritMerged openstack-infra/shade master: Add links to user list dict  https://review.openstack.org/47147416:07
*** gkadam has joined #openstack-shade16:12
openstackgerritOpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/47231016:15
openstackgerritMerged openstack-infra/shade master: Use get_discovery from keystoneauth  https://review.openstack.org/47150116:16
*** slaweq has joined #openstack-shade16:27
openstackgerritMerged openstack-infra/shade master: Don't remove top-container element for volume REST API calls  https://review.openstack.org/47226816:28
*** yfried has joined #openstack-shade16:51
*** cdent has quit IRC17:00
*** gkadam has quit IRC17:05
openstackgerritMerged openstack-infra/shade master: De-client-ify User Ops  https://review.openstack.org/46778417:12
*** yfried has quit IRC17:31
*** cdent has joined #openstack-shade17:33
*** iogg has quit IRC17:39
*** cdent has quit IRC17:39
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Remove unneeded calls to shade_exceptions  https://review.openstack.org/47236317:57
*** cdent has joined #openstack-shade18:53
*** cdent has quit IRC19:04
rodsmordred my understanding is that the shade.Adapter is only used by the raw client, is that correct? If so that work I'm doing can only be applied to those calls that have been already restified otherwise we won't be testing the change19:31
openstackgerritMerged openstack-infra/shade master: Remove unneeded calls to shade_exceptions  https://review.openstack.org/47236319:32
mordredrods: yes, that is correct. anything still using legacy_client does not the Adapter19:34
*** purplerbot has quit IRC20:36
*** purplerbot has joined #openstack-shade20:38
*** cdent has joined #openstack-shade20:51
*** slaweq has quit IRC20:52
*** slaweq has joined #openstack-shade20:53
*** slaweq has quit IRC20:58
rodsmordred are these lines still needed https://github.com/openstack-infra/shade/blob/master/shade/openstackcloud.py#L4645-L4647?21:04
mordredrods: yes - I believe so - they're sanity checking to make sure the cloud didn't do some crazy21:05
mordredrods: (we've seen much evil in th eworld)21:05
rodsok, thx21:06
*** rods has quit IRC21:11
*** rods has joined #openstack-shade21:30
*** yfried has joined #openstack-shade21:38
*** yfried has quit IRC21:49
*** slaweq has joined #openstack-shade21:54
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Don't remove top-container element for network REST API calls  https://review.openstack.org/47245321:55
*** slaweq has quit IRC21:59
*** cdent has quit IRC22:22
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Don't remove top-container element for network REST API calls  https://review.openstack.org/47245322:51
openstackgerritOpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/47231022:54
*** slaweq has joined #openstack-shade22:55
*** slaweq has quit IRC23:00
*** slaweq has joined #openstack-shade23:56
*** jamielennox is now known as jamielennox|away23:57

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