-
Notifications
You must be signed in to change notification settings - Fork 24
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/e2e-update: Bump to f34 grub #203
Conversation
I admittedly don't understand much about the CI setup here, but you did previously NAK this exact patch at #184 (comment) so I have the feeling this isn't really the path you wanted to follow. |
Hah. Oops. Thanks for pointing that out. Clearly the bespoke CI setup here has escaped my ability to understand it. |
Closes: coreos#193 When we switched to a split buildroot/cosa containers, this broke. We can just delete this line because the code-under-test is in `overrides/rootfs`, so doing `rm overrides/rpm/*` won't remove it.
Hum, interesting.
This looks like we're just dead at the UEFI console, which implies we may have broken things, or FCOS broke somehow here. |
OK this is easily replicable locally, which is cool. However, I still have no idea what's going wrong. It looks like the underlying error is:
But I can boot the image in bios mode just fine, and I see a filesystem with |
OK so inverting the order of the grub builds made things pass for me. I think What we clearly should be doing instead is synthesizing an RPM with modified binaries, to avoid any dependency on versions or random GRUB builds. |
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, thanks.
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 with @bgilbert suggestion but why does that comment not apply anymore?
So we can debug this.
Need to tweak this to auto-detect gold versus updates, or probably better just synthesize an update.
I was wrong previously; see this bit which does inject the binaries-under-test: Line 48 in d901c5f
We can also cross-verify after this merges with a PR that breaks bootupd. |
This is the F35 equivalent of coreos#203 Closes: coreos#242
Need to tweak this to auto-detect gold versus updates, or
probably better just synthesize an update.