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

Automate per-package release for specific components #2875

Merged
merged 25 commits into from
Oct 9, 2024

Conversation

lmolkova
Copy link
Contributor

@lmolkova lmolkova commented Sep 17, 2024

Description

Adds automation allowing to release individual components.
Packages that can be released individually are:

  • excluded from the main release
  • must have individual changelog
  • follow the same release process and versioning rules:
    • prepare minor/patch that creates
      • long-term package-release/{package}/v{version} branch
      • PRs to update changelog and versions in release branch and main
    • release
  • adds changelogs and versions for for components that are currently released manually as individual packages (opentelemetry-resource-detector-azure, opentelemetry-sdk-extension-aws, opentelemetry-propagator-aws-xray)

Related to #2858

Type of change

How Has This Been Tested?

Tested in the fork https://github.com/lmolkova/opentelemetry-python-contrib/actions

  • Preparing + releasing new minor or patch version of an unstable package
  • Preparing + releasing 'common'

Does This PR Require a Core Repo Change?

  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated

@lmolkova lmolkova marked this pull request as ready for review September 19, 2024 16:40
@lmolkova lmolkova requested a review from a team as a code owner September 19, 2024 16:40
CHANGELOG.md Show resolved Hide resolved
RELEASING.md Outdated Show resolved Hide resolved
@lmolkova lmolkova force-pushed the per-package-release-2 branch 2 times, most recently from e549a9e to 93d8df6 Compare September 20, 2024 00:36
RELEASING.md Outdated Show resolved Hide resolved
Copy link
Contributor

@lzchen lzchen left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for working on this! Just a few minor comments. Feel free to reach out to one of the maintainers to actually test this out with an actual package once it is merged.

@lmolkova
Copy link
Contributor Author

lmolkova commented Oct 1, 2024

@open-telemetry/opentelemetry-python-contrib-approvers could I get some eyes on it please? Thank you!

Copy link
Member

@aabmass aabmass left a comment

Choose a reason for hiding this comment

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

Thanks for working on it @lmolkova

It seems reasonable to me and I think @lzchen has the most context and has already approved. I'm OK to go ahead with this.

@lzchen lzchen merged commit 6a54106 into open-telemetry:main Oct 9, 2024
528 checks passed
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.

7 participants