Skip to content
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

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Aug 28, 2020

We regressed on this a few times. Let's add a test for it.

Copy link
Member

@cgwalters cgwalters left a 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
Copy link
Member

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.

jlebon added a commit to jlebon/ostree that referenced this pull request Aug 28, 2020
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.
@jlebon
Copy link
Member Author

jlebon commented Aug 28, 2020

CI here will fail until ostreedev/ostree#2186 reaches FCOS.

jlebon added a commit to jlebon/ostree that referenced this pull request Aug 28, 2020
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.
@cgwalters
Copy link
Member

I rebased 🏄‍♂️ this now that we have #596

@cgwalters
Copy link
Member

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.

@cgwalters
Copy link
Member

Hooray, green here! Let's make sure we don't break this again.

@cgwalters cgwalters merged commit 59fa7f0 into coreos:testing-devel Sep 3, 2020
cgwalters added a commit to cgwalters/ostree that referenced this pull request Sep 3, 2020
c4rt0 pushed a commit to c4rt0/fedora-coreos-config that referenced this pull request Mar 27, 2023
ci: add debug information about cosa/rpm-ostree
@jlebon jlebon deleted the pr/var-mount branch April 23, 2023 23:29
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