Add support for samba setup using ceph `smb` manager module

samba-in-kubernetes / sit-environment

SIT (Samba Integration Testing) Framework

GNU General Public License v3.0

1 stars 7 forks source link

Add support for samba setup using ceph `smb` manager module #113

Closed anoopcs9 closed 3 months ago

anoopcs9 commented 4 months ago

Enable and create samba shares with ceph smb mgr module.
Explicit creation of samba users via users.setup is skipped(achieved through smb mgr module).
Client access directed at node running smb service.

Finally, we have a new backend - cephfs.mgr.vfs 😀 .

anoopcs9 commented 4 months ago

EXTRA_VARS="backend=cephfs.mgr.vfs test_sanity_only=1" make test

Above command successfully completed.

anoopcs9 commented 4 months ago

I've added the do-not-merge label to include the suggestions from @phlogistonjohn and @xhernandez in the current PR itself. I'll update shortly.

anoopcs9 commented 4 months ago

Error ENOTSUP: Warning: due to ceph-mgr restart, some PG states may not be up to date\nModule 'smb' is not enabled/loaded (required by command 'smb apply'): use ceph mgr module enable smb to enable it

@phlogistonjohn I happen to see this error on CI cluster(not locally for me) while trying to create smb cluster immediately after smb mgr module is enabled. May be we wait for mgr to be running after ceph mgr module enable smb?

phlogistonjohn commented 4 months ago

I happen to see this error on CI cluster(not locally for me) while trying to create smb cluster immediately after smb mgr module is enabled. May be we wait for mgr to be running after ceph mgr module enable smb?

Yes, try waiting or waiting longer. Enabling the module triggers a mgr restart and it's not the fastest.

anoopcs9 commented 4 months ago

updated code lgtm (modulo the restart issue you are seeing)

Added a task to wait for mgr restart to complete and tests successfully passed 😊 .

anoopcs9 commented 4 months ago

updated code lgtm (modulo the restart issue you are seeing)

Added a task to wait for mgr restart to complete and tests successfully passed 😊 .

I spoke too soon. Full test suite run has some smbtorture failures.

anoopcs9 commented 3 months ago

updated code lgtm (modulo the restart issue you are seeing)

Added a task to wait for mgr restart to complete and tests successfully passed 😊 .

I spoke too soon. Full test suite run has some smbtorture failures.

I think its good to start with the CI runs and eventually debug and provide fixes. Removing the do-not-merge label.