-
Notifications
You must be signed in to change notification settings - Fork 270
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
feat(xo-server): add MBR to cloud-init drive #6889
Conversation
931413a
to
6a093e8
Compare
dumped the buffer + mbr in a file ,
|
7b51312
to
3fc717c
Compare
3fc717c
to
ad4d011
Compare
ad4d011
to
deb387e
Compare
We are waiting for some user testing before merging this PR. @fbeauchamp Just let me know when it's good for you 🙂 |
I did a test here, it doesn't break our Debian Cloud init template, and I can see the partition layout, and mount it manually. So LGTM. No idea for the Windows cloudinit thing. |
deb387e
to
6ecabad
Compare
@olivierlambert I think we should wait for Windows testing (at least a few days), if it does not work this PR is useless. |
Shall we merge this now? |
still no news from our tester , I think we should merge it |
Description
terraform need the cloud init trive to be bootable under windows. Added a MBR
disk size is limited to 32MB, is it enough ?
Checklist
Fixes #007
,See xoa-support#42
,See https://...
)Introduced by
CHANGELOG.unreleased.md