cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mandar Barve <mandar.ba...@sungard.com>
Subject Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should
Date Mon, 10 Mar 2014 04:05:27 GMT
Hi Mike,
     Did you get a chance to look at this?

Thanks,
Mandar


On Wed, Mar 5, 2014 at 10:12 AM, Mandar Barve <mandar.barve@sungard.com>wrote:

> I tested this with CS 4.3.
>
> Thanks,
> Mandar
>
>
> On Tue, Mar 4, 2014 at 9:09 PM, Mike Tutkowski <
> mike.tutkowski@solidfire.com> wrote:
>
>> Hi,
>>
>> Can you tell me what release you tested this with? I noticed the problem
>> while developing on CloudStack 4.3.
>>
>> Thanks!
>>
>>
>> On Tue, Mar 4, 2014 at 3:43 AM, Mandar Barve <mandar.barve@sungard.com>wrote:
>>
>>> Hi,
>>>         I tried to reproduce the issue but couldn't get this to fail for
>>> insufficient space. I then injected an exception trying to list files from
>>> a non existent path (added this code in the "try" block). This landed me
>>> into the exception handling code. It raised correct exception saying "file
>>> not found" which was captured in the management server vmops log file. It
>>> was not displayed by the GUI. GUI just reported Error (Are we looking for
>>> GUI displaying error code?). The plugin code returns "0" immediately after
>>> the line of code that raises exception but I think this applies only for
>>> successful execution of the plugin code that reverts the snapshot.
>>>
>>>        If any exception is raised (e.g. in the reported case here
>>> insufficient space) then the code should return appropriate error message
>>> to the caller as I found. In exception handling path return "0" wouldn't
>>> execute.
>>>
>>> I don't see any problem here. Let me know if I am missing anything.
>>>
>>> Thanks,
>>> Mandar
>>>
>>
>>
>>
>> --
>> *Mike Tutkowski*
>>  *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkowski@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play>
>> *(tm)*
>>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message