-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests/kola: add test for separate /var mount #586
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
#!/bin/bash | ||
set -xeuo pipefail | ||
|
||
# restrict to qemu for now because the primary disk path is platform-dependent |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not strictly related to this PR but I think we really need to introduce something like /dev/disk/coreos-root
or so.
That test will not make it into the fedora-coreos-config repo until the libostree fix gets percolated down. PR is: coreos/fedora-coreos-config#586 But we want to make sure that the fix does work and that we don't regress on it. So manually fetch it for now.
CI here will fail until ostreedev/ostree#2186 reaches FCOS. |
That test will not make it into the fedora-coreos-config repo until the libostree fix gets percolated down. PR is: coreos/fedora-coreos-config#586 But we want to make sure that the fix does work and that we don't regress on it. So manually fetch it for now.
We regressed on this a few times. Let's add a test for it.
0c5d20b
to
01d5ae2
Compare
I rebased 🏄♂️ this now that we have #596 |
OK hmm that last CI run did successfully get the latest ostree and the new test passed; the upgrade run just flaked downloading an image. |
Hooray, green here! Let's make sure we don't break this again. |
ci: add debug information about cosa/rpm-ostree
We regressed on this a few times. Let's add a test for it.