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 proxy enbaled shares with vfs_ceph_new (mgr) #130

Merged
merged 3 commits into from
Mar 1, 2025

Conversation

anoopcs9
Copy link
Collaborator

With ceph/ceph#59740 merged ceph smb manager module is capable of creating shares with ceph_new:proxy module option set to "yes". Thus an additional share with proxy support is created for mgr variant.

@anoopcs9 anoopcs9 changed the title sit.cephfs: Add proxy enbaled shares with vfs_ceph_new (mgr) sit.cephfs: Add proxy enbaled shares with vfs_ceph_new (mgr) Feb 28, 2025
It is not a big deal for us to fit the method name within the current
limit for share_id based on the following regular expression from smb
mgr module:

^[a-zA-Z0-9]($|[a-zA-Z0-9-]{,16}[a-zA-Z0-9]$)

Otherwise share id 'share-vfs-new-proxy' constituted goes beyond the
limit resulting in failure to create share using mgr variant.

Signed-off-by: Anoop C S <[email protected]>
@anoopcs9 anoopcs9 force-pushed the add-cephfs-proxy-mgr branch 2 times, most recently from b6f2005 to 2f69fc4 Compare February 28, 2025 15:40
@anoopcs9 anoopcs9 marked this pull request as ready for review February 28, 2025 18:21
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

I'm a bit rusty wrt how things work in SIT, but seems OK to me.

@anoopcs9 anoopcs9 merged commit b3b30bb into samba-in-kubernetes:main Mar 1, 2025
7 of 8 checks passed
@anoopcs9 anoopcs9 deleted the add-cephfs-proxy-mgr branch March 1, 2025 08:41
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