DaanHoogland commented on a change in pull request #3925: Add cache mode param 
properly
URL: https://github.com/apache/cloudstack/pull/3925#discussion_r387507238
 
 

 ##########
 File path: test/integration/smoke/test_disk_offerings.py
 ##########
 @@ -90,9 +90,81 @@ def test_01_create_disk_offering(self):
                         )
         return
 
+    @attr(tags=["advanced", "basic", "eip", "sg", "advancedns", "smoke"], 
required_hardware="false")
+    def test_02_create_disk_offering_with_cache_mode_type(self):
 
 Review comment:
   Why this renumbering/renaming? It can break statistic analysis of test 
breakage or comparison. Why not add them to the end with new numbers?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to