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

Add ALTS support to interop stress client #7723

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sagarsudo
Copy link

@sagarsudo sagarsudo commented Oct 10, 2024

Tested using:

./client --server_addresses="google-c2p:///directpath-grpclb-with-pick-first-test-jz.googleapis.com" --test_cases="empty_unary:100" --use_alts=true --test_duration_secs=10
Total calls made: 54346384

@sagarsudo sagarsudo marked this pull request as draft October 10, 2024 00:58
@sagarsudo sagarsudo marked this pull request as ready for review October 10, 2024 01:08
@purnesh42H purnesh42H added this to the 1.69 Release milestone Oct 16, 2024
@sagarsudo sagarsudo force-pushed the patch-1 branch 2 times, most recently from 8fd2b22 to 828981c Compare October 16, 2024 19:29
@eshitachandwani
Copy link
Member

Hi @sagarsudo, it seems you may want to include this flag in your application for testing purposes. Could you please clarify the reason for requesting this change in the gRPC-Go OSS repository?

@sagarsudo
Copy link
Author

Just noticed a discrepancy in between interop/client and interop/stress/client, where the former has ALTS support but the latter does not.

@sagarsudo
Copy link
Author

It would be good to have ALTS support in the stress client as it would make it easier to stress test against GCP services.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants