api: include tags in listvmsnapshots response (#3216)

This ensures that tags of a VM snapshot are listed in the UI, available
in the list vmsnapshots API response.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

Author: yadvr

api/src/main/java/org/apache/cloudstack/api/response/VMSnapshotResponse.java

@@ -18,18 +18,19 @@
 package org.apache.cloudstack.api.response;
 
 import java.util.Date;
-
-import com.google.gson.annotations.SerializedName;
+import java.util.LinkedHashSet;
+import java.util.Set;
 
 import org.apache.cloudstack.api.ApiConstants;
-import org.apache.cloudstack.api.BaseResponse;
+import org.apache.cloudstack.api.BaseResponseWithTagInformation;
 import org.apache.cloudstack.api.EntityReference;
 
 import com.cloud.serializer.Param;
 import com.cloud.vm.snapshot.VMSnapshot;
+import com.google.gson.annotations.SerializedName;
 
 @EntityReference(value = VMSnapshot.class)
-public class VMSnapshotResponse extends BaseResponse implements ControlledEntityResponse {
+public class VMSnapshotResponse extends BaseResponseWithTagInformation implements ControlledEntityResponse {
 
     @SerializedName(ApiConstants.ID)
     @Param(description = "the ID of the vm snapshot")
@@ -99,6 +100,10 @@ public class VMSnapshotResponse extends BaseResponse implements ControlledEntity
     @Param(description = "the domain associated with the disk volume")
     private String domainName;
 
+    public VMSnapshotResponse() {
+        tags = new LinkedHashSet<ResourceTagResponse>();
+    }
+
     @Override
     public String getObjectId() {
         return getId();
@@ -226,6 +231,9 @@ public void setDomainId(String domainId) {
     @Override
     public void setDomainName(String domainName) {
         this.domainName = domainName;
+    }
 
+    public void setTags(Set<ResourceTagResponse> tags) {
+        this.tags = tags;
     }
 }

server/src/main/java/com/cloud/api/ApiResponseHelper.java

@@ -619,6 +619,14 @@ public VMSnapshotResponse createVMSnapshotResponse(VMSnapshot vmSnapshot) {
             vmSnapshotResponse.setDomainName(domain.getName());
         }
 
+        List<? extends ResourceTag> tags = _resourceTagDao.listBy(vmSnapshot.getId(), ResourceObjectType.VMSnapshot);
+        List<ResourceTagResponse> tagResponses = new ArrayList<ResourceTagResponse>();
+        for (ResourceTag tag : tags) {
+            ResourceTagResponse tagResponse = createResourceTagResponse(tag, false);
+            CollectionUtils.addIgnoreNull(tagResponses, tagResponse);
+        }
+        vmSnapshotResponse.setTags(new HashSet<>(tagResponses));
+
         vmSnapshotResponse.setCurrent(vmSnapshot.getCurrent());
         vmSnapshotResponse.setType(vmSnapshot.getType().toString());
         vmSnapshotResponse.setObjectName("vmsnapshot");