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

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.

3 participants