Skip to content

Add deprecation warning for Google PubSub receivers build in encoder #39371

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

Closed
alexvanboxel opened this issue Apr 13, 2025 · 2 comments · Fixed by #39842
Closed

Add deprecation warning for Google PubSub receivers build in encoder #39371

alexvanboxel opened this issue Apr 13, 2025 · 2 comments · Fixed by #39842

Comments

@alexvanboxel
Copy link
Contributor

Component(s)

receiver/googlecloudpubsub

Describe the issue you're reporting

The Google Cloud PubSub receiver can handle several encodings, but now the receiver can handle external encoders, and we have alternatives for the build in once.

Start deprecation phase 1, and warn the user at startup when they have configured the encoding. Both Google Cloud LogEntry (encoder: #37531) and text encoding have an external encoder alternative and will get the warning log line at start up.

The binary OTLP encoding will still be the default and build in.

@alexvanboxel alexvanboxel added the needs triage New item requiring triage label Apr 13, 2025
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Apr 14, 2025
@crobert-1
Copy link
Member

Makes sense to me and was filed by a code owner, removing needs triage

@atoulme atoulme closed this as completed in 74c8b54 May 5, 2025
vincentfree pushed a commit to ing-bank/opentelemetry-collector-contrib that referenced this issue May 6, 2025
…pen-telemetry#39842)

#### Description
Add deprecation warning for the build-in encoders. The build-in encoders
`cloud_logging` and `raw_text` both have encoding extension alternatives
and will be removed in version v0.132.0 of the collector

#### Link to tracking issue
Fixes open-telemetry#39371

#### Testing
Started the collector with the encoders selected and see the warning

#### Documentation
Added deprecation to the README
vincentfree pushed a commit to ing-bank/opentelemetry-collector-contrib that referenced this issue May 20, 2025
…pen-telemetry#39842)

#### Description
Add deprecation warning for the build-in encoders. The build-in encoders
`cloud_logging` and `raw_text` both have encoding extension alternatives
and will be removed in version v0.132.0 of the collector

#### Link to tracking issue
Fixes open-telemetry#39371

#### Testing
Started the collector with the encoders selected and see the warning

#### Documentation
Added deprecation to the README
dragonlord93 pushed a commit to dragonlord93/opentelemetry-collector-contrib that referenced this issue May 23, 2025
…pen-telemetry#39842)

#### Description
Add deprecation warning for the build-in encoders. The build-in encoders
`cloud_logging` and `raw_text` both have encoding extension alternatives
and will be removed in version v0.132.0 of the collector

#### Link to tracking issue
Fixes open-telemetry#39371

#### Testing
Started the collector with the encoders selected and see the warning

#### Documentation
Added deprecation to the README
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants