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 to v0.59.0 #188

Merged
merged 2 commits into from
Aug 31, 2022
Merged

Conversation

Aneurysm9
Copy link
Member

Signed-off-by: Anthony J Mirabella [email protected]

Signed-off-by: Anthony J Mirabella <[email protected]>
@Aneurysm9 Aneurysm9 requested review from a team and dmitryax August 31, 2022 20:18
@jpkrohling jpkrohling requested review from jpkrohling and removed request for dmitryax August 31, 2022 20:41
@jpkrohling jpkrohling self-assigned this Aug 31, 2022
distributions/otelcol/manifest.yaml Show resolved Hide resolved
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/receiver/wavefrontreceiver v0.59.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/receiver/windowsperfcountersreceiver v0.59.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/receiver/zipkinreceiver v0.59.0
- gomod: github.com/open-telemetry/opentelemetry-collector-contrib/receiver/zookeeperreceiver v0.59.0

# When adding a replace, add a comment before it to document why it's needed and when it can be removed
replaces:
Copy link
Member

Choose a reason for hiding this comment

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

An exercise we run during the release is whether we still need the replaces statements here. Do you think we still need the ones we have here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure I can make a call on either of these. The gnostic issue is still open and people are still reporting issues, so probably safest to leave it. The go-connections link I don't have enough context to follow. @jamesmoessis or @MovieStoreGuy may be able to provide more details.

Since these are both versioned replacements it should be safe to leave them until the creator weighs in on their removal since the won't take effect if a newer version would be selected.

Signed-off-by: Anthony J Mirabella <[email protected]>
@bogdandrutu bogdandrutu merged commit b5da83e into open-telemetry:main Aug 31, 2022
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