Friday, 2017-03-24

*** jamielennox is now known as jamielennox|away01:39
*** jamielennox|away is now known as jamielennox01:48
*** ioggstream has joined #openstack-shade01:51
*** ioggstream has quit IRC02:26
*** larainema has quit IRC02:32
*** Matias has quit IRC03:44
*** Matias has joined #openstack-shade03:45
*** yfried has joined #openstack-shade06:10
*** yfried has quit IRC06:24
*** gouthamr has joined #openstack-shade06:44
*** gouthamr has quit IRC07:12
*** ioggstream has joined #openstack-shade08:49
*** yfried has joined #openstack-shade09:44
*** yfried has quit IRC09:48
*** cdent has joined #openstack-shade10:52
*** Matias has quit IRC11:27
*** Matias has joined #openstack-shade11:39
mordrednotmorgan, jamielennox: could one or both of you take a look at the email to the dev list with the subject "[openstack-dev] [os-client-config] get_session_endpoint() and Keystone admin endpoint"11:58
mordredI _think_ looking at the code that y'all have cleaned up the ksc code at this point and the "if identity ignore interface" workaround can go away. or?11:59
*** Matias has quit IRC12:00
*** Matias has joined #openstack-shade12:01
*** ioggstream has quit IRC12:59
*** cdent has quit IRC13:02
*** gouthamr has joined #openstack-shade13:08
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix old-style mocking of nova_client  https://review.openstack.org/44964514:23
openstackgerritMonty Taylor proposed openstack-infra/shade master: Migrate server console tests to requests_mock  https://review.openstack.org/44964614:23
openstackgerritMonty Taylor proposed openstack-infra/shade master: Migrate get_server_console to REST  https://review.openstack.org/44964714:23
mordredShrews: ^^ the first patch there is the bad version of mocking I mentioned earlier14:24
Shrewsmordred: so, using @mock.patch.object persists across all tests?14:25
Shrewsoh, no14:26
Shrewsthe with patch() does14:26
mordredShrews: it's actually the nova_client = Mock( thing14:27
Shrewswait... why the "with patch()" with the "nova_client = Mock()"?14:27
Shrewsthat's... weird14:27
mordredthe with patch() also _would_ be fine - but the direct assignment borks14:27
mordredShrews: yuah14:27
mordredI mean - it's just a thing we shouldn't have anyway14:27
Shrewsyeah14:27
mordredthat is all of the instances of that pattern that I can see though, so that's good14:28
*** cdent has joined #openstack-shade14:57
*** gouthamr has quit IRC16:23
*** gouthamr has joined #openstack-shade16:33
*** gouthamr has quit IRC17:01
*** gouthamr has joined #openstack-shade17:05
*** cdent has quit IRC18:15
*** gouthamr has quit IRC20:23
*** yolanda has quit IRC20:54
*** yolanda has joined #openstack-shade20:56
*** yolanda has quit IRC21:04
*** yolanda has joined #openstack-shade21:05
*** yolanda has quit IRC21:17
*** yolanda has joined #openstack-shade21:19
*** yolanda has quit IRC21:26
*** yolanda has joined #openstack-shade21:32
*** yolanda has quit IRC21:39
*** yolanda has joined #openstack-shade21:39
*** yolanda has quit IRC23:17

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