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

Prepare release 0.100.0 #2960

Merged
merged 3 commits into from
May 20, 2024

Conversation

VineethReddy02
Copy link
Contributor

@VineethReddy02 VineethReddy02 commented May 15, 2024

resolves #2929

@VineethReddy02 VineethReddy02 requested a review from a team as a code owner May 15, 2024 18:08
@VineethReddy02 VineethReddy02 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 15, 2024
@pavolloffay
Copy link
Member

CHANGELOG.md Outdated

* [OpenTelemetry Collector - v0.100.0](https://github.com/open-telemetry/opentelemetry-collector/releases/tag/v0.100.0)
* [OpenTelemetry Contrib - v0.100.0](https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.100.0)
* [Java auto-instrumentation - v1.32.0](https://github.com/open-telemetry/opentelemetry-java-instrumentation/releases/tag/v1.32.0)
Copy link
Member

Choose a reason for hiding this comment

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

this should be 1.32.1

Copy link
Member

Choose a reason for hiding this comment

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

The version still hasn't been updated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my bad, fixed it.

@@ -44,9 +44,9 @@ The operator should be released within a week after the [OpenTelemetry collector

| Version | Release manager |
|----------|-----------------|
| v0.100.0 | @TylerHelmuth |
Copy link
Contributor

Choose a reason for hiding this comment

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

We originally removed you from this rotation in #2957 @VineethReddy02. If you'd like to stay in it, you can simply add yourself to the end of the queue. And if not, you shouldn't deprive @TylerHelmuth of the pleasure of releasing the next version of the operator. :)

Copy link
Member

Choose a reason for hiding this comment

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

So close lol

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To be honest, I didn't see the release.md rotation as the first thing, Jacob DM'ed me in slack mentioning my release rotation for 0.100.0. I followed all the steps and made changes just before creating the PR I noticed Tyler as the release captain. Only then did I realize it had been updated. However, since I had already made the changes, I went ahead and created the release PR.

I certainly didn't intend to step on @TylerHelmuth's toes. As a result, I added Tyler to the rotation cycle and didn't include my name as it had already been removed.

Consider this as my final pleasure of releasing the operator. :)

@pavolloffay
Copy link
Member

I would like to block this until #2878 is merged

@pavolloffay
Copy link
Member

Changelog for this can be added as well #2964 (review)

@VineethReddy02
Copy link
Contributor Author

Okay, I see we are waiting for #2878 (I will update chlog once its merged)

@pavolloffay pavolloffay merged commit 2ae190e into open-telemetry:main May 20, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release the operator v0.100.0
5 participants