Skip to content

fix(ci): add --privileged to mock RPM build containers#9

Merged
maxamillion merged 1 commit into
mainfrom
rpm
Mar 29, 2026
Merged

fix(ci): add --privileged to mock RPM build containers#9
maxamillion merged 1 commit into
mainfrom
rpm

Conversation

@maxamillion

@maxamillion maxamillion commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Mock requires mounting tmpfs inside its chroot, which fails without
    privileged container access
  • Add --privileged to x86_64 container jobs and aarch64 run-on-arch-action
    dockerRunArgs in both rpm-repo.yml and ci.yml

Test plan

  • Verify all 6 RPM build jobs (3 x86_64 + 3 aarch64) pass the mock build
    step
  • Verify RPM Repository workflow completes and publishes to GitHub Pages

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated CI/CD pipeline container configurations to enable privileged execution for build processes.

Mock requires mounting tmpfs inside its chroot, which fails without
privileged container access. Add --privileged to both the x86_64
container jobs and aarch64 run-on-arch-action dockerRunArgs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Mar 29, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 855f0389-90d6-4519-91b5-9a469cce889f

📥 Commits

Reviewing files that changed from the base of the PR and between a802cc6 and 6fc7660.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • .github/workflows/rpm-repo.yml

📝 Walkthrough

Walkthrough

Two GitHub Actions workflow files have been modified to enable privileged container execution for RPM build jobs. Both the standard and aarch64 build variants now run with elevated privileges.

Changes

Cohort / File(s) Summary
CI Workflow
.github/workflows/ci.yml
Added options: --privileged to the rpm-build job container configuration and injected --privileged into the dockerRunArgs for the aarch64 variant.
RPM Repository Workflow
.github/workflows/rpm-repo.yml
Added options: --privileged to the build-rpms job container configuration and injected --privileged into the dockerRunArgs for the aarch64 cross-build step.

Possibly related PRs

Poem

🐰 Elevated privileges granted, the containers now run free,
Building RPMs with power, both x86 and aarch64 with glee,
No more permission denied, the workflows now shine bright,
Privileged mode enabled—let's build through the night!


🎯 1 (Trivial) | ⏱️ ~2 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rpm

Comment @coderabbitai help to get the list of available commands and usage tips.

@maxamillion maxamillion merged commit a79a3d2 into main Mar 29, 2026
5 of 7 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.

1 participant