istio / istio.io

Source for the istio.io site
https://istio.io/
Apache License 2.0
762 stars 1.53k forks source link

istioctl x wait removed from 1.23; test update needed #15448

Open craigbox opened 2 months ago

craigbox commented 2 months ago

_wait_for_istio() is used throughout the doc tests, but relies on an experimental feature which has been removed in 1.23.

/tests/util/helpers.sh:

# Wait for Istio config to propagate
# usage: _wait_for_istio <kind> <namespace> <name>
_wait_for_istio() {
    local kind="$1"
    local namespace="$2"
    local name="$3"
    local start=$(date +%s)
    if ! istioctl experimental wait --for=distribution --timeout=10s "$kind" "$name.$namespace"; then
        echo "Failed distribution of $kind $name in namespace $namespace"
        istioctl ps
        echo "TEST: wait for failed, but continuing."
    fi
    echo "Duration: $(($(date +%s)-start)) seconds"
}
craigbox commented 2 months ago

We can also update all the doc_test.go files to remove

values:
  pilot:
    env:
      PILOT_ENABLE_CONFIG_DISTRIBUTION_TRACKING: true

and remove https://github.com/istio/istio.io/blob/master/scripts/snip.py#L107 too.