-
Notifications
You must be signed in to change notification settings - Fork 20
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
addons: move the fetching of cephfsid only for non-provider mode #248
addons: move the fetching of cephfsid only for non-provider mode #248
Conversation
Testing:
|
/assign @umangachapagain @vbnrh |
/cherrypick release-4.18 |
@rewantsoni: once the present PR merges, I will cherry-pick it on top of In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Signed-off-by: Rewant Soni <[email protected]>
026d0d0
to
0d6d955
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rewantsoni, vbnrh 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 |
/hold for @umangachapagain |
/hold cancel |
1f8102f
into
red-hat-storage:main
@rewantsoni: new pull request created: #249 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Restricting to the fetching of cephFSID for when the MirrorPeer is not for StorageClients
Requeue if MirrorPeer is in Phase: ExchangingSecret