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

MDS names may not start with a numeric digit. #97

Closed
leseb opened this issue Jun 17, 2015 · 2 comments · Fixed by #101
Closed

MDS names may not start with a numeric digit. #97

leseb opened this issue Jun 17, 2015 · 2 comments · Fixed by #101

Comments

@leseb
Copy link
Member

leseb commented Jun 17, 2015

Just spotted this:

deprecation warning: MDS id 'mds.7cf73bf1fd74' is invalid and will be forbidden in a future version. MDS names may not start with a numeric digit.

An easy work around could be to do something like this on the MDS_NAME variable: : ${MDS_NAME:=a$(hostname -s)}

@Ulexus
Copy link
Contributor

Ulexus commented Jun 17, 2015

I would slightly prefer a more mnemonic "mds-" prefix, but otherwise, that sounds like a good idea.

One thing to note, though, is that metadata servers are not machine-restricted. They are not bound by their data directories and can move around the cluster. As a result, I generally give mine labels like "a", "b", "c".

In this case, obviously, we cannot do that, but I wonder if adding and removing arbitrarily-named metadata servers screws with the Ceph database.

@leseb
Copy link
Member Author

leseb commented Jun 19, 2015

I like the "daemon type" prefix and I agree about the statement of volatile MDSs in general.
Perhaps we should encourage users to use MDS_NAME and justify this with your explanations.

Let's start by fixing this and we can always add a bit of doc later.

leseb added a commit that referenced this issue Jun 19, 2015
This fixes #97

Signed-off-by: Sébastien Han <seb@redhat.com>
leseb added a commit that referenced this issue Jun 19, 2015
This fixes #97

Signed-off-by: Sébastien Han <seb@redhat.com>
mkkie pushed a commit to mkkie/ceph-container that referenced this issue Mar 1, 2018
Fix Bug. Got the wrong OSD_ID when verify OSD.
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 a pull request may close this issue.

2 participants