BUG 2078713: csi: add/remove mon IP from csi config #373
Conversation
If a mon is added or removed in a ceph cluster all the CSI clusterID's belongs to the ceph cluster should also gets updated. Added a new cluster identifier to the csi configmap, i.e ceph clusterNamespace which helps to identify the clusterID's belongs to that ceph cluster. Note:- to support the upgraded clusters,we can ask users to update the mon IP's for the problamatic clusterID as it wont be updated by the operator. Signed-off-by: Madhu Rajanna <madhupr007@gmail.com> (cherry picked from commit 8ab17e3)
openshift-ci
bot
added
bugzilla/severity-high
|
@Madhu-1: This pull request references Bugzilla bug 2078713, which is valid. The bug has been updated to refer to the pull request using the external bug tracker. 2 validation(s) were run on this bug
Requesting review from QA contact: 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/test-infra repository. |
|
@openshift-ci[bot]: GitHub didn't allow me to request PR reviews from the following users: nehaberry. Note that only red-hat-storage members and repo collaborators can review this PR, and authors cannot review their own PRs. 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/test-infra repository. |
|
/hold till we get QE ACK. |
openshift-ci
bot
added
the
do-not-merge/hold
|
/assign @travisn |
|
/unhold as we have required acks for the BZ |
|
/hold cancel |
openshift-ci
bot
removed
the
do-not-merge/hold
|
/lgtm |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: leseb, Madhu-1 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 |
openshift-ci
bot
added
the
approved
|
@Madhu-1: All pull requests linked via external trackers have merged: Bugzilla bug 2078713 has been moved to the MODIFIED state. 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/test-infra repository. |
Description of your changes:
If a mon is added or removed in a ceph cluster all the CSI clusterID belongs to the ceph cluster should also get updated.
Added a new cluster identifier to the csi configmap, i.e ceph cluster namespace which helps to identify the clusterID belongs to that ceph cluster.
Note:- To support the upgraded clusters, we can ask users to update the mon IPs for the problamatic clusterID as it wont be updated by the operator.
Signed-off-by: Madhu Rajanna madhupr007@gmail.com
(cherry picked from commit 8ab17e3)
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen) has been run to update object specifications, if necessary.