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
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions docs/developer/updating-otel/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,18 @@ Unfortunately, updating Otel dependencies is not straightforward:
You can refer to [PR grafana/agent#5290](https://github.com/grafana/agent/pull/5290)
for an example on how to update Alloy.

### Notify community component contributors

You can find their GitHub handles at the top of the community component files (the components that have "Community" set to true).
wildum marked this conversation as resolved.
Show resolved Hide resolved
Once the PR is created, you can ping them on it and message them on Slack. Each contributor should create a PR to your branch with the updated component.
wildum marked this conversation as resolved.
Show resolved Hide resolved
They should be notified a few days before the release so that they have enough time to do the update.
wildum marked this conversation as resolved.
Show resolved Hide resolved
It should not be a blocker for the release. If they can't do the update, you can fix the code to resolve the breaking changes or disable the component if that's too complicated.
wildum marked this conversation as resolved.
Show resolved Hide resolved

### Check for metric updates

Some Otel metrics are used in the k8s monitoring helm chart and in integrations.
Make sure to update this [list](https://github.com/grafana/k8s-monitoring-helm/blob/main/charts/k8s-monitoring-v1/default_allow_lists/alloy_integration.yaml) if any of these metrics has been removed or renamed.

## Testing

### Testing a tracing pipeline locally
Expand Down
Loading