Skip to content

nautilus: mgr: add mon metada using type of "mon"#40359

Merged
yuriw merged 1 commit intoceph:nautilusfrom
tchaikov:nautilus-pr-39937
Apr 12, 2021
Merged

nautilus: mgr: add mon metada using type of "mon"#40359
yuriw merged 1 commit intoceph:nautilusfrom
tchaikov:nautilus-pr-39937

Conversation

@tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Mar 24, 2021

backport of #39937

backport ticket: https://tracker.ceph.com/issues/49705


this change addresses a regression introduced by
c037f4c

also remove the "P" before the json command.

see also: https://tracker.ceph.com/issues/48905

Fixes: https://tracker.ceph.com/issues/49661
Signed-off-by: Kefu Chai kchai@redhat.com
(cherry picked from commit 8fc290b)

Conflicts:
src/mgr/Mgr.cc: trivial resolution

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

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 api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

this change addresses a regression introduced by
c037f4c

also remove the "P" before the json command.

see also: https://tracker.ceph.com/issues/48905

Fixes: https://tracker.ceph.com/issues/49661
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 8fc290b)

 Conflicts:
	src/mgr/Mgr.cc: trivial resolution
@github-actions github-actions bot added this to the nautilus milestone Mar 24, 2021
@tchaikov tchaikov added nautilus-batch-1 nautilus point releases needs-qa labels Mar 24, 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
@smithfarm
Copy link
Contributor

jenkins test signed

@smithfarm
Copy link
Contributor

jenkins test make check

Copy link
Member

@ideepika ideepika left a comment

Choose a reason for hiding this comment

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

@smithfarm
Copy link
Contributor

The following tests FAILED:
	 57 - unittest_lockdep (Timeout)

@smithfarm
Copy link
Contributor

jenkins test make check

@smithfarm
Copy link
Contributor

@tchaikov Am I seeing correctly that the commit that introduced this bug was released as part of Nautilus 14.2.17? I'm seeing that here: https://tracker.ceph.com/issues/49005

Is this a problem for users of 14.2.17, 14.2.18, and 14.2.19? Would you recommend waiting for this fix to get released before upgrading to these versions?

@tchaikov
Copy link
Contributor Author

@tchaikov Am I seeing correctly that the commit that introduced this bug was released as part of Nautilus 14.2.17? I'm seeing that here: https://tracker.ceph.com/issues/49005

yes.

Is this a problem for users of 14.2.17, 14.2.18, and 14.2.19? Would you recommend waiting for this fix to get released before upgrading to these versions?

ideally, yes.

@yuriw yuriw merged commit 533a855 into ceph:nautilus Apr 12, 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.

4 participants