-
Notifications
You must be signed in to change notification settings - Fork 23
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
make storageclient api to clusterscope resource #133
make storageclient api to clusterscope resource #133
Conversation
reviewing on per commit basis will be good. /hold testing in progress as this needs changes at provider end as well to test. |
5631944
to
7a14a0d
Compare
/hold There's one more point in my saved notes around claim deletion, will be posting that commit soon. |
/unhold |
2454f7f
to
3e47d32
Compare
fixed commit lint. |
Signed-off-by: Leela Venkaiah G <[email protected]>
Signed-off-by: Leela Venkaiah G <[email protected]>
Signed-off-by: Leela Venkaiah G <[email protected]>
3bdf9ef
to
bd6beef
Compare
Signed-off-by: Leela Venkaiah G <[email protected]>
Signed-off-by: Leela Venkaiah G <[email protected]>
Signed-off-by: Leela Venkaiah G <[email protected]>
bd6beef
to
ab89e87
Compare
Signed-off-by: Leela Venkaiah G <[email protected]>
ab89e87
to
c893ec3
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: leelavg, nb-ohad 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 |
05917a6
into
red-hat-storage:main
as of now storageclient api is namespaced w/ no known benefits that could outweigh ease of operations performed in controllers if it can be cluster scoped.
this PR makes storageclient to clusterscope and majorly benefits from existing tracking of clusterscoped resource (storageclaim) by namespaced resource (existing storageclient) replaced w/ owner references.
design wise, no two storageclient even if namespaced shouldn't be allowed to connect to single provider and as such storageclient better be clusterscoped and only con is for user to choose explicit names to avoid collision and that'd be less of an issue when compared to all the plumbing that is being done in controllers as of now.