open-telemetry / opentelemetry-go-contrib

Collection of extensions for OpenTelemetry-Go.
https://opentelemetry.io/
Apache License 2.0
1.15k stars 548 forks source link

Deprecate and remove go.opentelemetry.io/contrib/samplers/aws/xray #5554

Closed MrAlias closed 1 day ago

MrAlias commented 4 months ago

This module has been identified to not have an owner. Based on the project's ownership policy, this module will be deprecated and then removed.

How to keep this module

For this module to continue in this repository, it needs a sponsor.

If you would like to sponsor this module and become an owner, please comment in this issue about your desire. As an owner you will assume the following responsibilities:

You will need to have a good working knowledge of the code this module is instrumenting and, ideally, familiarity with the existing module code.

How this module will be removed

This module is in the process of being deprecated. After that deprecation notice has been published, we will wait 3 months or 2 two releases (whichever is the longer time period). After that time period, this module will be removed from this repository and no more versions of the module will be published.

Resurrection

If a sponsor is found after the module has been deprecated or removed, these operations can be reversed (i.e. coded added back, deprecation notice removed).

MrAlias commented 4 months ago

cc @Aneurysm9

wangzlei commented 4 months ago

@Aneurysm9 @MrAlias I'm willing to be code owner for this component

jj22ee commented 4 months ago

I can be a code owner for this component @Aneurysm9 @MrAlias

MrAlias commented 4 months ago

@jj22ee linking my response from here:

@jj22ee I see you are not already a CNCF project member . This is one of the requirements we have for becoming a Code Owner.

I'm going to hold off on adding you as a Code Owner for now. If you do become a CNCF member in the future and still wish to be a Code Owner of this module, please open an issue to apply then.

MrAlias commented 4 months ago

@Aneurysm9 @MrAlias I'm willing to be code owner for this component

Awesome @wangzlei! It looks like you meet all the requirements to become a Code Owner of this module. I will open a PR to add you once https://github.com/open-telemetry/opentelemetry-go-contrib/pull/5712 merges (to help prevent merge conflicts).