Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce xDS GCE interop framework to only run ping_pong #27355

Merged
merged 2 commits into from
Sep 16, 2021

Conversation

lidizheng
Copy link
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
Contributor

YifeiZhuang commented Sep 15, 2021

prerequisite migrating api listener tests #26993

@lidizheng
Copy link
Contributor Author

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

@srini100
Copy link
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
Contributor

Does this affect only master branch for now?

@lidizheng
Copy link
Contributor Author

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

@lidizheng
Copy link
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