Skip to content

Commit 806afda

Browse files
Legacy service accounts: Update log6x-collection-setup.adoc
- More than one command is mentioned in the single code block - Here is the documentation link [1]: [1] https://docs.openshift.com/container-platform/4.18/observability/logging/logging-6.0/log6x-clf.html#legacy-service-accounts - As per the standard rule, using more than one command per code block is not recommended. - Please check Standard rule [2] for reference: [2] https://github.com/openshift/openshift-docs/blob/main/contributing_to_docs/doc_guidelines.adoc#single-command-per-code-block - When commands are bunched together, the copy to clipboard functionality might not break the lines up correctly. Using a single command per code block makes it copy-and-paste friendly. - Hence we need to mention 2 separate code blocks for above 2 commands. - Here is the updated look of the documentation: ~~~ $ oc adm policy add-cluster-role-to-user collect-application-logs system:serviceaccount:openshift-logging:logcollector ~~~ ~~~ $ oc adm policy add-cluster-role-to-user collect-infrastructure-logs system:serviceaccount:openshift-logging:logcollector ~~~
1 parent a8f1d57 commit 806afda

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

modules/log6x-collection-setup.adoc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ To use the existing legacy service account `logcollector`, create the following
1818
[source,terminal]
1919
----
2020
$ oc adm policy add-cluster-role-to-user collect-application-logs system:serviceaccount:openshift-logging:logcollector
21+
----
22+
23+
[source,terminal]
24+
----
2125
$ oc adm policy add-cluster-role-to-user collect-infrastructure-logs system:serviceaccount:openshift-logging:logcollector
2226
----
2327

0 commit comments

Comments
 (0)