[
https://issues.apache.org/jira/browse/CLOUDSTACK-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14599081#comment-14599081
]
ASF GitHub Bot commented on CLOUDSTACK-8578:
--------------------------------------------
Github user resmo commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/513#discussion_r33126603
--- Diff: server/src/com/cloud/api/query/QueryManagerImpl.java ---
@@ -968,7 +968,11 @@
}
if (state != null) {
- sc.setParameters("stateEQ", state);
+ if (state.equalsIgnoreCase("present")) {
--- End diff --
Good catch, IMHO `Expunged` is not necessary but `Expunging` like we use
for user e.g.
https://github.com/resmo/cloudstack/blob/fix/CLOUDSTACK-8578-extend/server/src/com/cloud/api/query/QueryManagerImpl.java#L984
> listVirtualMachines does not return deleted machines when zone is specified
> ---------------------------------------------------------------------------
>
> Key: CLOUDSTACK-8578
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8578
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Components: API
> Affects Versions: 4.1.1, 4.2.1, 4.3.1, 4.4.3, 4.5.1
> Reporter: René Moser
> Priority: Minor
>
> A REST API query "command=listVirtualMachines&zoneid=<zone-uuid>" does not
> return destroyed virtual machines in the response.
> There was a similar issue reported in #284 and fixed but in accidentally
> re-implemented in
> https://github.com/apache/cloudstack/commit/c167ad45e3480160c086e5bf532a72d9704c072b#diff-20f157170d9a50e3010375d071fd4db5R747
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
