weizhouapache commented on issue #6209:
URL: https://github.com/apache/cloudstack/issues/6209#issuecomment-1102694473

   @nvazquez @slavkap @Pearl1594 
   
   my opinion, for kvm with NFS storage,
   (1) the record should be removed from DB, because no snapshot exists in 
primary storage.
   (2) the primary datastore info should be optional (not required) when revert 
snapshot on KVM, because kvm agent tries to copy snapshot from secondary 
storage. the operation should succeed even if primary datastore info is not 
provided. however, from code it looks it is required.
   (3) it looks ceph storage and managed storage are not impacted. correct me 
if I am wrong


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