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 tha tif this 
is enabled then all RBD snapshots will fail. At least it should raise a warn 
then taking RBD volume snapshots such as _"backup on secondray pool is not 
supported for RBD snapshots, snapshot is kept 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