<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 12, 2013 at 7:47 AM, Zane Bitter <span dir="ltr"><<a href="mailto:zbitter@redhat.com" target="_blank">zbitter@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 02/11/13 05:30, Clint Byrum wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Excerpts from Christopher Armstrong's message of 2013-11-01 11:34:56 -0700:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Vijendar and I are trying to figure out if we need to set the resource_id<br>
of a resource to None when it's being deleted.<br>
<br>
This is done in a few resources, but not everywhere. To me it seems either<br>
<br>
a) redundant, since the resource is going to be deleted anyway (thus<br>
deleting the row in the DB that has the resource_id column)<br>
b) actively harmful to useful debuggability, since if the resource is<br>
soft-deleted, you'll not be able to find out what physical resource it<br>
represented before it's cleaned up.<br>
<br>
Is there some specific reason we should be calling resource_id_set(None) in<br>
a check_delete_complete method?<br>
<br>
</blockquote>
<br>
I've often wondered why some do it, and some don't.<br>
<br>
Seems to me that it should be done not inside each resource plugin but<br>
in the generic resource handling code.<br>
<br>
However, I have not given this much thought. Perhaps others can provide<br>
insight into why it has been done that way.<br>
</blockquote>
<br></div>
There was a time in the very early days of Heat development when deleting something that had already disappeared usually resulted in an error (i.e. we mostly weren't catching NotFound exceptions). I expect this habit dates from that era.<br>

<br>
I can't think of any reason we still need this, and I agree that it seems unhelpful for debugging.<br>
<br>
cheers,<br>
Zane.<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Thanks Zane and others who have responded. My recent patch (now already merged) won't delete the resource_id.</div><div> </div>
</div><div><br></div>-- <br><div dir="ltr"><div>IRC: radix</div>Christopher Armstrong<div>Rackspace</div></div>
</div></div>