GabrielBrascher edited a comment on issue #5161:
URL: https://github.com/apache/cloudstack/issues/5161#issuecomment-870667689


   @div8cn thanks for confirming.
   
   @rhtyd I am having the same issue; the UI button for reverting a snapshot is 
"missing".
   
   The new UI presents the button only if the snapshot response has 
`revertable` as `true`. However, I am getting `revertable` as `false` in 
snapshots. This includes also a snapshot taken on NFS, not just Ceph/RBD.
   
   List Snapshots via API.
   ```
   cmk> list snapshots filter=name,state,revertable
   +-------------------+----------+------------+
   |       NAME        |  STATE   | REVERTABLE |
   +-------------------+----------+------------+
   | primary-nfs-snap2 | BackedUp | false      |
   | primary-nfs-snap  | BackedUp | false      |
   | test-snap1-rbd    | BackedUp | false      |
   +-------------------+----------+------------+
   ```
   
   Mysql select on snapshot_store_ref checking that it indeed happens with 
snapshots taken on `NFS` & `RBD`.
   
   ```
   > select store_role,install_path,state from snapshot_store_ref order by id 
desc limit 3;
   +------------+---------------------------+-------+
   | store_role | install_path              | state |
   +------------+---------------------------+-------+
   | Primary    | /mnt/<uuid>/<uuid>/<uuid> | Ready |
   | Primary    | /mnt/<uuid>/<uuid>/<uuid> | Ready |
   | Primary    | rbd/<uuid>/<uuid>         | Ready |
   +------------+---------------------------+-------+
   ```
   
   The UI then will not present the revertable button.
   
   1. Example of the UI buttons for the tested snapshots:
   
![image](https://user-images.githubusercontent.com/5025148/123819017-d4813700-d8cf-11eb-98fc-a77ab63224b0.png)
   
   
   2. Example with the UI "ignoring" the revertable in the response:
   
![image](https://user-images.githubusercontent.com/5025148/123818857-b74c6880-d8cf-11eb-9a7a-fb061ae02dfa.png)
   
   
   It seems that in the past releases it worked because the legacy UI did not 
check this; the new UI on the other hand has a validation and does not present 
the button in case revertable is false.


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