Skip to content

nautilus: core: mon/MonCommands: "smart" only needs read permission#31111

Merged
yuriw merged 1 commit intoceph:nautilusfrom
tchaikov:nautilus/42455
Nov 6, 2019
Merged

nautilus: core: mon/MonCommands: "smart" only needs read permission#31111
yuriw merged 1 commit intoceph:nautilusfrom
tchaikov:nautilus/42455

Conversation

@tchaikov
Copy link
Contributor

Fixes: https://tracker.ceph.com/issues/42455
Signed-off-by: Kefu Chai kchai@redhat.com

Conflicts: this change is not cherry-picked from master, because in
master, we had a massive refactor in asok. see
#30859. and #30859 is not backported
to nautilus.

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 crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs

Fixes: https://tracker.ceph.com/issues/42455
Signed-off-by: Kefu Chai <kchai@redhat.com>

Conflicts: this change is not cherry-picked from master, because in
master, we had a massive refactor in asok. see
ceph#30859. and ceph#30859 is not backported
to nautilus.
@tchaikov tchaikov added this to the nautilus milestone Oct 24, 2019
@smithfarm smithfarm added core nautilus-batch-1 nautilus point releases needs-qa labels Oct 24, 2019
@smithfarm smithfarm changed the title nautilus: mon/MonCommands: "smart" only needs read permission nautilus: core: mon/MonCommands: "smart" only needs read permission Oct 24, 2019
@yuriw
Copy link
Contributor

yuriw commented Nov 5, 2019

@yuriw yuriw merged commit e5e7b9f into ceph:nautilus Nov 6, 2019
@tchaikov tchaikov deleted the nautilus/42455 branch November 11, 2019 09:42
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.

5 participants