-
Notifications
You must be signed in to change notification settings - Fork 3
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
Expand release management workflow for dom0-config
to include yum-qa
process
#116
base: main
Are you sure you want to change the base?
Conversation
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.
Flagged some confusing/inaccurate bits. IMO more specific detail (CLI stuff etc) would be required, but that might be best generated during a release run.
|
||
1. Create a release branch in the ``securedrop-workstation`` repository. | ||
2. Push a changelog commit. | ||
3. Push an rc tag in the format ``<major>.<minor>.<patch>~rcN`` on your new commit. |
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.
This is confusing - are we creating the same tag twice as https://github.com/freedomofpress/securedrop-dev-docs/pull/116/files#diff-4734e6704c88c316b866a490f32ca0f4ed2965f9c43c73d2bcc1c6141ec558a3R149 suggests?
signing ceremonies, and stakeholder communications. | ||
|
||
1. Push a release tag on the same commit of the rc tag that was approved during QA. | ||
2. :ref:`Sign the tag with the SecureDrop release key` (or ask another maintainer to do this). |
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.
RPMs need to be signed as well/instead. IIRC this happens automatically for test repos but a signer will be needed for prod.
|
||
Once the package on ``yum-test`` has passed QA and it appears that no additional | ||
release candidates are needed, repeat the process from Step 2, this time opening | ||
the pull request against ``securedrop-yum-qa``. Once the package is uploaded there, |
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.
There is no securedrop-yum-qa, I think. The preflight procedure as I understand it is the same as for the repos:
- create and sign prod RPM
- create a
release
branch on thesecuredrop-yum-prod
repo and add the RPM there - submit a PR for that branch - a webhook will deploy it to yum-qa.securedrop.org
- do preflight checks (as described below)
- when good, merge to
main
, which will deploy it to prod.
@nathandyer Worth reviving? |
Status
Ready for review
Description of Changes
This PR expands the RPM release management section, or at least the portion dedicated to
securedrop-workstation-dom0-config
, to match theapt
counterpart above, and incorporate theyum-test
andyum-qa
repos.Testing
Checklist (Optional)
make docs-lint
) passed locallymake docs-linkcheck
) passedmake docs
) docs at http://localhost:8000