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 updating OpenTelemetry Collector dependencies doc #2049

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

wildum
Copy link
Contributor

@wildum wildum commented Nov 6, 2024

No description provided.

@wildum wildum marked this pull request as ready for review November 6, 2024 15:00
@wildum wildum requested a review from a team as a code owner November 6, 2024 15:00
docs/developer/updating-otel/README.md Outdated Show resolved Hide resolved
docs/developer/updating-otel/README.md Outdated Show resolved Hide resolved
docs/developer/updating-otel/README.md Outdated Show resolved Hide resolved
docs/developer/updating-otel/README.md Outdated Show resolved Hide resolved
@dehaansa
Copy link
Contributor

dehaansa commented Nov 7, 2024

Should we define a specific window of time that we're going to allow for community maintainers before we can opt to disable/remove the component from that release if it's too much effort?

@wildum
Copy link
Contributor Author

wildum commented Nov 8, 2024

Should we define a specific window of time that we're going to allow for community maintainers before we can opt to disable/remove the component from that release if it's too much effort?

disabeling/removing the component is really a last resort option that I hope we don't ever have to take. If community maintainers are not available to solve some breaking changes, we should be able to quickly solve them (doing the minimum to get it to build and run should be easy in most cases, in the end these are just wrappers around the otel components). I'd prefer to not specify a time window for now and do case-by-case


You can find the community contributor GitHub handles at the top of the community component files (the components that have "Community" set to true).
Once the PR is created, you can ping them in the PR and message them on Slack. Each contributor should create a PR in your branch with the updated component.
Notify the contributors a few days before the release so they have enough time to do the update.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of the time when you read these docs you will be doing the release, so I'm not sure if it's the right place for a reminder like this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the "updating-otel" doc. The update should be done prior to the release and should be started at least one week before cutting the RC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants