You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by gi...@git.apache.org on 2017/08/29 11:08:53 UTC
[GitHub] DaanHoogland commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider
DaanHoogland commented on a change in pull request #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA provider
URL: https://github.com/apache/cloudstack/pull/1960#discussion_r135757989
##########
File path: api/src/org/apache/cloudstack/api/command/admin/host/PrepareForMaintenanceCmd.java
##########
@@ -108,4 +108,8 @@ public void execute() {
throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, "Failed to prepare host for maintenance");
}
}
+
+ public void setHostId(final Long hostId) {
+ id = hostId;
Review comment:
this setter is still to be called setId(..) instead of setHostId(..), or should the field be renamed?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services