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

[chore] update CHANGELOG in preperation for v0.110 release #5413

Merged
merged 7 commits into from
Sep 26, 2024

Conversation

hughesjj
Copy link
Contributor

No description provided.

@hughesjj hughesjj marked this pull request as ready for review September 26, 2024 04:03
@hughesjj hughesjj requested review from a team as code owners September 26, 2024 04:03
Copy link
Contributor

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

We need to include changelog notes for any upstream change that is relevant and impactful to our end-users as well.

CHANGELOG.md Outdated Show resolved Hide resolved
@hughesjj
Copy link
Contributor Author

We need to include changelog notes for any upstream change that is relevant and impactful to our end-users as well.

Gotcha, is that just "copy everything relevant to components we use as-is" or should I workshop the inclusion/exclusion with Product?

@crobert-1
Copy link
Contributor

Gotcha, is that just "copy everything relevant to components we use as-is" or should I workshop the inclusion/exclusion with Product?

I'd say start with copy, and then as part of the review we can remove anything we may not be interested in, or reword as necessary 👍

Copy link
Contributor

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

Looking good! Some whitespace nits, and comments:

  1. Any component needs to be formatted with the formatting, I see a couple missing.
  2. We can remove any changelog entries for components not in our distro.

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
hughesjj and others added 5 commits September 26, 2024 16:30
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Paulo Janotti <[email protected]>
Co-authored-by: Paulo Janotti <[email protected]>
@hughesjj hughesjj merged commit ccf047e into main Sep 26, 2024
2 checks passed
@hughesjj hughesjj deleted the changelog_v0.110 branch September 26, 2024 23:56
@github-actions github-actions bot locked and limited conversation to collaborators Sep 26, 2024

- (Core) `service`: Ensure process telemetry is registered when internal telemetry is configured with readers instead of an address. ([#11093](https://github.com/open-telemetry/opentelemetry-collector/pull/11093))
- (Contrib) `splunkenterprise`: Fix a flaky search related to iops metrics. ([#35081](https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/35081))
- (Contrib) `azuremonitorexporter`: fix issue for property endpoint is ignored when using instrumentation_key ([#33971](https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33971))
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we have this exporter. Please double-check that we include items for components that we provide with our distro.

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

Successfully merging this pull request may close these issues.

4 participants