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

feat(xo-server): add MBR to cloud-init drive #6889

Merged
merged 4 commits into from
Jun 28, 2023
Merged

Conversation

fbeauchamp
Copy link
Collaborator

@fbeauchamp fbeauchamp commented Jun 12, 2023

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

  • Commit
    • Title follows commit conventions
    • Reference the relevant issue (Fixes #007, See xoa-support#42, See https://...)
    • If bug fix, add Introduced by
  • Changelog
    • If visible by XOA users, add changelog entry
    • Update "Packages to release" in CHANGELOG.unreleased.md
  • PR
    • If UI changes, add screenshots
    • If not finished or not tested, open as Draft

@fbeauchamp fbeauchamp marked this pull request as ready for review June 12, 2023 16:53
@fbeauchamp fbeauchamp requested a review from ggunullu June 13, 2023 06:31
@ggunullu ggunullu requested a review from julien-f June 13, 2023 13:53
packages/xo-server/src/fatfs-buffer.mjs Outdated Show resolved Hide resolved
packages/xo-server/src/fatfs-buffer.mjs Outdated Show resolved Hide resolved
@fbeauchamp
Copy link
Collaborator Author

fbeauchamp commented Jun 13, 2023

dumped the buffer + mbr in a file , fdisk -l=>

Disque /tmp/partition : 10 MiB, 10486272 octets, 20481 secteurs
Unités : secteur de 1 × 512 = 512 octets
Taille de secteur (logique / physique) : 512 octets / 512 octets
taille d'E/S (minimale / optimale) : 512 octets / 512 octets
Type d'étiquette de disque : dos
Identifiant de disque : 0x00000000

Périphérique    Amorçage Début   Fin Secteurs Taille Id Type
/tmp/partition1 *            1 20481    20481    10M  e W95 FAT16 (LBA)

@julien-f
Copy link
Member

We are waiting for some user testing before merging this PR.

@fbeauchamp Just let me know when it's good for you 🙂

@olivierlambert
Copy link
Member

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.

@julien-f
Copy link
Member

@olivierlambert I think we should wait for Windows testing (at least a few days), if it does not work this PR is useless.

@julien-f
Copy link
Member

Shall we merge this now?

@fbeauchamp
Copy link
Collaborator Author

Shall we merge this now?

still no news from our tester , I think we should merge it

@julien-f julien-f changed the title feat(cloudinit): add a MBR to the cloudconfig drive feat(xo-server): add MBR to cloud-init drive Jun 28, 2023
@julien-f julien-f merged commit 161c20b into master Jun 28, 2023
1 check passed
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.

None yet

4 participants