PaulAngus commented on pull request #4283:
URL: https://github.com/apache/cloudstack/pull/4283#issuecomment-754037964


   @RodrigoDLopez
   
   1. the tests above are regression tests. They therefore cannot check whether 
the feature works when used as they can only look backwards.  They are also 
only one small part of the testing rigour that should be applied to any new 
feature.
   
   2. the community is working RCs for the 4.15 release, and certainly, I have 
not heard that master is available to add new features.
   
   3. The tag validations are there to stop the service offerings from being 
broken - if you allow **anyone** to move VMs or storage contrary to the applied 
tags they are highly likely to pollute the database with resources in places 
that their service offerings say that they can't be.
   
   4. If the problem is with moving between conflicting tags, then there needs 
to be a mechanism to facilitate changing the service offerings applied to a 
VM/volume, where the VM/storage is automatically migrated to matching 
hosts/storage; keeping the integrity of the database intact.
   
   


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


Reply via email to