controllers: send the client profile mapping required for DR #2879
Conversation
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: rewantsoni The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
rewantsoni
force-pushed
the
clientProfile
branch
3 times, most recently
from
350e6bf
to
fbc1c87
Compare
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
| ) | ||
|
|
||
| const ( | ||
| monConfigMap = "rook-ceph-mon-endpoints" | ||
| monSecret = "rook-ceph-mon" | ||
| ) | ||
|
|
||
| const peerPoolIDConfigMapName = "peer-pool-id" |
There was a problem hiding this comment.
How about moving this above after line 60.
| @@ -16,6 +16,7 @@ rules: | |||
| resources: | |||
| - cephfilesystemsubvolumegroups | |||
| - cephblockpoolradosnamespaces | |||
| - cephblockpools | |||
| verbs: | |||
| - get | |||
| - list | |||
There was a problem hiding this comment.
For the Ceph block pool, you only need to get access. I am not sure Is it fair to add a new entry for the same API group?
openshift-merge-robot
added
the
needs-rebase
rewantsoni
force-pushed
the
clientProfile
branch
from
fbc1c87
to
45e6d61
Compare
openshift-merge-robot
removed
the
needs-rebase
this mapping is sent only if rns is enabled for mirroring remote poolID is fetched from GetBlockPoolInfo rpc, which is then written to a configMap to be read in provider server local poolID is fetched from cephBlockPool status Signed-off-by: Rewant Soni <[email protected]>
rewantsoni
force-pushed
the
clientProfile
branch
from
45e6d61
to
a84e6fb
Compare
This mapping is sent only if rns is enabled for mirroring