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

have 3 MDSs (1 up while 2 in standby) for the backend ceph cluster #25

Open
dparmar18 opened this issue Sep 6, 2023 · 4 comments
Open
Labels
enhancement New feature or request

Comments

@dparmar18
Copy link

I'm not sure if I might be missing out on some code that I didn't look at but I was going through #17 and it seems there's just one MDS being created, right? I'd say have at least 3 which makes it easy to handle any workload and/or any failure/crash in active MDS.

@obnoxxx obnoxxx added the enhancement New feature or request label Sep 6, 2023
@obnoxxx
Copy link
Collaborator

obnoxxx commented Sep 6, 2023

Thanks for this suggestion, @dparmar18, this is very useful!

we can certainly look into adding this enhancement.
@xhernandez , @phlogistonjohn - what do you think?

@phlogistonjohn
Copy link
Collaborator

Sure, it should just be a matter of setting the count for the placement spec.
https://docs.ceph.com/en/latest/cephadm/services/mds/#deploy-cephfs

@xhernandez
Copy link
Collaborator

What's exactly the purpose of the additional MDS ?

If they are there just in case of a failure of the active MDS, I would prefer to catch that and see the failure. The purpose of this test environment is to identify functional issues in both Samba and CephFS to fix them and improve both projects, not to create a highly resilient environment (specially if this could hide some unwanted failures).

@spuiuk
Copy link
Collaborator

spuiuk commented Sep 11, 2023

I agree with Xavi here. We are not testing cephfs directly but the cephfs over SMB and for that we don't really require 3 MDS machines which are required for high availability. That will simply take more resources and is not really tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants