Skip to content

interop-test: Opt-in GKE tests && Remove duplicated GCE tests (v1.39.x backport)#9202

Merged
ejona86 merged 11 commits intogrpc:v1.39.xfrom
lidizheng:rm-39
May 27, 2022
Merged

interop-test: Opt-in GKE tests && Remove duplicated GCE tests (v1.39.x backport)#9202
ejona86 merged 11 commits intogrpc:v1.39.xfrom
lidizheng:rm-39

Conversation

@lidizheng
Copy link
Copy Markdown
Contributor

@lidizheng lidizheng commented May 24, 2022

@lidizheng
Copy link
Copy Markdown
Contributor Author

lidizheng commented May 25, 2022

Retry test cases in xds-url-map failed because Retry is only supported after v1.39.x: https://github.com/grpc/grpc/blob/master/doc/grpc_xds_features.md

The Retry test cases require >v1.40.x for C++, Python, Java, and >v1.41.x for Go: https://github.com/grpc/grpc/blob/master/tools/run_tests/xds_k8s_test_driver/tests/url_map/retry_test.py#L69

This is expected, because we are triggering grpc/java/master/branch/xds_url_map. Ideally, we want to trigger the test in v1.39.x branch, which looks like grpc/java/v1.39.x/branch/xds_url_map. However, this job doesn't exist and depend on this PR to be merged.

@lidizheng lidizheng marked this pull request as ready for review May 25, 2022 22:32
@lidizheng
Copy link
Copy Markdown
Contributor Author

Test re-triggered:

lidizheng and others added 9 commits May 27, 2022 12:25
The test driver install script is read directly from the core repo master branch and the copy in the Java repo is deleted.
This test suite is only fully enabled for C++ and Python at this
moment. This commit enables the rest of the test suite for grpc-java.

* change_backend_service_test
* failover_test
* remove_neg_test
* round_robin_test
* affinity_test
We are solving the issue of grpc/grpc delaying the release, causing GCE
tests to fail. Updating the master branch prevents similar cases from
happening.

Tested: [prod:grpc/java/master/branch/xds_v3](http://sponge/010f5353-e65e-4b1a-b6d3-f5a84e31546b)
Related PR grpc/grpc#26764
Related CL cl/386366746

This test suite will be run every 6 hours, and takes around 30 minutes. The script is copied from xds-k8s.sh, and removed the generation of server image.
As title. We recently had one flake caused by the Kokoro job timeout.
@ejona86
Copy link
Copy Markdown
Member

ejona86 commented May 27, 2022

d068588 looks busted, as it doesn't resemble the master commit 8ac9a4e at all.

@lidizheng
Copy link
Copy Markdown
Contributor Author

lidizheng commented May 27, 2022

There is one line of code change, so re-triggering the xDS tests:

@ejona86 ejona86 merged commit 9b5960b into grpc:v1.39.x May 27, 2022
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants