Skip to content

ci: Fix kernel downgrade to let scriptlets run with TMPDIR=/var/tmp#2198

Merged
Johan-Liebert1 merged 1 commit into
bootc-dev:mainfrom
cgwalters:undo-hacks
May 14, 2026
Merged

ci: Fix kernel downgrade to let scriptlets run with TMPDIR=/var/tmp#2198
Johan-Liebert1 merged 1 commit into
bootc-dev:mainfrom
cgwalters:undo-hacks

Conversation

@cgwalters

Copy link
Copy Markdown
Collaborator

Undo hacky --noscripts - this was working around an rpm-ostree bug (working on a PR for that).

Assisted-by: OpenCode (claude-sonnet-4-6@default)

Undo hacky --noscripts - this was working around an rpm-ostree
bug (working on a PR for that).

Assisted-by: OpenCode (claude-sonnet-4-6@default)
Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters cgwalters added the ci/merge Run full CI suite (all OSes) — equivalent to merge queue label May 13, 2026
@bootc-bot bootc-bot Bot requested a review from jeckersb May 13, 2026 14:00

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request modifies kernel provisioning and upgrade test logic, specifically updating the kernel downgrade process in 'hack/provision-fetch.sh' and removing manual 'dracut' execution in 'tmt/tests/Dockerfile.upgrade-source'. The review identifies critical issues: the removal of module cleanup in the provision script leads to image bloat, and the aggressive 'rm -rf /boot/*' command risks deleting necessary boot files. Additionally, the removal of the 'dracut' block in the Dockerfile may cause boot failures by failing to place the 'initrd' in the location expected by 'bootc'.

Comment thread hack/provision-fetch.sh
Comment thread tmt/tests/Dockerfile.upgrade-source

@Johan-Liebert1 Johan-Liebert1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reran failed jobs

@Johan-Liebert1 Johan-Liebert1 enabled auto-merge May 14, 2026 06:20
@Johan-Liebert1 Johan-Liebert1 added this pull request to the merge queue May 14, 2026
Merged via the queue into bootc-dev:main with commit 280d7f1 May 14, 2026
88 of 113 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/merge Run full CI suite (all OSes) — equivalent to merge queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants