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

nautilus: mgr/telemetry: check if 'ident' channel is active #39923

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm commented Mar 8, 2021

NOTE: Backport omits eb3b27c which was also omitted from the pacific backport PR #39727

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


backport of #39538
parent tracker: https://tracker.ceph.com/issues/49349

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

When compiling the telemetry reports we check for active channels. The
'ident' channel check was missing, hence even if users turned this
channel on, its content could not be appended to the report.

Fixes: https://tracker.ceph.com/issues/49349
Signed-off-by: Yaarit Hatuka <yaarit@redhat.com>
(cherry picked from commit 2484919)
@smithfarm smithfarm added this to the nautilus milestone Mar 8, 2021
@smithfarm smithfarm added the core label Mar 8, 2021
@smithfarm
Copy link
Contributor Author

Test failures:

	 33 - unittest_mempool (Failed)

@smithfarm
Copy link
Contributor Author

jenkins test make check

@smithfarm smithfarm added nautilus-batch-1 nautilus point releases needs-qa labels Mar 22, 2021
@jdurgin jdurgin changed the base branch from nautilus to nautilus-saved March 31, 2021 00:14
@jdurgin jdurgin changed the base branch from nautilus-saved to nautilus March 31, 2021 00:14
@yuriw yuriw merged commit bcb1cce into ceph:nautilus Apr 5, 2021
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.

3 participants