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

examples/csm_observability: add a flag to use XdsCredentials #7588

Closed
wants to merge 4 commits into from

Conversation

kannanjgithub
Copy link

To demonstrate using the CSM mesh client to talk to cloud run services in user guide, introduce a command line option to use XdsCredentials.

@kannanjgithub kannanjgithub changed the title GCP CSM Observability Java client use XdsCredentials GCP CSM Observability Go client use XdsCredentials Sep 4, 2024
Copy link

codecov bot commented Sep 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.72%. Comparing base (92111dc) to head (7443f7a).
Report is 16 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7588      +/-   ##
==========================================
- Coverage   81.73%   81.72%   -0.02%     
==========================================
  Files         361      361              
  Lines       27813    27813              
==========================================
- Hits        22734    22731       -3     
- Misses       3869     3871       +2     
- Partials     1210     1211       +1     

see 20 files with indirect coverage changes

@aranjans aranjans added this to the 1.67 Release milestone Sep 10, 2024
@aranjans aranjans added the Area: xDS Includes everything xDS related, including LB policies used with xDS. label Sep 10, 2024
@aranjans
Copy link
Contributor

@kannanjgithub Can you please fix the vet errors?

@dfawley dfawley changed the title GCP CSM Observability Go client use XdsCredentials examples/csm_observability: add a flag to use XdsCredentials Sep 10, 2024
@purnesh42H purnesh42H modified the milestones: 1.67 Release, 1.68 Release Sep 10, 2024
@kannanjgithub
Copy link
Author

I was advised to not use the Observability example for service-mesh userguide for cloudrun. Dropping this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: xDS Includes everything xDS related, including LB policies used with xDS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants