[
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)