Skip to content

Commit

Permalink
k8s: clarify v3 always refers to the integration
Browse files Browse the repository at this point in the history
  • Loading branch information
roobre committed Jan 31, 2022
1 parent 93edcb2 commit b8a72ad
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ Please note that this only applies to the API Server and that etcd, the schedule

This section covers how to configure control plane monitoring on versions 2 and earlier of the integration.

Please note that these versions had a less flexible autodiscovery options, and did not support external endpoints. We strongly recommend you to update to the [version 3](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3) at your earliest convenience. [See what's changed](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3).
Please note that these versions had a less flexible autodiscovery options, and did not support external endpoints. We strongly recommend you to update to [version 3](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3) at your earliest convenience. [See what's changed](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3) of the Kubernetes integration.

<CollapserGroup>
<Collapser
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ newrelic-infrastructure:
```

<Callout variant="tip">
Notice we are specifying `--tls --port 10250`. Earlier versions of the integration might have worked without this, but it is mandatory since [version 3](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3) as we now connect to the Kubelet using the nodeIP rather than `localhost`.
Notice we are specifying `--tls --port 10250`. Earlier versions of the integration might have worked without this, as of [version 3](/docs/kubernetes-pixie/kubernetes-integration/get-started/changes-since-v3) of the Kubernetes integration it is mandatory to specify them. The reason for this change is that the integration now connects to the Kubelet using the nodeIP rather than `localhost`, with the former requiring TLS while the latter did not.
</Callout>

Integrations targeting other pods should have their own section next to `redis-sampleapp.yaml`.
Expand Down

0 comments on commit b8a72ad

Please sign in to comment.