Skip to content

cloudrun: GKE client to cloudrun server test#154

Merged
eshitachandwani merged 61 commits intogrpc:mainfrom
eshitachandwani:eshita_gke_cloudrun
Apr 22, 2025
Merged

cloudrun: GKE client to cloudrun server test#154
eshitachandwani merged 61 commits intogrpc:mainfrom
eshitachandwani:eshita_gke_cloudrun

Conversation

@eshitachandwani
Copy link
Copy Markdown
Member

@eshitachandwani eshitachandwani commented Feb 26, 2025

Tests the case where the client is in GKE and the server is running in cloudrun. Verifies that 100 RPCs reach the server correctly and return with the correct response.

@eshitachandwani eshitachandwani requested a review from a team as a code owner February 26, 2025 20:11
Comment thread bin/run_test_server_c6n.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/compute.py Outdated
Comment thread framework/infrastructure/gcp/compute.py Outdated
Comment thread framework/infrastructure/traffic_director.py Outdated
Comment thread framework/infrastructure/traffic_director_cloudrun.py Outdated
Comment thread framework/infrastructure/traffic_director_cloudrun.py Outdated
Comment thread framework/infrastructure/traffic_director_cloudrun.py Outdated
Comment thread bin/lib/common.py
Comment thread bin/lib/common.py
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py Outdated
Comment thread framework/xds_k8s_testcase.py Outdated
Comment thread framework/xds_k8s_testcase.py Outdated
Comment thread tests/baseline_test_cloud_run_server.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py
Comment thread framework/test_app/runners/cloud_run/cloud_run_base_runner.py Outdated
@eshitachandwani
Copy link
Copy Markdown
Member Author

Include this in your PR:

@eshitachandwani
Copy link
Copy Markdown
Member Author

eshitachandwani commented Apr 15, 2025

Comment thread framework/infrastructure/gcp/api.py
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/compute.py Outdated
Comment thread framework/infrastructure/gcp/compute.py Outdated
Comment thread framework/infrastructure/gcp/compute.py Outdated
Comment thread framework/test_app/runners/k8s/k8s_xds_client_runner.py Outdated
Comment thread framework/test_cases/cloud_run_testcase.py Outdated
Comment thread framework/test_cases/cloud_run_testcase.py
Comment thread framework/xds_k8s_testcase.py Outdated
Comment thread kubernetes-manifests/client.deployment.yaml Outdated
Comment thread kubernetes-manifests/client.deployment.yaml Outdated
Comment thread framework/infrastructure/mesh_resource_manager/cloud_run_mesh_manager.py Outdated
Comment thread .github/psm-interop.code-workspace Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/test_cases/cloud_run_testcase.py Outdated
Comment thread framework/test_cases/cloud_run_testcase.py Outdated
Comment thread framework/infrastructure/mesh_resource_manager/cloud_run_mesh_manager.py Outdated
Comment thread tests/cloud_run_csm_inbound_test.py Outdated
Comment thread framework/infrastructure/mesh_resource_manager/cloud_run_mesh_manager.py Outdated
@sergiitk sergiitk force-pushed the eshita_gke_cloudrun branch from 08d4831 to 5128b98 Compare April 17, 2025 16:46
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.

backend patching is broken

Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/test_app/runners/k8s/k8s_xds_client_runner.py Outdated
Comment thread framework/test_app/runners/k8s/k8s_xds_client_runner.py Outdated
Comment thread framework/infrastructure/gcp/cloud_run.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py Outdated
Comment thread framework/test_app/runners/cloud_run/cloud_run_xds_server_runner.py Outdated
Comment thread tests/cloud_run_csm_inbound_test.py
Comment thread framework/infrastructure/gcp/compute.py Outdated
@eshitachandwani
Copy link
Copy Markdown
Member Author

eshitachandwani commented Apr 21, 2025

Comment thread framework/test_cases/cloud_run_testcase.py Outdated
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.

You missed a few things, but overall LGTM.

One of the things is looks like these fields are gone from backend_service_add_cloudrun_backends now - I don't think this was intentional

                "balancingMode": balancing_mode,
                "capacityScaler": capacity_scaler,

Also please remember to run the tests one more time before merging!

@eshitachandwani
Copy link
Copy Markdown
Member Author

@eshitachandwani eshitachandwani enabled auto-merge (squash) April 22, 2025 13:13
@eshitachandwani eshitachandwani merged commit d92653b into grpc:main Apr 22, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants