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

doc/rados/ops: edit health-checks.rst (1 of 5) #50674

Conversation

zdover23
Copy link
Contributor

@zdover23 zdover23 commented Mar 25, 2023

Edit docs/rados/operations/health-checks.rst (1 of 5).

https://tracker.ceph.com/issues/58485

Contribution Guidelines

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
  • jenkins test windows

Copy link
Contributor

@anthonyeleven anthonyeleven left a comment

Choose a reason for hiding this comment

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

I've made the usual complement of pedantic suggestions. Incorporate as you see fit.

doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
doc/rados/operations/health-checks.rst Outdated Show resolved Hide resolved
Edit docs/rados/operations/health-checks.rst (1 of x).

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
@zdover23 zdover23 force-pushed the wip-doc-2023-03-26-rados-operations-health-checks-1-of-5 branch from 6a0546f to cb1e20f Compare March 30, 2023 01:28
@zdover23 zdover23 merged commit 37280a3 into ceph:main Mar 30, 2023
10 of 11 checks passed
@zdover23
Copy link
Contributor Author

#50796 - Reef backport
#50797 - Quincy backport
#50798 - Pacific backport

zdover23 added a commit to zdover23/ceph that referenced this pull request Apr 1, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
zdover23 added a commit to zdover23/ceph that referenced this pull request Apr 6, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
zdover23 added a commit to zdover23/ceph that referenced this pull request Apr 6, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 0565f59)
zdover23 added a commit to zdover23/ceph that referenced this pull request Apr 6, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 0565f59)
zdover23 added a commit to zdover23/ceph that referenced this pull request Apr 6, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
(cherry picked from commit 0565f59)
guits pushed a commit to guits/ceph that referenced this pull request May 22, 2023
Edit docs/rados/operations/health-checks.rst (2 of x). PR#50674, the PR
that immediately precedes this PR in the series of PRs that line-edit
health-checks.rst, wrongly identified this series as having five
sections. This has been rectified by using the "2 of x" formulation.

Follows ceph#50674

https://tracker.ceph.com/issues/58485

Signed-off-by: Zac Dover <zac.dover@proton.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants