<div class="gmail_quote">On Fri Dec 12 2014 at 1:06:08 PM Tripp, Travis S <<a href="mailto:travis.tripp@hp.com">travis.tripp@hp.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">​>>Do we really need the lines:​<br>
<br>
>> project = api.keystone.tenant_get(<u></u>request, id)<br>
>> kwargs = _tenant_kwargs_from_DATA(<u></u>request.DATA, enabled=None)<br>
​<br>
I agree that if you already have all the data it is really bad to have to do another call. I do think there is room for discussing the reasoning, though.<br>
As far as I can tell, they do this so that if you are updating an entity, you have to be very specific about the fields you are changing. I actually see this as potentially a protectionary measure against data loss and a sometimes very nice to have feature. It perhaps was intended to *help* guard against race conditions *sometimes*.<br></blockquote><div><br></div><div>Yep, it looks like I broke this API by implementing it the way I did, and I'll alter the API so that you pass both the "current" object (according to the client) and the parameters to alter.</div><div><br></div><div>Thanks everyone for the great reviewing!</div><div><br></div><div><br></div><div>     Richard</div></div>