Skip to content

Reduce xDS GCE interop framework to only run ping_pong#27355

Merged
lidizheng merged 2 commits intogrpc:masterfrom
lidizheng:reduce-gce
Sep 16, 2021
Merged

Reduce xDS GCE interop framework to only run ping_pong#27355
lidizheng merged 2 commits intogrpc:masterfrom
lidizheng:reduce-gce

Conversation

@lidizheng
Copy link
Copy Markdown
Contributor

@lidizheng lidizheng commented Sep 15, 2021

As title.

All the tests have been migrated to the GKE framework (except api_listener series, which will be soon). This PR proposes to only keep a minimum test case in GCE framework (ensure the common workflow in GCE v2/v3 is okay). This PR only affects production tests.

Test runs:

@lidizheng lidizheng added area/test release notes: no Indicates if PR should not be in release notes labels Sep 15, 2021
@lidizheng lidizheng marked this pull request as ready for review September 15, 2021 17:12
@YifeiZhuang
Copy link
Copy Markdown
Member

YifeiZhuang commented Sep 15, 2021

prerequisite migrating api listener tests #26993

@lidizheng
Copy link
Copy Markdown
Contributor Author

CC @srini100 to decide how we coordinate #26993, changes in staging, and this PR.

@srini100
Copy link
Copy Markdown
Contributor

Since PHP and Ruby are not yet ported to GKE, remove those from the list here.
Regarding the API listener and metadata filter tests, I think it is fine to ignore those until they are ported to GKE. Meanwhile, Go and Java have API listener and metadata filter tests running in GCE.

@srini100
Copy link
Copy Markdown
Contributor

Does this affect only master branch for now?

@lidizheng
Copy link
Copy Markdown
Contributor Author

Yes, this PR affects master only. I plan to backport it to 1.41 and 1.40.

@lidizheng
Copy link
Copy Markdown
Contributor Author

Test failures are irrelevant to the Kokoro cfg changes.

@lidizheng lidizheng merged commit 299be43 into grpc:master Sep 16, 2021
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Sep 16, 2021
lidizheng added a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
* Reduce xDS GCE interop framework to only run ping_pong

* Revert changes in PHP and Ruby
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 6, 2023
sergiitk added a commit that referenced this pull request Jan 6, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
sergiitk added a commit to sergiitk/grpc that referenced this pull request Jan 9, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
ejona86 pushed a commit that referenced this pull request Jan 10, 2023
sergiitk added a commit that referenced this pull request Jan 10, 2023
sergiitk added a commit that referenced this pull request Jan 10, 2023
wanlin31 pushed a commit that referenced this pull request May 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test imported Specifies if the PR has been imported to the internal repository release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants