opentelemetry-go-contrib icon indicating copy to clipboard operation
opentelemetry-go-contrib copied to clipboard

Deprecate and remove go.opentelemetry.io/contrib/instrumentation/go.mongodb.org/mongo-driver/mongo/otelmongo

Open MrAlias opened this issue 1 year ago • 10 comments

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 be a member of the OpenTelemetry organization and maintain that membership.
    • If you are not an existing member, we can add you to the community. This is not a disqualification, but something you will need in the process of becoming a sponsor.
  • You will be responsible for keeping up with the instrumented library. Any "upstream" changes that impact this module need to be proactively handle by you.
  • You will be expected to review any Pull Requests or Issues created that relate to this module.
  • You will be responsible for the stability and versioning compliance of the module.

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 avatar May 09 '24 19:05 MrAlias

Duplicates https://github.com/open-telemetry/opentelemetry-go-contrib/issues/4678

There is an informal agreement that the MongoDB (Go driver) team will take over the instrumentation library. Reference: https://jira.mongodb.org/browse/GODRIVER-2938

However, there is no progress so I propose to follow the plan to deprecate it in 4 weeks if no code owner is found.

pellared avatar May 10 '24 09:05 pellared

I have posted about this removal in the mongo community forum. https://www.mongodb.com/community/forums/t/otelmongo-at-risk-of-removal/282989

dmathieu avatar May 31 '24 09:05 dmathieu

I also added a comment in https://jira.mongodb.org/browse/GODRIVER-2938

pellared avatar May 31 '24 09:05 pellared

@dmathieu / @pellared thank you for reminding us that this still needed an owner. We'll circle back with more details, but apologies for dropping the ball here.

alexbevi avatar Jun 03 '24 13:06 alexbevi

Hi all, we at MongoDB want to thank the maintainers of the OpenTelemetry Go autoinstrumentation for their hard work and dedication to the community. Please note that we will be assuming ownership of this module, so it will not be deprecated. A few MongoDB folks will plan to attend the next Go SIG in mid-June to discuss further.

If you are using this module and wish to discuss feature requests or ideas for improvement, please reach out via feedback.mongodb.com. Thanks!

ashni-mehta avatar Jun 03 '24 21:06 ashni-mehta

Hi @MrAlias, @pellared, and @dmathieu. One of the engineers on our Go driver team, @prestonvasquez, is interested in becoming the code owner for otelmongo. We are currently looking through the steps for becoming an OpenTelemetry member. Also, we will be joining you for the Go SIG meeting on Thursday (June 6).

Thank you once again for the work you’ve done in maintaining the module thus far!

jtazin avatar Jun 04 '24 16:06 jtazin

@MrAlias @pellared @dmathieu Thank you for the work you've done maintaining otelmongo ❤️ As @jtazin notes, I am interested in becoming the code owner and will work toward membership over the coming week to ensure the continued development of the library.

prestonvasquez avatar Jun 06 '24 18:06 prestonvasquez

@pellared @dmathieu Thanks for reviewing and merging Preston's PR! We would like to continue contributing to otelmongo and have Preston become the official code owner. Would you be willing to sponsor @prestonvasquez in becoming a member of the OpenTelemetry organization?

jtazin avatar Sep 20 '24 22:09 jtazin

The membership requirements state that new members should have multiple contributions to the project. While Preston's contribution is great, and due to its size, probably counts more than a typo fix, AFAIK, it's the only one. So a membership request seems a bit early.

Also, this is a side note but I'm mentioning it because he's not the one making the request. Membership and code ownership is an individual thing, not a brand/company thing. That means, if Preston becomes a code owner, that will be under his name, not under the Mongo brand. Similarly, if he happens to move to another company, code ownership is not interchangeable with another developer (except by contributing and individually becoming an owner).

dmathieu avatar Sep 23 '24 08:09 dmathieu

@dmathieu thanks for your note. your comment regarding membership/ownership is in line with our understanding.

jtazin avatar Sep 23 '24 20:09 jtazin

@MrAlias @dmathieu @pellared Here is my activity since the last update:

  • https://github.com/open-telemetry/opentelemetry-go-contrib/issues/6171
  • https://github.com/open-telemetry/semantic-conventions/issues/1502#issuecomment-2430788127
  • https://github.com/open-telemetry/opentelemetry-go-contrib/pull/6272
  • https://github.com/open-telemetry/opentelemetry-go-contrib/pull/6172

Could you give us an update on progress toward sponsorship?

CC: @jtazin

prestonvasquez avatar Nov 12 '24 19:11 prestonvasquez

First of all, I really want to thank you for your activity.

As for me, given the requirements and responsibilities from both:

  • https://github.com/open-telemetry/community/blob/main/guides/contributor/membership.md#member
  • https://github.com/open-telemetry/opentelemetry-go-contrib/blob/main/CONTRIBUTING.md#code-owners

I can sponsor you in both becoming a OTel member and otelmongo code owner when you take care of adding OTEL_SEMCONV_STABILITY_OPT_IN to otelmongo in order to support the latest changes in the semantic conventions.

Long story short, solving https://github.com/open-telemetry/opentelemetry-go-contrib/issues/6171 is good enough for me.

pellared avatar Nov 12 '24 19:11 pellared

I'm also happy to sponsor you once the opt-in environment variable is added to support the latest semconv.

dmathieu avatar Nov 13 '24 07:11 dmathieu

@pellared @dmathieu With the closur eof https://github.com/open-telemetry/opentelemetry-go-contrib/issues/6171, how do we move forward with sponsorship for org membership and code ownership of otelmongo?

CC: @jtazin

prestonvasquez avatar Mar 07 '25 16:03 prestonvasquez

@pellared @dmathieu With the closur eof #6171, how do we move forward with sponsorship for org membership and code ownership of otelmongo?

CC: @jtazin

LGMT, I can sponsor you. I am pretty sure, @dmathieu will as well.

pellared avatar Mar 07 '25 18:03 pellared

@MrAlias Can this be closed?

prestonvasquez avatar Apr 22 '25 22:04 prestonvasquez

Closed per https://github.com/open-telemetry/opentelemetry-go-contrib/pull/7242

pellared avatar Apr 23 '25 06:04 pellared