[GitHub] [cloudstack] rhtyd commented on a change in pull request #5453: Updated the event message with proper json format for cmd info and job result

[GitBox <gi...@apache.org>]

rhtyd commented on a change in pull request #5453:
URL: https://github.com/apache/cloudstack/pull/5453#discussion_r709785276



##########
File path: server/src/main/java/com/cloud/api/ApiServer.java
##########
@@ -347,9 +355,9 @@ public void handleAsyncJobPublishEvent(String subject, String senderAddress, Obj
         eventDescription.put("instanceType", instanceType);
         eventDescription.put("commandEventType", cmdEventType);
         eventDescription.put("jobId", job.getUuid());
-        eventDescription.put("jobResult", job.getResult());
-        eventDescription.put("cmdInfo", job.getCmdInfo());
-        eventDescription.put("status", "" + job.getStatus() );
+        eventDescription.put("jobResult", ApiSerializerHelper.fromSerializedStringToMap(job.getResult()));

Review comment:
       @sureshanaparti could this possibly break someone's scripts? 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org