Enable the ambient/getting-started istio.io tests

[ericvn]

Is this the right place to submit this?

• This is not a security vulnerability or a crashing bug
• This is not a question about how to use Istio

Bug Description

Tests get (ex: https://gcsweb.istio.io/gcs/istio-prow/pr-logs/pull/istio_istio.io/14775/doc.test.profile-none_istio.io/1773380790281834496/artifacts/tests-setup-profile-none-f8b3f6/TestDocs/ops/ambient/getting-started/test.sh/test.sh/_test_context):

VERIFY FAILED snip_layer_7_authorization_policy_4 (timeout after 120s):
received:
"upstream connect error or disconnect/reset before headers. reset reason: connection termination"
expected:
"RBAC: access denied"

Version

main istio branch

Additional Information

No response

[linsun]

One thing I noted is this cmd output needs to be updated:

kubectl logs ds/ztunnel -n istio-system  | grep inpod_enabled

as of latest master, this returns number of pods vs true|false.

Found 3 pods, using pod/ztunnel-lg4ft

[linsun]

Hi @ilrudie I tried to enable test in istio/istio.io#14922 as it seem to work pretty well when i tested manually.

[ilrudie]

Sounds good. I will take a look at your pr shortly!

[ericvn]

This test has not been enabled, so re-opening.

[linsun]

Sorry I misunderstood the issue. I had thought it is the tests button in the doc, but it is not. @ilrudie give this back to you :)

[ericvn]

Just enabling the tests to see if they maybe work in istio/istio.io#14941

[ericvn]

There are test failures noted in that PR. One may be a needed test framework update, but also it seems that some renumbering was done in the snippets in another PR which need to be pulled into the tests. Since the testing was disabled, the renumbering is uncovered until now.

[ilrudie]

Thanks Eric. I will take a look shortly

[ericvn]

@ilrudie You can start with my PR. It should have a good start for the test.sh, but might need some renumbering/changes in the gtw_test.sh. If my verify fix PR isn't merged, just grab the one line change in verify.sh to make headway.

[ilrudie]

@ericvn, ty.

[ilrudie]

Closed by istio/istio.io#14953