Bug 1839039

Summary: [DDF] Why is this note to override cluster-wide default node selector in this section?
Product: [Red Hat Storage] Red Hat OpenShift Container Storage Reporter: Direct Docs Feedback <ddf-bot>
Component: documentationAssignee: Disha Walvekar <dwalveka>
Status: CLOSED CURRENTRELEASE QA Contact: Neha Berry <nberry>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 4.3CC: aclewett, asriram, bkunal, dwalveka, nberry, ocs-bugs
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-19 06:50:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1885167    

Comment 3 Neha Berry 2020-10-06 06:27:40 UTC
Hi bipin,

Not sure when it was planned, but the reason for having the Note in 2 places is :

1. 1st Note Before Creation of Storagecluster(1.1.1. Installing Red Hat OpenShift Container Storage Operator using the Operator Hub)

>> It was added so as to override cluster-wide default node selector before creation of Storage cluster. This step does not involve respinning CSI pods as they are not yet created

2. 2nd Note (2.1. Verify that the pods are in running state)

>> This was added for users who want to override the cluster-wide default selector after Storage Cluster is already created. This step involves the extra step to respin CSI pods as well.


IMO, we need some changes in the statement to clearly point out the reason for adding the notes in these 2 sections. Also, need to confirm from Annette if these commands are needed in OCS 4.5 and above ?

Comment 6 Neha Berry 2020-10-08 07:32:12 UTC
Hi Disha,

I am OK with Bipin's suggestion to move the content from Verifying pods to some other section.

But we would definitely need to keep both the content, or join them together, with a note : One is for pre-install of OCS and the 2nd one is for changing the selector after OCS is already installed.

Comment 13 Neha Berry 2020-10-14 15:43:01 UTC
Thank You so much Disha for including the change in all these docs listed in Comment#9

Verified that the following content is removed from Verifying the state of the pods -> Point #2. Moving to verified.

NOTE
When you need to override the cluster-wide default node selector for OpenShift Container Storage, you can perform the following steps through the command line interface:

Specify a blank node selector for the openshift-storage namespace.

$ oc annotate namespace openshift-storage openshift.io/node-selector=
Delete the original pods generated by the DaemonSets.

oc delete pod -l app=csi-cephfsplugin -n openshift-storage
oc delete pod -l app=csi-rbdplugin -n openshift-storage


Verified that the new content is added to the troubleshooting guide of all the versions (4.3, 4.4, 4.5). We need to decide if we need it in 4.6 as well. we will check and raise a new BZ if needed for the same

````````

When a cluster-wide default node selector is used for Openshift Container Storage, the pods generated by CSI daemonsets are able to start only on the nodes that match the selector. To be able to use Openshift Container Storage from nodes which do not match the selector, override the cluster-wide default node selector by performing the following steps in the command line interface :

Procedure

Specify a blank node selector for the openshift-storage namespace.

$ oc annotate namespace openshift-storage openshift.io/node-selector=
Delete the original pods generated by the DaemonSets.

oc delete pod -l app=csi-cephfsplugin -n openshift-storage
oc delete pod -l app=csi-rbdplugin -n openshift-storage

``````