DaanHoogland commented on a change in pull request #5831:
URL: https://github.com/apache/cloudstack/pull/5831#discussion_r797738473



##########
File path: 
engine/schema/src/main/java/com/cloud/upgrade/SystemVmTemplateRegistration.java
##########
@@ -690,7 +679,13 @@ public void 
registerTemplate(Pair<Hypervisor.HypervisorType, String> hypervisorA
         }
     }
 
-    public static void parseMetadataFile() {
+    /**
+     * This method parses the metadata file consisting of the systemVM 
templates information
+     * @return the version of the systemvm template that is to be used. This 
is done to in order
+     * to fallback on the latest available version of the systemVM template 
when there does not
+     * exist a template corresponding to the current code version.
+     */

Review comment:
       ```suggestion
       /**
        * This method parses the metadata file consisting of the systemVM 
templates information
        * @return the version of the systemvm template that is to be used. This 
is done in order
        * to fallback on the latest available version of the systemVM template 
when there doesn't
        * exist a template corresponding to the current code version.
        */
   ```




-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to