[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9297?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15199780#comment-15199780
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9297:
--------------------------------------------

Github user rafaelweingartner commented on the pull request:

    https://github.com/apache/cloudstack/pull/1441#issuecomment-197948544
  
    I understand that your point that some other classes can overwrite that 
method and may return other elements of the enum. However, I do not see 
anything else in the code that used the “canHandle” method return that uses the 
returned value. I mean, it is used in the “revertSnapshot” method at line 285, 
but it only checks if the return is equal to “StrategyPriority.CANT_HANDLE” and 
then throws an exception if it is. To me, that is the same as if the method 
were returning a boolean value.


> delete snapshot without id is failing with Unable to determine the storage 
> pool of the snapshot
> -----------------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-9297
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9297
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: API
>    Affects Versions: 4.7.0, 4.8.0
>         Environment: Ubuntu 14.04.3 management + kvm
>            Reporter: Özhan Rüzgar Karaman
>            Assignee: Mike Tutkowski
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to