Skip to content

ci: show failed unit details when boot smoke test is degraded - #184

Merged
cgwalters merged 1 commit into
bootc-dev:mainfrom
jeckersb:degraded-debug
Jul 31, 2026
Merged

ci: show failed unit details when boot smoke test is degraded#184
cgwalters merged 1 commit into
bootc-dev:mainfrom
jeckersb:degraded-debug

Conversation

@jeckersb

Copy link
Copy Markdown
Contributor

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: John Eckersberg jeckersb@redhat.com

@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@jeckersb jeckersb mentioned this pull request May 13, 2026
1 task
@jeckersb

Copy link
Copy Markdown
Contributor Author

Hm well on main this doesn't exercise the degraded path so let me add #172 on top of this

@jeckersb

Copy link
Copy Markdown
Contributor Author

ok that's much more useful

May 13 20:18:20 fedora systemd[1]: Starting bootloader-update.service - Update bootloader on boot...
May 13 20:18:22 fedora bootupctl[812]: error: Finding block device from boot or sysroot: Failed to find block device from /boot or /sysroot
May 13 20:18:22 fedora systemd[1]: bootloader-update.service: Main process exited, code=exited, status=1/FAILURE

@jeckersb

Copy link
Copy Markdown
Contributor Author

Let me remove the cherry-pick of #172 and then I think we can merge this overtop of the failures, then we can fix the actual problem over in #172

@cgwalters
cgwalters enabled auto-merge (rebase) May 14, 2026 11:58
Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: John Eckersberg <jeckersb@redhat.com>
@jeckersb

Copy link
Copy Markdown
Contributor Author

Noticed this was still open while going through and cleaning out old TODOs, rebased it, should still be good to go if everything passes still.

@jeckersb

Copy link
Copy Markdown
Contributor Author

Notable because #194 is still showing degraded today

@cgwalters
cgwalters merged commit f242005 into bootc-dev:main Jul 31, 2026
8 of 12 checks passed
run: |
image="localhost/${{ matrix.name }}:${{ matrix.tag }}"
bcvk ephemeral run-ssh "${image}" -- systemctl is-system-running
bcvk ephemeral run-ssh "${image}" -- bash -c '

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.

I think the best fix though is in bootc-dev/bcvk#244

For now I think we're likely still blocked on coreos/bootupd#1072

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.

2 participants