-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
namespace: adding namespace to all rook-ceph namespaces references #13078
Conversation
90c45b7
to
f325318
Compare
@@ -9,4 +9,4 @@ spec: | |||
schedulingInterval: "12m" | |||
schedulingStartTime: "16:18:43" | |||
replication.storage.openshift.io/replication-secret-name: rook-csi-rbd-provisioner | |||
replication.storage.openshift.io/replication-secret-namespace: rook-ceph | |||
replication.storage.openshift.io/replication-secret-namespace: rook-ceph # namespace:cluster |
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.
@Rakshith-R This is the csi driver namespace, right? So it would be the operator
tag.
replication.storage.openshift.io/replication-secret-namespace: rook-ceph # namespace:cluster | |
replication.storage.openshift.io/replication-secret-namespace: rook-ceph # namespace:operator |
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.
well, let's wait for the confirmation from @Rakshith-R but as I can see on my local cluster all the csi related config is part of the operator namespace.
d3c9e5b
to
d1a13b2
Compare
To make it easier for the user to switch to a namespace other than the default 'rook-ceph', the tag '# namespace:cluster' must be inserted in all the places where this namespace is used. This way, a simple 'sed' command can update all the Yaml files to the new namespace. Signed-off-by: Redouane Kachach <rkachach@redhat.com>
60840f1
to
ff081cc
Compare
namespace: adding namespace to all rook-ceph namespaces references (backport #13078)
Description of your changes:
To make it easier for the user to switch to a namespace other than the default 'rook-ceph', the tag '# namespace:cluster' must be inserted in all the places where this namespace is used. This way, a simple 'sed' command can update all the Yaml files to the new namespace.
related docs: https://rook.github.io/docs/rook/v1.9/ceph-advanced-configuration.html
Which issue is resolved by this Pull Request:
Resolves #
Checklist: