JoaoJandre commented on PR #9270:
URL: https://github.com/apache/cloudstack/pull/9270#issuecomment-2736535321

   > > @winterhazel the last three commits fixed the behaviors you pointed out. 
Could you check if your concerns are met?
   > 
   > @JoaoJandre the two bugs I pointed out were indeed fixed. However, I found 
another one while doing some extra tests:
   > 
   >     1. Disable backup to secondary storage, and enable incremental 
snapshots
   > 
   >     2. Take a snapshot
   > 
   >     3. Enable backup to secondary storage
   > 
   >     4. Try to take another snapshot
   > 
   > 
   > Doing this results in an exception at step 4. I'm unable to take snapshots 
of this VM unless I delete the snapshot that exists in primary storage or 
disable backup to secondary storage.
   
   @winterhazel The last commit should fix the bug. Now when changing storage 
types, a new chain will be started. I tested going from secondary to primary 
and the opposite. Also tested just chaining snapshots on either primary or 
secondary. 


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