New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUILD-353: update with latest CRD,group,csi driver renames #914
BUILD-353: update with latest CRD,group,csi driver renames #914
Conversation
/assign @adambkaplan |
66a47e1
to
4ec2c3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
Want someone to double-check, I think this looks right.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adambkaplan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/assign @coreydaley please provide the double-check @adambkaplan asks .... mostly lines up with your openshift/api work |
@@ -164,16 +162,28 @@ An admin creates a new cluster level custom resource for encapsulating the shara | |||
|
|||
|
|||
```yaml | |||
kind: ShareResource | |||
apiVersion: storage.openshift.io/v1alpha1 | |||
kind: ShareConfigMap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kind: ShareConfigMap | |
kind: SharedConfigMap |
``` | ||
|
||
```yaml | ||
kind: ShareSecret |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kind: ShareSecret | |
kind: SharedSecret |
resources: | ||
- shareresources | ||
- sharedsecrets | ||
- sharedconfigmaps | ||
verbs: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add watch
here also?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
4ec2c3b
to
0155b6b
Compare
updates pushed @coreydaley thanks ! |
/lgtm |
Update on the names or the various CRD/group/driver names for the sharedconfigmap / sharedsecret / sharedresource following the meeting between myself and
@deads2k
@adambkaplan
@bparees
@jsafrane
@coreydaley
on Sep 24 2021
@sbose78 @siamaksade FYI