-
Notifications
You must be signed in to change notification settings - Fork 406
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
Release Automation Process #9555
Release Automation Process #9555
Conversation
9327042
to
12d895c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MV88 this PR is ready for the final review the only missing part is the one generating the docker image for which we can continue using the jenkins for the moment. That part for the docker image will be included later with a dedicated PR.
We need someone familiar with the release process like you are for reviewing this and finally merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some comments here and there. maybe we can talk a bit @offtherailz ?
there is a wrong link inside release_steps.md which is the mapfish print link check if here
it points to http_proxy (line 18)
Here a flow chart that describes the process: | ||
|
||
```mermaid | ||
flowchart TD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Matteo V. <[email protected]>
@offtherailz please backport to stable branch. Thanks |
the commit 39e9019 looks to be present in 2024.01.xx commits list For some reason |
* Add actions * Updated workflows for deprecations and security * Fixed changelog stuff * renamed release workflow file * Fixed concurrency * Separated release steps * Update for uniformity the scripts. Fixed wrong java modules set * Fixed wrong version system in binary * Fixed version update script * Updated documentation. Last fixes * Removed file committed in error * Improved docs * Improved docs * Improved docs * Minor fixes to doc * Fixed release procedure missing paragraph * Update procedure for linking latest stable doc * Update release_steps.md * Update .github/ISSUE_TEMPLATE/release_steps.md * Apply suggestions from code review Co-authored-by: Matteo V. <[email protected]> --------- Co-authored-by: Matteo V. <[email protected]>
Description
This PR introduces release procedure automations for:
At the moment, we still rely on jenkins for :
While we rely on github actions to build and publish artifacts (If we want to use the artifacts from jenkins, we can still re-upload them)
We can later automate docker stuff and updates to MapStoreExtension repository accordingly to finalize the automation
(the following examples are on my fork, of course the final operations have to be applied to the real repository)
Cut release branch
Lunching the Cut release branch workflow from github
screencast-github.com-2023.10.04-11_41_42.webm
At the end the branch will be created with the version fixed accordingly, and a PR will be sent to the master branch with the versions updates.
screencast-github.com-2023.10.04-12_23_55.webm
Prepare Release
The release process will
This process have to be executed on the release branch (a check protect it to accidentally exceute on master).
Create release
This action creates effectively the relase. This has been separated from prepare-release in order to be able to lunch them separately (e.g. if something went wrong, we can fix and generate only the release).
After execution succeed, the release will be ready. You can run manually in draft with the binaries attached (only docker part missing).
You can in the meanwhile:
Post release
After the release, you can trigger the post release process to restore snapshots and increase package.json versions on the release branch.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
#9001
What is the new behavior?
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information