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

Jump to bottom

doc: mention MDS name and ID format in ceph-mds.rst #44971

Merged
merged 1 commit into from Feb 14, 2022
Merged

doc: mention MDS name and ID format in ceph-mds.rst #44971

merged 1 commit into from Feb 14, 2022

Conversation

joscollin
Copy link
Member

@joscollin joscollin commented Feb 10, 2022 •
edited

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@joscollin joscollin requested a review from a team as a code owner February 10, 2022 11:14
@github-actions github-actions bot added cephfs Ceph File System documentation labels Feb 10, 2022
@joscollin
Copy link
Member Author

jenkins test docs

@joscollin
Copy link
Member Author

jenkins retest this please

@joscollin joscollin removed the request for review from a team February 10, 2022 11:48
@joscollin
Copy link
Member Author

jenkins render docs

@joscollin
Copy link
Member Author

jenkins retest this please

@joscollin
Copy link
Member Author

joscollin commented Feb 10, 2022 •
edited

@vshankar I don't know how to make the docs/readthedocs build successful. But these changes build successfully in my local repo (admin/build-doc).

@vshankar
Copy link
Contributor

IRC chat:

<jcollin> overclk: will do that soon. By the way, -n and -i options lead to ceph-mds crash at present. global_init() crashes.

Where does it exactly crash?

@vshankar vshankar requested a review from a team February 10, 2022 14:32
@vshankar
Copy link
Contributor

@vshankar I don't know how to make the docs/readthedocs build successful. But these changes build successfully in my local repo (admin/build-doc).

probably rebase and update.

@joscollin
doc: mention MDS name and ID format in ceph-mds.rst
3404d4c 
Signed-off-by: Jos Collin <jcollin@redhat.com>
@joscollin
Copy link
Member Author

Where does it exactly crash?

It was a misunderstanding of the defaults. There was no crash. Sorry, my mistake.

@joscollin
Copy link
Member Author

@vshankar I don't know how to make the docs/readthedocs build successful. But these changes build successfully in my local repo (admin/build-doc).

probably rebase and update.

Yeah, that worked.

@vshankar vshankar merged commit 2de993b into ceph:master Feb 14, 2022
11 checks passed
@joscollin joscollin deleted the wip-doc-fix-name-id branch February 14, 2022 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vshankar vshankar vshankar approved these changes

Assignees
No one assigned
Labels
cephfs Ceph File System documentation
Projects
None yet
Milestone
No milestone
2 participants
@joscollin @vshankar