Thursday, 2016-12-08

openstackgerritMerged openstack/os-client-config: Fix interface_key for identity clients  https://review.openstack.org/40828205:29
*** gouthamr has joined #openstack-shade06:16
*** abregman has joined #openstack-shade06:22
*** yfried has joined #openstack-shade06:34
*** gouthamr_ has joined #openstack-shade07:27
*** gouthamr has quit IRC07:29
*** gouthamr_ has quit IRC08:08
*** gouthamr has joined #openstack-shade08:33
*** gouthamr has quit IRC09:11
openstackgerritArie Bregman proposed openstack-infra/shade: Add support for limits  https://review.openstack.org/39523509:45
*** gouthamr has joined #openstack-shade10:14
*** gouthamr_ has joined #openstack-shade10:23
*** gouthamr has quit IRC10:25
*** gouthamr has joined #openstack-shade10:27
*** cdent has joined #openstack-shade10:30
*** gouthamr_ has quit IRC10:30
*** gouthamr has quit IRC11:55
*** yfried has quit IRC12:33
*** yfried has joined #openstack-shade13:04
*** gouthamr has joined #openstack-shade14:21
*** abregman has quit IRC14:26
*** yfried has quit IRC14:31
*** abregman has joined #openstack-shade14:49
*** abregman has quit IRC15:02
*** abregman has joined #openstack-shade15:08
*** abregman has quit IRC15:29
Shrewsmordred: found a problem with https://review.openstack.org/38519018:32
Shrewsi wonder if we should have a test to exercise that exception clause (which would have caught that)18:33
*** cdent has quit IRC19:20
*** cdent has joined #openstack-shade21:32
*** gouthamr has quit IRC22:15
*** yfried has joined #openstack-shade22:42
*** cdent has quit IRC22:43
mordredShrews: I'd say if you found a problem then yes, we should hav ea test23:10
Shrewsmordred: that was my passive aggressive way of asking for a test  :)23:24
*** yfried has quit IRC23:24
mordredShrews: it's response.status_code, fwiw23:24
ShrewsYeah23:24
mordredShrews: and yes - although I'm going to do the test ina follow up patch because it'll be _much_ easier with the requests_mock stuff later in the stack23:25
Shrewscool cool cool23:26
mordredShrews: oh - that's not true. I need to get swiftclient done before I can really convert that test properly. I'll do it the ugly way23:27
mordredShrews: actually- gross. I'm going to need to just add a todo to properly add a test for this23:32
mordredShrews: there are like 4 things I need to do first23:32
* mordred starts down the rabbit hole23:32
Shrewsmordred: don't sweat it for now23:34
openstackgerritMonty Taylor proposed openstack-infra/shade: Convert test_create_image_put_v2 to requests_mock  https://review.openstack.org/40767023:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Move image tests from caching to image test file  https://review.openstack.org/40766823:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Move image tasks to REST  https://review.openstack.org/38519023:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Remove caching config from test_image  https://review.openstack.org/40766923:38
mordredShrews: well, it's the next rabbit hole to go down anyway23:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Remove a few glance client mocks we missed  https://review.openstack.org/38519123:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Update image downloads to use direct REST  https://review.openstack.org/38519223:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Change image update to REST and remove glanceclient  https://review.openstack.org/38519323:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Remove glanceclient and warlock from shade  https://review.openstack.org/38519423:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Change register_uri to use the per-method calls  https://review.openstack.org/40776723:38
openstackgerritMonty Taylor proposed openstack-infra/shade: Plumb nat_destination through for ip_pool case  https://review.openstack.org/40757823:47

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