Skip to content
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.

Sign up for GitHub

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

Update/Fix Istioctl links and docs #349

Merged
merged 3 commits into from
Oct 1, 2024
Merged

Update/Fix Istioctl links and docs #349

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2f52f51
Update/Fix istioctl links in README.md
longmuir Sep 19, 2024
21ebb8c
Update install-istioctl-tool.md to clarify steps and link to upstream…
longmuir Sep 19, 2024
d8330f6
remove namespace from get istio command
longmuir Sep 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions docs/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ When the `InPlace` strategy is used, the existing Istio control plane is replace

Prerequisites:
* Sail Operator is installed.
* `istioctl` is [installed](common/istio-addons-integrations.md).
* `istioctl` is [installed](common/install-istioctl-tool.md).

Steps:
1. Create the `istio-system` namespace.
Expand Down Expand Up @@ -336,7 +336,7 @@ When the `RevisionBased` strategy is used, a new Istio control plane instance is

Prerequisites:
* Sail Operator is installed.
* `istioctl` is [installed](common/istio-addons-integrations.md).
* `istioctl` is [installed](common/install-istioctl-tool.md).

Steps:

Expand Down Expand Up @@ -476,7 +476,7 @@ You can use the Sail Operator and the Sail CRDs to manage a multi-cluster Istio

Each deployment model requires you to install the Sail Operator and the Sail CRDs to every cluster that is part of the mesh.

- Install [istioctl](https://istio.io/latest/docs/setup/install/istioctl) and have it included in your `$PATH`.
- Install [istioctl](common/install-istioctl-tool.md).
- Two kubernetes clusters with external lb support. (If using kind, `cloud-provider-kind` is running in the background)
- kubeconfig file with a context for each cluster.

Expand Down
7 changes: 3 additions & 4 deletions docs/common/install-istioctl-tool.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ operators to debug and diagnose Istio service mesh deployments.

Use an `istioctl` version that is the same version as the Istio control plane
for the Service Mesh deployment. See [Istio Releases](https://github.com/istio/istio/releases) for a list of valid
releases, including Beta releases.

releases, including Beta releases.

### Procedure

Expand All @@ -24,7 +23,7 @@ the following command at the terminal:
at the terminal:

```sh
$ oc -n istio-system get istio
$ oc get istio
```

3. Install `istioctl` by running the following command at the terminal:
Expand All @@ -47,6 +46,6 @@ at the terminal:
```sh
$ istioctl version
```

For more information on usage, see the [Istioctl documentation](https://istio.io/latest/docs/ops/diagnostic-tools/istioctl/).

*Note*: `istioctl install` is not supported. The Sail Operator installs Istio.