-
Notifications
You must be signed in to change notification settings - Fork 0
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
OSModifier: Extend EMU API to update verity and root device #33
base: main
Are you sure you want to change the base?
Conversation
// Must be 'root' for the rootfs (/) filesystem. | ||
Name string `yaml:"name"` | ||
// The ID of the 'Partition' to use as the data partition. | ||
DataDeviceId string `yaml:"dataDeviceId"` |
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.
Rename to DataDevice
and HashDevice
.
(The Id
suffix refers to the Id
property on Partition
type.)
if err != nil { | ||
return err | ||
} | ||
err = imagecustomizerlib.AddKernelCommandLine(osConfig.KernelCommandLine.ExtraCommandLine, dummyChroot) |
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.
You could call bootCustomizer.AddKernelCommandLine
here.
…nto user/elaine/emu-grub
Might be worth checking whether the differences between MIC and Trident's verity handling are because Trident doesn't need all the same functionality, or if they're partly things that Trident just hasn't implemented yet. For instance, I know that /usr verity is something that we've talked about wanting for Trident, though that would be instead of root verity so there'd still only be one verity device. |
This is a pr that moved from https://github.com/microsoft/azurelinux/pull/10584/files from old repo with comments addressed.
Changes included:
Checklist