-
Notifications
You must be signed in to change notification settings - Fork 218
docs: standardize CLI examples and correct linkerd-cni namespace references #2082
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
base: main
Are you sure you want to change the base?
Changes from all commits
8bfe2cf
36f176a
37bdbd4
6601e95
d3f44f8
32490a5
5b85d89
38f6d7a
679e4bd
c7a1dc8
0e2ad0b
9585281
799dd23
970d2ce
90bdda4
958f5f3
2675fc2
43d883c
d8314a2
394b88f
5de2a73
6d336d1
c034ec2
cd2f3be
1b2565c
90f7e61
bb76891
4d8987c
46dfe3b
cadd025
f8bd8a1
7267702
000807a
8f0d56c
b1c5bac
2cf1b05
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1789,12 +1789,12 @@ Make sure that the `proxy-injector` is working correctly by running | |
| Ensure all the prometheus related resources are present and running correctly. | ||
|
|
||
| ```bash | ||
| ❯ kubectl -n linkerd-viz get deploy,cm | grep prometheus | ||
| $ kubectl -n linkerd-viz get deploy,cm | grep prometheus | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Replaced the ❯ prompt with $ to keep shell examples consistent across the docs. |
||
| deployment.apps/prometheus 1/1 1 1 3m18s | ||
| configmap/prometheus-config 1 3m18s | ||
| ❯ kubectl get clusterRoleBindings | grep prometheus | ||
| $ kubectl get clusterRoleBindings | grep prometheus | ||
| linkerd-linkerd-viz-prometheus ClusterRole/linkerd-linkerd-viz-prometheus 3m37s | ||
| ❯ kubectl get clusterRoles | grep prometheus | ||
| $ kubectl get clusterRoles | grep prometheus | ||
| linkerd-linkerd-viz-prometheus 2021-02-26T06:03:11Zh | ||
| ``` | ||
|
|
||
|
|
@@ -1810,7 +1810,7 @@ Example failure: | |
| Verify that the metrics API pod is running correctly | ||
|
|
||
| ```bash | ||
| ❯ kubectl -n linkerd-viz get pods | ||
| $ kubectl -n linkerd-viz get pods | ||
| NAME READY STATUS RESTARTS AGE | ||
| metrics-api-7bb8cb8489-cbq4m 2/2 Running 0 4m58s | ||
| tap-injector-6b9bc6fc4-cgbr4 2/2 Running 0 4m56s | ||
|
|
@@ -2166,7 +2166,7 @@ Agent version: v0.4.4 | |
| To update to the latest version: | ||
|
|
||
| ```bash | ||
| linkerd-buoyant install | kubectl apply -f - | ||
| $ linkerd-buoyant install | kubectl apply -f - | ||
| ``` | ||
|
|
||
| ### √ buoyant-cloud-agent Deployment is running a single pod | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -375,7 +375,7 @@ remove the Helm release config for the old `linkerd2` chart (assuming you used | |
| the "Secret" storage backend, which is the default): | ||
|
|
||
| ```bash | ||
| $ kubectl -n default delete secret \ | ||
| kubectl -n default delete secret \ | ||
| --field-selector type=helm.sh/release.v1 \ | ||
| -l name=linkerd,owner=helm | ||
| ``` | ||
|
|
@@ -555,7 +555,7 @@ chart or installing the Linkerd-Viz chart. See below for a complete list of | |
| values which have moved. | ||
|
|
||
| ```bash | ||
| helm repo update | ||
| helm repo up | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated |
||
| # Upgrade the control plane (this will remove viz components). | ||
| helm upgrade linkerd2 linkerd/linkerd2 --reset-values -f values.yaml --atomic | ||
| # Install the Linkerd-Viz extension to restore viz functionality. | ||
|
|
||
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.
This block contains command-only examples; both commands can be executed sequentially.