Skip to content

Update OTLP Trace Example to use Auth Extension#391

Merged
psx95 merged 3 commits intoGoogleCloudPlatform:mainfrom
psx95:update-otlp-samples
Jan 31, 2025
Merged

Update OTLP Trace Example to use Auth Extension#391
psx95 merged 3 commits intoGoogleCloudPlatform:mainfrom
psx95:update-otlp-samples

Conversation

@psx95
Copy link
Copy Markdown
Contributor

@psx95 psx95 commented Jan 31, 2025

Updates the OTLP Trace example to use the GCP Auth Extension to export traces to GCP.

@psx95 psx95 requested a review from a team as a code owner January 31, 2025 22:00
@psx95 psx95 requested a review from aabmass January 31, 2025 22:00
Comment on lines +30 to +31
// the shaded variant name is empty
implementation("${libraries.opentelemetry_gcp_auth_extension}:")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🙈

Not sure if it's only .kts, but is there a way to pass named arguments so you could make the empty string more explicit?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah for some reason,

implementation group: 'io.opentelemetry.contrib', name: 'opentelemetry-gcp-auth-extension', version: '1.43.0-alpha', classifier: ''

is not working.

I made a change that makes the intentional use of empty classifier more explicit. This is temporary and will be fixed in the next release of the auth extension.

@psx95 psx95 merged commit aa3cdbd into GoogleCloudPlatform:main Jan 31, 2025
@psx95 psx95 deleted the update-otlp-samples branch January 31, 2025 23:45
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.

3 participants