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 k8s release docs #57

Merged
merged 14 commits into from
Apr 10, 2024
Merged

Prepare k8s release docs #57

merged 14 commits into from
Apr 10, 2024

Conversation

addyess
Copy link
Contributor

@addyess addyess commented Mar 28, 2024

Updates the publish-* charm jobs to be prepared for release-* branches

  • release branches will push to 1.xx/beta on successful merges of PRs

Creates release documentation process.

New workflow adjustments require this PR to merged first

@addyess addyess requested a review from a team as a code owner March 28, 2024 20:27
@addyess addyess force-pushed the KU-536/prepare-operator-releases branch from af92ab9 to 04b2e8f Compare March 28, 2024 20:54
Copy link
Contributor

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

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

Great docs, added some comments.

.github/workflows/publish-charms.yaml Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
### Send announcement

Email announcement to k8s-crew with any relevant information.

Copy link
Contributor

Choose a reason for hiding this comment

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

should we also add a section about how backports are handled in this release process?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool, i'll add another document for this called bug-fix

Copy link
Contributor

@louiseschmidtgen louiseschmidtgen left a comment

Choose a reason for hiding this comment

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

Great work on these k8s release docs!
I added a few notes on punctuation and little fixes.

docs/releases/stable/index.md Outdated Show resolved Hide resolved

Choose the environment based build-on base in `charms/worker/k8s/charmcraft.yaml`

In the following example, building on focal yields packages for py38.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we use python 3.8 instead of py38?

docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
docs/releases/stable/index.md Outdated Show resolved Hide resolved
Copy link
Contributor

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

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

LGTM

@bschimke95 bschimke95 merged commit 03d5875 into main Apr 10, 2024
33 checks passed
@bschimke95 bschimke95 deleted the KU-536/prepare-operator-releases branch April 10, 2024 12:09
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