rafaelweingartner commented on a change in pull request #2578: api: add command to list management servers URL: https://github.com/apache/cloudstack/pull/2578#discussion_r225156473
########## File path: engine/schema/src/main/java/com/cloud/upgrade/dao/Upgrade41120to41200.java ########## @@ -52,7 +57,15 @@ public boolean supportsRollingUpgrade() { @Override public void performDataMigration(Connection conn) { + updateManagementServerHostUuid(conn); + } + private void updateManagementServerHostUuid(Connection conn) { + try (final PreparedStatement updateStatement = conn.prepareStatement("UPDATE cloud.mshost SET uuid=UUID()")) { + updateStatement.executeUpdate(); + } catch (SQLException e) { + LOG.error("Failed to set dynamic.apichecker.enabled to true, please run migrate-dynamicroles.py script to manually migrate to dynamic roles.", e); Review comment: This message looks a little bit misleading, or am I mistaken? ---------------------------------------------------------------- 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 With regards, Apache Git Services