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


   @rhtyd ceph snapshot work really well indeed; however, it throws an 
exception when taking RBD volume snapshots in case of the Zone has 
`snapshot.backup.to.secondary` set to **true**.
   
   #### Scenario where it can happen:
   
   For instance, if there is a zone with local storage, NFS, and RBD as primary 
storage pools it could be interesting to have `snapshot.backup.to.secondary` 
enabled (at least for the local storage and NFS). The problem is that if backup 
is enabled then all RBD snapshots fail. At least it should allow the snapshot 
and then raise a warn such as _"backup on secondary pool is not supported for 
RBD snapshots, RBD snapshots will remain only on the primary storage"_.
   
   #### Exception example:
   
   For now it fails as it tries to access the RBD storage pool locally, failing 
to parse configuration files and pool the path.
   
   ```
   2020-11-20 20:23:59,916 ERROR [o.a.c.s.v.VolumeServiceImpl] 
(Work-Job-Executor-4:ctx-3de4fd53 job-57289/job-57290 ctx-3e53cbdd) 
(logid:05d7071d) Take snapshot: 12888 failed
   com.cloud.utils.exception.CloudRuntimeException: 
org.apache.cloudstack.utils.qemu.QemuImgException: 2020-11-20 20:23:18.810175 
7f96fa5b48c0 -1 Errors while parsing config file!2020-11-20 20:23:18.810179 
7f96fa5b48c0 -1 parse_file: cannot open /etc/ceph/ceph.conf: (2) No 
   such file or directory2020-11-20 20:23:18.810181 7f96fa5b48c0 -1 parse_file: 
cannot open ~/.ceph/ceph.conf: (2) No such file or directory2020-11-20 
20:23:18.810182 7f96fa5b48c0 -1 parse_file: cannot open ceph.conf: (2) No such 
file or directoryqemu-img: Could not open 'r
   
bd:rbd/9aaef3ea-4d3a-40d0-ab36-5e3db7393a50@28d4acf2-6886-4da1-925e-62910dcf2fd4:mon_host=ceph.address\\:6789:auth_supported=cephx:id=cloudstack:key=BGCADKsHdh7BWChDABVsp5WHinnDCBOZ91dHfgw==:rbd_default_format=2:client_mount_timeout=30':
 invalid conf option Н�I^]
   V: No such file or directory
           at 
org.apache.cloudstack.storage.snapshot.SnapshotServiceImpl.backupSnapshot(SnapshotServiceImpl.java:301)
           at 
org.apache.cloudstack.storage.snapshot.DefaultSnapshotStrategy.backupSnapshot(DefaultSnapshotStrategy.java:171)
   ```
   
   #### Proposal of next steps:
   
   There is two things that can be done:
   **A.** fix the handling allowing multiple stortage pools to handle snapshot 
backup, as described above;
   **B.** evaulate alternatives to implement RBD snapshots backup.
   
   My proposal: I will check first the alternative **A** as a bug fix, creating 
a PR (for 4.15.1, or 4.16?). Later I can evaluate viability of implementing 
**B** as a new feature.
   
   **Note:** I do not consider this issue a blocker for 4.15.


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