Skip to content

Conversation

ingwonsong
Copy link
Contributor

Merge back #38918, which was reverted by #39069, with the changes below:

Reduce delay between retries
Increase total timeout

@ingwonsong ingwonsong requested a review from a team May 31, 2022 16:12
@ingwonsong ingwonsong requested a review from a team as a code owner May 31, 2022 16:12
@istio-testing istio-testing added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label May 31, 2022
@ingwonsong ingwonsong added the release-notes-none Indicates a PR that does not require release notes. label May 31, 2022
ingwonsong and others added 3 commits May 31, 2022 18:25
…8918)

* Add the integration test for the wasm plugin.

* fix lint

* fix lint

* Fix the template parameter

* Fix test name and secret name

* Move the tests into the existing wasm folder, and share the suite with the others.

* Fix lint errors

* Use c++ wasm codes

* Add license and copyright

* Add "image-pull-policy" test feature

* Reflect the comments

* Simplify the retry structure
@ingwonsong ingwonsong force-pushed the revert-wasm-integration-test branch from b4997b3 to 80427ec Compare May 31, 2022 18:25
@istio-testing istio-testing merged commit 7cd8db4 into istio:master Jun 1, 2022
@ingwonsong ingwonsong deleted the revert-wasm-integration-test branch June 1, 2022 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes-none Indicates a PR that does not require release notes. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants