Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose CephFS snapshots as shadow-copies #105

Merged

Conversation

Shwetha-Acharya
Copy link
Collaborator

Use kernel CephFS mounted share path to create the shadow copy using ceph_snapshots module

Refer: https://www.samba.org/samba/docs/4.11/man-html/vfs_ceph_snapshots.8.html

@Shwetha-Acharya Shwetha-Acharya force-pushed the snapshot-in-ceph branch 2 times, most recently from 018fbea to 99b7b03 Compare June 11, 2024 08:39
@Shwetha-Acharya
Copy link
Collaborator Author

/retest all

@Shwetha-Acharya
Copy link
Collaborator Author

/retest centos-ci/cephfs

2 similar comments
@Shwetha-Acharya
Copy link
Collaborator Author

/retest centos-ci/cephfs

@Shwetha-Acharya
Copy link
Collaborator Author

/retest centos-ci/cephfs

@Shwetha-Acharya Shwetha-Acharya force-pushed the snapshot-in-ceph branch 3 times, most recently from 437cb8d to 8888df7 Compare June 25, 2024 12:31
@Shwetha-Acharya
Copy link
Collaborator Author

/retest all

xhernandez
xhernandez previously approved these changes Jun 27, 2024
Copy link
Collaborator

@xhernandez xhernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks fine. The only question is whether we should make the inclusion of ceph_snapshots configurable or not, but this can be done in another patch if necessary.

@Shwetha-Acharya
Copy link
Collaborator Author

The change looks fine. The only question is whether we should make the inclusion of ceph_snapshots configurable or not, but this can be done in another patch if necessary.

yes, that can be done if required

Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see an odd z at the very beginning of the commit subject zExpose CephFS snapshots as shadow-copies.

create shadow copy with the help of ceph_snapshots module:
- use kernel CephFS mounted share path for cephfs
- use absolute path within the Ceph filesystem for cephfs.vfs

Refer: https://www.samba.org/samba/docs/4.11/man-html/vfs_ceph_snapshots.8.html
Signed-off-by: Shwetha K Acharya <[email protected]>
Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks.

@xhernandez xhernandez merged commit 7b38d47 into samba-in-kubernetes:main Jun 28, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants