Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Add VolumeSnapshot related RBACs to provider service account for TKC/GC #4491

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

akankshapanse
Copy link
Contributor

What this PR does / why we need it

This PR adds RBAC rules for volume snapshot objects to provider service account when CSI controller is running in paravirtual mode. That provides the guest cluster with all the permissions to operate volume snapshot objects.
This is being done for adding snapshot support in WCP.

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

Release note

Add VolumeSnapshot related RBACs to provider service account for TKC/GC

Additional information

Special notes for your reviewer

@codecov
Copy link

codecov bot commented Mar 17, 2023

Codecov Report

Merging #4491 (c4f7be0) into main (36dadba) will decrease coverage by 0.93%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main    #4491      +/-   ##
==========================================
- Coverage   49.70%   48.78%   -0.93%     
==========================================
  Files         452      482      +30     
  Lines       45152    47272    +2120     
==========================================
+ Hits        22445    23061     +616     
- Misses      20582    22032    +1450     
- Partials     2125     2179      +54     
Impacted Files Coverage Δ
tkg/client/upgrade_region.go 26.62% <0.00%> (ø)
tkg/clusterclient/clusterclient.go 49.14% <33.33%> (ø)

... and 34 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@akankshapanse akankshapanse changed the title [WIP] Add VolumeSnapshot related RBACs to provider service account for TKC/GC Add VolumeSnapshot related RBACs to provider service account for TKC/GC Mar 21, 2023
Copy link
Contributor

@jvrahav jvrahav left a comment

Choose a reason for hiding this comment

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

LGTM !!

@akankshapanse
Copy link
Contributor Author

/test install-vc7
/test upgrade-vc7

@jeffwubj jeffwubj added the ok-to-merge PRs should be labelled with this before merging label Mar 24, 2023
@jeffwubj jeffwubj merged commit 12c4dab into main Mar 24, 2023
@jeffwubj jeffwubj deleted the add_snapshot_rbac branch March 24, 2023 05:43
@alfredthenarwhal
Copy link
Collaborator

@akankshapanse: /test install-vc7 /test upgrade-vc7
Commit: c4f7be0

Tests failed! Build no: 4550

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants