Skip to content

feat(ci)!: switch to rpmbuild #2792

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

Merged
merged 8 commits into from
Dec 31, 2024
Merged

feat(ci)!: switch to rpmbuild #2792

merged 8 commits into from
Dec 31, 2024

Conversation

madonuko
Copy link
Member

Rationale

As proven in the experimental Terra EL10 branch (el10), switching to -rrpmbuild has significantly decreased time required to build packages, speeding up CI.

Since frawhide is our other "half-pseudo-experimental" branch, we will start deploying this change to this branch first, then backporting this to older branches.

This supposedly does not affect reproducibility because we have been using our own Terra builders inside our mock configs.
Assuming we set up an environment identical to what the mock configs had specified, this should not cause any breaking changes.

However

Some existing anda options, including extra_repos, are not supported under -rrpmbuild.

@madonuko madonuko marked this pull request as ready for review December 31, 2024 18:57
@madonuko madonuko requested a review from lleyton December 31, 2024 18:57
@madonuko madonuko added the enhancement New feature or request label Dec 31, 2024
@madonuko madonuko enabled auto-merge (squash) December 31, 2024 18:57
@madonuko madonuko merged commit 3566e7e into frawhide Dec 31, 2024
10 checks passed
@madonuko madonuko deleted the mado/rpmbuild branch December 31, 2024 20:23
@raboneko
Copy link
Member

💔 All backports failed

Status Branch Result
f39 Backport failed because of merge conflicts

You might need to backport the following PRs to f39:
- feat(ci): fix bugs with update backporting, add weekly mode, remove f39, add el10 (#2610)
- fix(ci): try $GITHUB_SHA instead (#1624)
f40 An unhandled error occurred. Please see the logs for details
f41 An unhandled error occurred. Please see the logs for details
el10 Backport failed because of merge conflicts

You might need to backport the following PRs to el10:
- fix(ci/{build,json-build}): support the extra label in anda.hcl (#2731)
- feat(ci): fix bugs with update backporting, add weekly mode, remove f39, add el10 (#2610)

Manual backport

To create the backport manually run:

backport --pr 2792

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants