Skip to content

squid: qa: correct daemon for warning conf #61304

Merged
rishabh-d-dave merged 1 commit intoceph:squidfrom
batrick:wip-68244-squid
Oct 14, 2025
Merged

squid: qa: correct daemon for warning conf #61304
rishabh-d-dave merged 1 commit intoceph:squidfrom
batrick:wip-68244-squid

Conversation

@batrick
Copy link
Member

@batrick batrick commented Jan 9, 2025

Note: this backport drops qa: print config dump after setting because of conflicts and it's not necessary.

backport tracker: https://tracker.ceph.com/issues/68244


backport of #59467
parent tracker: https://tracker.ceph.com/issues/67737

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

The mgr emits this health check instead of the mons.

Fixes: https://tracker.ceph.com/issues/67737
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 9aee68a)

Conflicts:
	qa/cephfs/conf/mon.yaml: no cluster-conf
@batrick batrick added this to the squid milestone Jan 9, 2025
@batrick batrick added tests cephfs Ceph File System labels Jan 9, 2025
@batrick
Copy link
Member Author

batrick commented Jan 10, 2025

jenkins test make check

@kotreshhr
Copy link
Contributor

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

@joscollin
Copy link
Member

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

Copy link
Contributor

@rishabh-d-dave rishabh-d-dave left a comment

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants