[I] managementserverid value inconsistency in shutdown APIs [cloudstack]

[via GitHub]

adietrich-ussignal opened a new issue, #10669:
URL: https://github.com/apache/cloudstack/issues/10669

   ### problem
   
   When using the shutdown APIs (`prepareForShutdown`, `triggerShutdown`, etc.) 
the management server ID is required, however, the return value appears to be 
the database ID.
   
   ```
   prepare forshutdown managementserverid=0b2faa01-706f-4f37-82ff-3e5e8f0971f8
   {
     "prepareforshutdown": {
       "managementserverid": 1,
       "pendingjobscount": 0,
       "readyforshutdown": true,
       "shutdowntriggered": true
     }
   }
   ```
   
   ### versions
   
   ACS 4.19.2.0
   CMK 6.4.0
   
   ### The steps to reproduce the bug
   
   1. `prepare forshutdown managementserverid=<UUID of management server>`
   2. Observe return values.
   ...
   
   
   ### What to do about it?
   
   To align with other API calls, it would be more valuable to pass the UUID of 
the management server in the return value for `managementserverid`. 


-- 
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-unsubscr...@cloudstack.apache.org.apache.org

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