Skip to content

qa: correct daemon for warning conf #59467

Merged
batrick merged 2 commits intoceph:mainfrom
batrick:i67737
Sep 25, 2024
Merged

qa: correct daemon for warning conf #59467
batrick merged 2 commits intoceph:mainfrom
batrick:i67737

Conversation

@batrick
Copy link
Member

@batrick batrick commented Aug 27, 2024 •
edited
Loading

The mgr emits this health check instead of the mons.

Fixes: https://tracker.ceph.com/issues/67737

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

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
  • jenkins test rook e2e

@github-actions github-actions bot added the tests label Aug 27, 2024
@batrick
Copy link
Member Author

batrick commented Aug 27, 2024

jenkins test make check arm64

@batrick
Copy link
Member Author

batrick commented Aug 27, 2024

This PR is under test in https://tracker.ceph.com/issues/67757.

@batrick
Copy link
Member Author

batrick commented Aug 28, 2024

jenkins test make check

1 similar comment
@batrick
Copy link
Member Author

batrick commented Aug 28, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented Aug 29, 2024

/teuthology/pdonnell-2024-08-28_20:52:52-fs-wip-pdonnell-testing-20240828.032152-debug-distro-default-smithi/7878588/teuthology.log

Still present because cephadm doesn't edit the /etc/ceph/ceph.conf apparently.

@batrick batrick self-assigned this Aug 29, 2024
@batrick batrick requested a review from a team as a code owner August 29, 2024 18:14
@batrick
Copy link
Member Author

batrick commented Aug 29, 2024

debugging this

@batrick batrick removed their assignment Aug 29, 2024
@batrick batrick requested a review from a team August 29, 2024 18:19
@batrick batrick changed the title qa: turn off POOL_APP_NOT_ENABLED via ceph.conf qa: correct daemon for warning conf Aug 29, 2024
@batrick
Copy link
Member Author

batrick commented Aug 30, 2024

jenkins test dashboard cephadm

3 similar comments
@batrick
Copy link
Member Author

batrick commented Aug 30, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Aug 30, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 3, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 11, 2024

jenkins test api

@batrick
Copy link
Member Author

batrick commented Sep 11, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented Sep 11, 2024

jenkins test make check arm64

@batrick
Copy link
Member Author

batrick commented Sep 11, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 16, 2024

This PR is under test in https://tracker.ceph.com/issues/68089.

@batrick
Copy link
Member Author

batrick commented Sep 17, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 17, 2024

jenkins test api

1 similar comment
@batrick
Copy link
Member Author

batrick commented Sep 17, 2024

jenkins test api

@batrick
Copy link
Member Author

batrick commented Sep 17, 2024

jenkins test dashboard cephadm

2 similar comments
@batrick
Copy link
Member Author

batrick commented Sep 19, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 20, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Sep 20, 2024

This PR is under test in https://tracker.ceph.com/issues/68170.

@batrick
Copy link
Member Author

batrick commented Sep 25, 2024

https://tracker.ceph.com/issues/68170#note-1

@batrick batrick merged commit 3474820 into ceph:main Sep 25, 2024
@batrick batrick deleted the i67737 branch September 25, 2024 21:02
This was referenced Jan 9, 2025
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adk3798 adk3798 adk3798 approved these changes

Assignees

No one assigned

Labels

cephadm cephfs Ceph File System needs-review tests wip-pdonnell-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@batrick @adk3798