nicoschmdt commented on code in PR #9504:
URL: https://github.com/apache/cloudstack/pull/9504#discussion_r1971730462


##########
engine/schema/src/main/java/com/cloud/upgrade/dao/Upgrade42010to42100.java:
##########
@@ -57,9 +60,52 @@ public InputStream[] getPrepareScripts() {
         return new InputStream[] {script};
     }
 
+    protected void performKeyPairMigration(Connection conn) throws 
SQLException {

Review Comment:
   I modified the code to migrate the key pairs directly through SQL. 
   
   To test the migration, I created, in version 4.20 of CloudStack, one domain 
admin account and added to it three users, and for the third user I generated a 
key pair. When updating CloudStack to version 4.21, I verified the key pair was 
correctly migrated to the new table and asserted the CloudMonkey requisitions 
were functioning.



-- 
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]

Reply via email to