Skip to content

[Backport][v1.39.x] xds-k8s tests: Use test driver from master branch#27692

Merged
srini100 merged 2 commits intogrpc:v1.39.xfrom
temawi:v1.39.x-xds-k8s-install-from-master
Oct 13, 2021
Merged

[Backport][v1.39.x] xds-k8s tests: Use test driver from master branch#27692
srini100 merged 2 commits intogrpc:v1.39.xfrom
temawi:v1.39.x-xds-k8s-install-from-master

Conversation

@temawi
Copy link
Copy Markdown
Contributor

@temawi temawi commented Oct 12, 2021

These were missed when  creating the Python virtual env.
@temawi temawi marked this pull request as draft October 12, 2021 17:35
@temawi temawi force-pushed the v1.39.x-xds-k8s-install-from-master branch from 9546c78 to 596c25b Compare October 12, 2021 18:03
Instead of directly sourcing the test driver provisioning script from the same branch, the script is downloaded (with curl) and sourced from the master branch.

This allows changes made to the test driver to be reflected in all future release branches. A separate PR will backport this change to existing release branches.

All cluster definitions are also moved to the install script, allowing any cluster changes to be done in one place in the master branch.
@temawi temawi marked this pull request as ready for review October 12, 2021 18:08
@sergiitk sergiitk added area/test release notes: no Indicates if PR should not be in release notes labels Oct 12, 2021
@sergiitk sergiitk self-requested a review October 12, 2021 21:44
# Constants
readonly GITHUB_REPOSITORY_NAME="grpc"
# GKE Cluster
readonly GKE_CLUSTER_NAME="interop-test-psm-sec-v2-us-central1-a"
Copy link
Copy Markdown
Member

@sergiitk sergiitk Oct 12, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL This's still using PSM Sec cluster. That's why we need these backports, and activate_gke_cluster feature. Thanks, Terry.
cc @lidizheng

Copy link
Copy Markdown
Contributor

@lidizheng lidizheng Oct 12, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the GKE cluster!

Copy link
Copy Markdown
Member

@sergiitk sergiitk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good to go! 🚀

@sergiitk sergiitk changed the title V1.39.x xds k8s install from master [Backport][v1.39.x] xds-k8s tests: Use test driver from master branch Oct 12, 2021
@srini100 srini100 merged commit 8da5aad into grpc:v1.39.x Oct 13, 2021
@temawi temawi deleted the v1.39.x-xds-k8s-install-from-master branch October 13, 2021 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test 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.

4 participants