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

sit.cephfs: Add share configured with vfs_ceph_new(mgr) #118

Merged
merged 2 commits into from
Sep 12, 2024

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Aug 14, 2024

depends on ceph/ceph#58994
depends on #116

Copy link

dpulls bot commented Aug 14, 2024

⚠️ Dpulls not installed on repository ceph/ceph. Checkout our quickstart for how to install.

@anoopcs9 anoopcs9 marked this pull request as ready for review August 15, 2024 17:35
Copy link

dpulls bot commented Aug 27, 2024

⚠️ Dpulls not installed on repository ceph/ceph. Checkout our quickstart for how to install.

Copy link

dpulls bot commented Aug 28, 2024

🎉 All dependencies have been resolved !

@anoopcs9 anoopcs9 force-pushed the vfs-ceph-new2-mgr branch 2 times, most recently from 16a528b to a0698a2 Compare September 11, 2024 17:49
"share_id" field for defining share resources cannot contain a dot(.),
underscore(_) or forward slash(/) in its value. Therefore replace dot
with hyphen(-) in the list of methods which will be used further to
constitute unique "share_id".

Signed-off-by: Anoop C S <[email protected]>
@anoopcs9 anoopcs9 merged commit dd10e56 into samba-in-kubernetes:main Sep 12, 2024
8 checks passed
@anoopcs9 anoopcs9 deleted the vfs-ceph-new2-mgr branch September 12, 2024 11:45
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.

2 participants