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

Update googlecloud dependency to v0.43.1 #26071

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

dashpole
Copy link
Contributor

@dashpole dashpole commented Aug 23, 2023

Description:

Update googlecloud dependency from v0.42.0 to v0.43.1. See the full set of changes here: https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/releases/tag/v0.43.0

I refactored the API surface of the googlemanagedprometheus package to minimize future breaking changes in GoogleCloudPlatform/opentelemetry-operations-go#709. This created a Config struct in that package with configuration options for functions in the package. It is backwards-compatible with the configuration currently in the googlemanagedprometheus exporter, but adds a new option add_metric_suffixes.

This also removes the deprecated BuildPromCompliantName now that the googlemanagedprometheus library has migrated to the new function.

Testing:

Expanded config unit test to cover all fields.

Documentation:

Updated the README with new add_metric_suffixes configuration. This configuration matches the configuration used by the prometheus remote write and prometheus exporters.

@dashpole dashpole requested a review from a team August 23, 2023 13:54
@dashpole dashpole force-pushed the update_googlecloud_version branch 2 times, most recently from 697cc35 to c4d780d Compare August 23, 2023 13:59
@dashpole
Copy link
Contributor Author

dashpole commented Aug 23, 2023

Looks like this is blocked on updating to go 1.21. I'll convert to a draft for now.

Edit: reverted back to 1.20 in a patch release. Should be good to go now.

@dashpole dashpole marked this pull request as draft August 23, 2023 14:16
@dashpole dashpole marked this pull request as ready for review August 28, 2023 14:25
@codeboten codeboten merged commit d4f10f8 into open-telemetry:main Aug 29, 2023
94 checks passed
@github-actions github-actions bot added this to the next release milestone Aug 29, 2023
@dashpole dashpole changed the title Update googlecloud dependency to v0.43.0 Update googlecloud dependency to v0.43.1 Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants