rhtyd commented on a change in pull request #5681:
URL: https://github.com/apache/cloudstack/pull/5681#discussion_r748010960
##########
File path:
api/src/main/java/org/apache/cloudstack/api/command/admin/host/AddHostCmd.java
##########
@@ -60,6 +60,9 @@
@Parameter(name = ApiConstants.PASSWORD, type = CommandType.STRING,
description = "the password for the host; required to be passed for hypervisors
other than VMWare")
private String password;
+ @Parameter(name = ApiConstants.PRIVATE_KEY, type = CommandType.STRING,
description = "the private key for the host; support KVM only.", length = 16384)
+ private String privateKey;
Review comment:
No this beats the purpose, we should assume that admin will setup and
sync mgmt server ssh key for the cloud user across hosts.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
