[ https://issues.apache.org/jira/browse/CLOUDSTACK-10132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16253696#comment-16253696 ]
ASF GitHub Bot commented on CLOUDSTACK-10132: --------------------------------------------- nvazquez commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r151170875 ########## File path: agent/src/com/cloud/agent/Agent.java ########## @@ -265,6 +273,8 @@ public void start() { s_logger.info("Attempted to connect to the server, but received an unexpected exception, trying again..."); } } + _shell.updateConnectedHost(); Review comment: @rhtyd I decided to take that path as when agent is launched, the next host is set on constructor and then from `start()` method we couldn't know which host the agent is trying to connect to unless we do the same as in AgentShell with `hostToConnect` but on Agent class. Which approach do you think it would be best? ---------------------------------------------------------------- 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: us...@infra.apache.org > Multiple Management Servers Support for agents > ---------------------------------------------- > > Key: CLOUDSTACK-10132 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10132 > Project: CloudStack > Issue Type: Improvement > Security Level: Public(Anyone can view this level - this is the > default.) > Affects Versions: 4.11.0.0 > Reporter: Nicolas Vazquez > Assignee: Nicolas Vazquez > > Multiple Management Servers Support for agents -- This message was sent by Atlassian JIRA (v6.4.14#64029)