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

osd: Add cephcluster status for deprecated osds to replace (backport #14187) #14195

Merged
merged 1 commit into from May 13, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented May 13, 2024

To follow up from #14100, we add status to the cephcluster CR that at least gives a clue that the OSDs should be replaced.
LVM-based OSDs on PVCs are legacy and are not well tested, so this is a small attempt to raise that fact in the status, even while we still attempt to keep them running.

This will result in status on the CephCluster such as:

  status:
    storage:
      deviceClasses:
      - name: hdd
      legacyOSDs:
      - id: 0
        reason: LVM-based OSD on a PVC (id=0) is deprecated and should be replaced
      - id: 1
        reason: LVM-based OSD on a PVC (id=1) is deprecated and should be replaced
      - id: 2
        reason: LVM-based OSD on a PVC (id=2) is deprecated and should be replaced
      osd:
        storeType:
          bluestore: 3

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

This is an automatic backport of pull request #14187 done by [Mergify](https://mergify.com).

LVM-based OSDs on PVCs are legacy and are not well tested.
Therefore, we add status to the cephcluster CR that at least
gives a clue that the OSDs should be replaced.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 3628e83)
@mergify mergify bot merged commit 4d075a6 into release-1.14 May 13, 2024
53 checks passed
@mergify mergify bot deleted the mergify/bp/release-1.14/pr-14187 branch May 13, 2024 14:59
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 this pull request may close these issues.

None yet

2 participants