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

[extension/httpforwarder] Promote httpforwarder as stable #17435

Closed
atoulme opened this issue Jan 7, 2023 · 7 comments
Closed

[extension/httpforwarder] Promote httpforwarder as stable #17435

atoulme opened this issue Jan 7, 2023 · 7 comments
Labels
extension/httpforwarder on hold This is blocked by another PR/issue Stale

Comments

@atoulme
Copy link
Contributor

atoulme commented Jan 7, 2023

Component(s)

extension/httpforwarder

Describe the issue you're reporting

I would like to propose that the extension httpforwarder be considered stable, as it has not changed for some time, is used in production in many environments, and no further development is expected.

@atoulme atoulme added the needs triage New item requiring triage label Jan 7, 2023
@dmitryax
Copy link
Member

dmitryax commented Jan 7, 2023

Can we resolve #11128 first?

@dmitryax
Copy link
Member

dmitryax commented Jan 7, 2023

The reason this extension was added is to proxy Signalfx property updates that should be replaced with something that is OTel-compliant going forward. Not sure if this extension is used for any other use cases. I suggest keeping this issue open to gather more evidence that this extension is being used for something else before declaring it stable.

@atoulme
Copy link
Contributor Author

atoulme commented Jan 11, 2023

On 1/11, we had during the SIG meeting a discussion of this issue.
This ongoing issue needs to be resolved first.
Juraci also noted that if the component depends on a library such as confighttp or configtls which are not marked stable yet, that blocks the component from being declared stable.

@github-actions
Copy link
Contributor

Pinging code owners for extension/httpforwarder: @atoulme. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@mx-psi mx-psi added the on hold This is blocked by another PR/issue label Jan 12, 2023
@atoulme atoulme removed the needs triage New item requiring triage label Feb 8, 2023
@github-actions
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

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

@github-actions
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

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

@github-actions github-actions bot added the Stale label Jun 23, 2023
@atoulme
Copy link
Contributor Author

atoulme commented Jun 27, 2023

Closing as we are not going to work on this right now. We can reopen when it's time to move to stable.

@atoulme atoulme closed this as completed Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension/httpforwarder on hold This is blocked by another PR/issue Stale
Projects
None yet
Development

No branches or pull requests

4 participants