Skip to content

Commit

Permalink
move ambient tests to ambient profile (istio#15241)
Browse files Browse the repository at this point in the history
Signed-off-by: Faseela K <[email protected]>
  • Loading branch information
kfaseela authored and craigbox committed Jun 26, 2024
1 parent 5e7178e commit 7180428
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 13 deletions.
9 changes: 8 additions & 1 deletion content/en/docs/ambient/getting-started/cleanup/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,19 @@ $ kubectl label namespace default istio.io/dataplane-mode-
$ kubectl label namespace default istio.io/use-waypoint-
{{< /text >}}

## Remove waypoint proxies and uninstall Istio
## Remove waypoint proxies

To remove waypoint proxies, installed policies, and uninstall Istio, run the following commands:

{{< text bash >}}
$ istioctl x waypoint delete --all
{{< /text >}}

## Uninstall Istio

To uninstall Istio:

{{< text syntax=bash snip_id=none >}}
$ istioctl uninstall -y --purge
$ kubectl delete namespace istio-system
{{< /text >}}
Expand Down
4 changes: 1 addition & 3 deletions content/en/docs/ambient/getting-started/cleanup/snips.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,8 @@ kubectl label namespace default istio.io/dataplane-mode-
kubectl label namespace default istio.io/use-waypoint-
}

snip_remove_waypoint_proxies_and_uninstall_istio_1() {
snip_remove_waypoint_proxies_1() {
istioctl x waypoint delete --all
istioctl uninstall -y --purge
kubectl delete namespace istio-system
}

snip_remove_the_sample_application_1() {
Expand Down
7 changes: 3 additions & 4 deletions content/en/docs/ambient/getting-started/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

# @setup profile=none
# @setup profile=ambient

set -e
set -u
Expand All @@ -27,7 +27,6 @@ source "content/en/docs/ambient/getting-started/enforce-auth-policies/snips.sh"
source "content/en/docs/ambient/getting-started/manage-traffic/snips.sh"
source "content/en/docs/ambient/getting-started/cleanup/snips.sh"

snip_install_ambient
snip_install_k8s_gateway_api

_wait_for_deployment istio-system istiod
Expand Down Expand Up @@ -71,7 +70,7 @@ _verify_lines snip_test_traffic_split "

# @cleanup
snip_remove_the_ambient_and_waypoint_labels_1
snip_remove_waypoint_proxies_and_uninstall_istio_1
snip_remove_waypoint_proxies_1
snip_remove_the_sample_application_1
samples/bookinfo/platform/kube/cleanup.sh
snip_remove_the_kubernetes_gateway_api_crds_1
snip_remove_the_kubernetes_gateway_api_crds_1
7 changes: 2 additions & 5 deletions content/en/docs/ambient/usage/extend-waypoint-wasm/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

# @setup profile=none
# @setup profile=ambient

set -e
set -u
Expand All @@ -31,9 +31,6 @@ source "content/en/docs/ambient/getting-started/cleanup/snips.sh"
# Kubernetes Gateway API CRDs are required by waypoint proxy.
snip_install_k8s_gateway_api

# install istio with ambient profile
snip_install_ambient

_wait_for_deployment istio-system istiod
_wait_for_daemonset istio-system ztunnel
_wait_for_daemonset istio-system istio-cni-node
Expand Down Expand Up @@ -98,7 +95,7 @@ _verify_same snip_verify_the_traffic_targeting_the_service_3 "$snip_verify_the_t
# @cleanup
snip_cleanup_1
snip_remove_the_ambient_and_waypoint_labels_1
snip_remove_waypoint_proxies_and_uninstall_istio_1
snip_remove_waypoint_proxies_1
snip_remove_the_sample_application_1
samples/bookinfo/platform/kube/cleanup.sh
snip_remove_the_kubernetes_gateway_api_crds_1

0 comments on commit 7180428

Please sign in to comment.