-
Notifications
You must be signed in to change notification settings - Fork 305
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
ci: Ensure composefs is enabled on Fedora #3067
Conversation
3be9e2f
to
f331bad
Compare
For some reason we're not picking this up in the Prow build, which breaks things because now rpm-ostree hard requires it. Let's make this a fatal build time error for more clear debugging.
f331bad
to
2f76b03
Compare
OK now we're hitting
hmm...thinking this is some sort of drift w/rpm-ostree. Need to dig in |
Right we need rpm-ostree v2023.8 which enables composefs client side for this, but now I'm confused how it worked before? |
Oh, I wonder if this is the issue @ericcurtin hit, where somehow the default to enable composefs was disabled in the fedora packages after a new release? I forgot the details about that though... |
/retest |
I think #3071 is related |
OK now we're getting:
which is good actually but yes, to make this test really work we currently need to either eat the hit and reprovision to ext4 on boot with ignition or use !coreos. |
Need to change the framework to boot with an Ignition config that switches to ext4 soon.
27cfb87
to
734ea3e
Compare
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.
Looks sane to me.
For some reason we're not picking this up in the Prow build, which breaks things because now rpm-ostree hard requires it.
Let's make this a fatal build time error for more clear debugging.