Wednesday, 2017-03-22

notmorganthis one is particularly slow because the responses are fairly complex00:00
notmorganit's going to be another absolutely beastly patch =/00:00
notmorganbut on the plus side it means all mocks of keystone_client will be gone00:00
notmorganand we can start going with pure rest00:01
mordrednotmorgan: yay!00:04
mordrednotmorgan: happily, the restification patches should be much smaller and easier00:04
openstackgerritMonty Taylor proposed openstack-infra/shade master: Get rid of magnumclient dependency  https://review.openstack.org/44757800:09
mordredit helps when you delete all of the lines ...00:09
notmorganmordred: https://review.openstack.org/#/c/439119/ going to push that through unless you think someone is going to complain00:10
notmorganit's been sitting for quite a while00:10
mordrednotmorgan: do it00:21
openstackgerritMerged openstack/os-client-config master: Add support for bailing on invalid service versions  https://review.openstack.org/43911901:22
*** gouthamr has joined #openstack-shade01:32
*** gouthamr has quit IRC02:45
*** gouthamr has joined #openstack-shade04:25
*** gouthamr has quit IRC05:02
*** yfried has joined #openstack-shade09:39
*** cdent has joined #openstack-shade09:43
openstackgerritMerged openstack-infra/shade master: attach_volume should always return a vol attachment.  https://review.openstack.org/43496411:45
openstackgerritMerged openstack-infra/shade master: delete_server: make sure we sleep a bit when waiting for server deletion  https://review.openstack.org/44821511:45
openstackgerritMerged openstack-infra/shade master: wait_for_server: ensure we sleep a bit when waiting for server  https://review.openstack.org/44822011:45
*** gouthamr has joined #openstack-shade13:21
*** yfried has quit IRC13:30
openstackgerritMerged openstack-infra/shade master: Cleanup some workarounds for old OCC versions  https://review.openstack.org/44754215:16
openstackgerritMerged openstack-infra/shade master: Get the ball rolling on magnumclient  https://review.openstack.org/44754315:21
openstackgerritMerged openstack-infra/shade master: Add normalization for cluster templates  https://review.openstack.org/44754415:22
openstackgerritMerged openstack-infra/shade master: RESTify cluster template tests  https://review.openstack.org/44754515:22
openstackgerritMerged openstack-infra/shade master: Convert magnum service to requests_mock  https://review.openstack.org/44754615:26
openstackgerritMerged openstack-infra/shade master: Get rid of magnumclient dependency  https://review.openstack.org/44757815:27
openstackgerritMerged openstack-infra/shade master: Use data when the request has a non-json content type  https://review.openstack.org/44758115:29
*** cdent has quit IRC17:27
*** cdent has joined #openstack-shade17:29
*** yfried has joined #openstack-shade19:57
openstackgerritMonty Taylor proposed openstack-infra/shade master: Accept device_id for update_port  https://review.openstack.org/44878120:38
mordredShrews, clarkb: if you have a sec, could you rubberstamp that ^^ ? it's a simple bugfix for an upstream ansible bug20:42
mordredI lifted it from a gh pr - but since it's a one-word change I am not worried about proper attribution/cla20:42
clarkbmordred: would it be terrible to update the docstring too?20:42
clarkblooks like the other valid kwargs have docstring entries20:43
mordredclarkb: nope - can do real quick20:43
clarkbcool thanks20:43
openstackgerritMonty Taylor proposed openstack-infra/shade master: Accept device_id for update_port  https://review.openstack.org/44878120:43
mordredwow. I knew the docstring answer without referring to upstream docs. that's scary20:44
notmorganmordred: thats...21:04
*** gouthamr has quit IRC21:26
*** yfried has quit IRC21:35
*** yfried has joined #openstack-shade21:36
*** cdent has quit IRC21:54
mordrednotmorgan: ikr?22:13
*** yfried has quit IRC22:32

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