Skip to content

pybind/mgr/cephadm: remove mds auth cap for nfs recovery db cred#35159

Merged
sebastian-philipp merged 1 commit intoceph:masterfrom
batrick:i45632
May 27, 2020
Merged

pybind/mgr/cephadm: remove mds auth cap for nfs recovery db cred#35159
sebastian-philipp merged 1 commit intoceph:masterfrom
batrick:i45632

Conversation

@batrick
Copy link
Member

@batrick batrick commented May 20, 2020

There's no reason for this cap to be used for talking to the MDS. The
individual exports have their own caps for talking to the file system.

Fixes: https://tracker.ceph.com/issues/45632
Signed-off-by: Patrick Donnelly pdonnell@redhat.com

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

There's no reason for this cap to be used for talking to the MDS. The
individual exports have their own caps for talking to the file system.

Fixes: https://tracker.ceph.com/issues/45632
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
@sebastian-philipp
Copy link
Contributor

thanks for the fix!

@mgfritch
Copy link
Contributor

@sebastian-philipp sebastian-philipp added the wip-swagner-testing My Teuthology tests label May 27, 2020
@sebastian-philipp
Copy link
Contributor

@sebastian-philipp sebastian-philipp merged commit d45c6fa into ceph:master May 27, 2020
@batrick batrick deleted the i45632 branch May 27, 2020 19:53
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