deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jvlcek <>
Subject Re: DTACLOUD-379 using Marios's 409 solution.
Date Wed, 12 Dec 2012 14:40:45 GMT
On 12/12/2012 09:24 AM, jvlcek wrote:
> On 12/11/2012 07:05 PM, David Lutterkort wrote:
>> On Tue, 2012-12-11 at 17:27 -0500, wrote:
>>> DTACLOUD-379 using Marios's 409 solution.
> Thank you David,
>> ACK to this; stylistically, the two patches should be squashed into one,
>> since after applying 1/2 triggering this error will cause a server error
>> because of a missing template.
> My intention was to clearly show credit to Marios for the work he had done.
> Collapsing the patches into one would have made it appear that the
> entire solution was only my doing. That said I fully agree with your
> point about the one patch alone leading to issues.
>> Also, the commit message is way too terse; it should contain a sentence
>> on what the problem was (RHEV-M does not allow deletion of templates
>> that are in use) and what the fix was (propagating the error cleanly)
> OK will do in the future.
>> Related to this: looking at the XML template for image, we generate
>> links for 'create_instance' and 'delete_image' unconditionally. That
>> probably needs to be changed so we only create links for actions that
>> can currently be performed.
> I'm  working another Jira for exactly the same issue on an
> Openstack template.
> I'll add this issue to that work.
> Thanks for the feedback!
>   Joe


Michal Fojtik has pointed out this patch breaks tests.

I will fix the tests and resubmit the patch.


View raw message