DaanHoogland commented on code in PR #13719:
URL: https://github.com/apache/cloudstack/pull/13719#discussion_r3659233994
##########
server/src/main/java/com/cloud/resource/ResourceManagerImpl.java:
##########
@@ -3213,9 +3216,61 @@ private HostVO getNewHost(StartupCommand[]
startupCommands) {
}
logger.debug(String.format("Could not find Host by guid %s",
fullGuid));
+
+ // No live host matches this GUID. Before letting the caller create a
brand-new host,
+ // make sure this GUID does not belong to a host that was previously
deleted. Otherwise a
+ // still-running agent whose host was deleted would silently
re-register itself as a new host.
Review Comment:
You can physically move a host, right? to add it in another cluster I
mean/rewiring/screwing it in another rack. In this case you need to reinstall
it or at least make sure a new GUID is assigned. I guess it is alright to lose
this “feature”, just so long as we are aware.
--
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]
