bootc-dev / bootc-dev/bootc

Add tests which cover `--apply`

Open
#1,419 3 comments 0 reactions 1 assignee Claimed by @henrywang View on GitHub
area/ci good first issue
Dominant language
Rust
Stars
2.3k
Forks
230
Avg merge
3d 12h
Merged PRs (30d)
38

Description

We are missing tests here which cover `upgrade --apply` and hence the reboot flow we initiate which would have caught https://github.com/bootc-dev/bootc/issues/1416

To fix this in the tmt tests we probably need to add `tmt-reboot-prepare`; actually I filed https://github.com/teemtee/tmt/issues/3886

---

That said we can and should cover this with a testing flow that runs outside of tmt; actually I'd like to have some of our tests create machines which don't run ssh at all and report their status instead of being puppeted over ssh. Something like an "e2e-upgrade" test that runs a complete scenario instead of being synthetic too, i.e. run a representative workload.

Of course more ideally tmt would support something like that, but it'd be a pretty fundamental change for it AFAIK.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.