Bug 1929319

Summary: Indentation issue in cluster logging installation guide
Product: OpenShift Container Platform Reporter: Andreas Karis <akaris>
Component: DocumentationAssignee: landerso
Status: CLOSED CURRENTRELEASE QA Contact: Xiaoli Tian <xtian>
Severity: unspecified Docs Contact: Claire Bremble <cbremble>
Priority: unspecified    
Version: 4.6CC: aos-bugs, lmurthy, stk
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-02-27 17:04:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Andreas Karis 2021-02-16 16:37:01 UTC
Document URL: 
https://docs.openshift.com/container-platform/4.6/logging/cluster-logging-deploying.html

Section Number and Name: 
Under 5. Create a Cluster Logging instance

Describe the issue: 

Indentation issue for CR ClusterLogging.
~~~
[root@openshift-jumpserver-0 ~]# oc apply -f cl.yaml 
error: error parsing cl.yaml: error converting YAML to JSON: yaml: line 16: did not find expected key
~~~


Suggestions for improvement:
Correctly indent 4.6 ClusterLogging CR (look at the 4.5 guide where it's correct) 

Additional information: 

Compare https://docs.openshift.com/container-platform/4.6/logging/cluster-logging-deploying.html with bad indentation
to 
https://docs.openshift.com/container-platform/4.5/logging/cluster-logging-deploying.html with valid indentation

~~~
[root@openshift-jumpserver-0 ~]# diff 45.yaml 46.yaml  -y
apiVersion: "logging.openshift.io/v1"				apiVersion: "logging.openshift.io/v1"
kind: "ClusterLogging"						kind: "ClusterLogging"
metadata:							metadata:
  name: "instance" 						  name: "instance" 
  namespace: "openshift-logging"				  namespace: "openshift-logging"
spec:								spec:
  managementState: "Managed"  					  managementState: "Managed"  
  logStore:							  logStore:
    type: "elasticsearch"  					    type: "elasticsearch"  
    retentionPolicy: 						    retentionPolicy: 
      application:						      application:
        maxAge: 1d						        maxAge: 1d
      infra:							      infra:
        maxAge: 7d						        maxAge: 7d
      audit:							      audit:
        maxAge: 7d						        maxAge: 7d
    elasticsearch:					      |	   elasticsearch:
      nodeCount: 3 						      nodeCount: 3 
      storage:							      storage:
        storageClassName: "<storage-class-name>" 		        storageClassName: "<storage-class-name>" 
        size: 200G						        size: 200G
        resources: 					      |	      resources: 
          requests:					      |	        requests:
            memory: "8Gi"				      |	          memory: "8Gi"
        proxy: 						      |	      proxy: 
          limits:					      |	        limits:
            memory: 256Mi				      |	          memory: 256Mi
          requests:					      |	        requests:
             memory: 256Mi				      |	           memory: 256Mi
      redundancyPolicy: "SingleRedundancy"			      redundancyPolicy: "SingleRedundancy"
  visualization:						  visualization:
    type: "kibana"  						    type: "kibana"  
    kibana:							    kibana:
      replicas: 1						      replicas: 1
  curation:							  curation:
    type: "curator"						    type: "curator"
    curator:							    curator:
      schedule: "30 3 * * *" 					      schedule: "30 3 * * *" 
  collection:							  collection:
    logs:							    logs:
      type: "fluentd"  						      type: "fluentd"  
      fluentd: {}						      fluentd: {}
~~~

Comment 3 Andreas Karis 2023-02-27 17:04:38 UTC
lgtm