Skip to content

ceph: add permissions to rook-ceph-mgr role for osd removal in rook orchestator#8568

Merged
travisn merged 1 commit intorook:masterfrom
josephsawaya:rook-ceph-mgr-removal-role
Aug 19, 2021
Merged

ceph: add permissions to rook-ceph-mgr role for osd removal in rook orchestator#8568
travisn merged 1 commit intorook:masterfrom
josephsawaya:rook-ceph-mgr-removal-role

Conversation

@josephsawaya
Copy link
Contributor

Description of your changes:
This commit adds permissions to the rook-ceph-mgr role to delete PVCs and deployments and patch deployment scale
to remove OSDs using ceph orch osd rm in the rook orchestrator.

Which issue is resolved by this Pull Request:
Resolves #

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
  • Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
  • Reviewed the developer guide on Submitting a Pull Request
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.
  • Pending release notes updated with breaking and/or notable changes, if necessary.
  • Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
  • Code generation (make codegen) has been run to update object specifications, if necessary.

@mergify mergify bot added the ceph main ceph tag label Aug 19, 2021
- deployments/scale
verbs:
- patch
- apiGroups:
Copy link
Member

Choose a reason for hiding this comment

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

It seems fine to combine this with the previous one so that deployments/scale and deployments can have both patch and delete access.

This commit adds permissions to the rook-ceph-mgr role
to delete PVCs and deployments and patch deployment scale
to remove OSDs using `ceph orch osd rm` in the rook orchestrator.

Signed-off-by: Joseph Sawaya <jsawaya@redhat.com>
@travisn travisn merged commit 8565627 into rook:master Aug 19, 2021
mergify bot added a commit that referenced this pull request Aug 19, 2021
ceph: add permissions to rook-ceph-mgr role for osd removal in rook orchestator (backport #8568)
parth-gr pushed a commit to parth-gr/rook that referenced this pull request Aug 20, 2021
…role

ceph: add permissions to rook-ceph-mgr role for osd removal in rook orchestator
Signed-off-by: parth-gr <paarora@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ceph main ceph tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants