Skip to content

[WIP] feat: add integration testing for images - #617

Closed
ledif wants to merge 15 commits into
ublue-os:mainfrom
ledif:integration-test
Closed

[WIP] feat: add integration testing for images#617
ledif wants to merge 15 commits into
ublue-os:mainfrom
ledif:integration-test

Conversation

@ledif

@ledif ledif commented Jun 19, 2025

Copy link
Copy Markdown
Member

Not functional at the moment. Based on rsturla-homelab/bootc-centos-base#45.

@p5

p5 commented Jun 19, 2025

Copy link
Copy Markdown
Member

Not functional at the moment

Happy to help out if wanted! :D

@p5 p5 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ignored the obvious bits you've not yet got round to (like calling Python rather than Go in the workflow), but LGTM!

Comment thread .github/workflows/integration-test.yml Outdated
Comment thread .github/workflows/integration-test.yml Outdated
Comment on lines +86 to +94
- name: Stop QEMU
env:
QEMU_PID: ${{ steps.run-qemu.outputs.QEMU_PID }}
run: |
echo "Stopping QEMU with PID $QEMU_PID"
kill $QEMU_PID
wait $QEMU_PID || true
sleep 5
echo "QEMU stopped"

@p5 p5 Jun 19, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This probably isn't necessary since the runners will be terminated by GitHub at the end of the run, but also doesn't harm anything.

ledif and others added 2 commits June 19, 2025 11:56
Co-authored-by: Robert Sturla <robertsturla@outlook.com>
@inffy

inffy commented Jun 19, 2025

Copy link
Copy Markdown
Member

Ooooo lets gooooo

@ledif

ledif commented Aug 2, 2025

Copy link
Copy Markdown
Member Author

We might want to use https://github.com/secureblue/bootc-integration-test-action instead.

@renner0e

renner0e commented Oct 5, 2025

Copy link
Copy Markdown
Member

openqa has a container btw

https://github.com/os-autoinst/openQA/blob/master/docs/Installing.asciidoc#container-based-setup

@ledif

ledif commented Feb 1, 2026

Copy link
Copy Markdown
Member Author

Closing as stale.

@ledif ledif closed this Feb 1, 2026
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.

4 participants