Skip to content

build_library: use smaller rootfs for Akamai - #4270

Merged
tormath1 merged 1 commit into
mainfrom
tormath1/linode
Sep 11, 2026
Merged

build_library: use smaller rootfs for Akamai#4270
tormath1 merged 1 commit into
mainfrom
tormath1/linode

Conversation

@tormath1

@tormath1 tormath1 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

This was working fine until Linode enforced the size limit (i.e 6Gb uncompressed image).

rootfs should expand to the available disk size anyway.


This partially reverts: f0c94a9

$ find alpha-44* -type f -exec du --si {} +;
545M    alpha-4459/flatcar_production_akamai_image.bin.gz
4.8G    alpha-4459/flatcar_production_akamai_image.bin
544M    alpha-4487/flatcar_production_akamai_image.bin.gz
14G     alpha-4487/flatcar_production_akamai_image.bin

This creates a raw disk of 7.88Gib:

$ fdisk -l ./flatcar_production_akamai_image.bin
Disk ./flatcar_production_akamai_image.bin: 7.88 GiB, 8455716864 bytes, 16515072 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: gpt
Disk identifier: 00000000-0000-0000-0000-000000000001

Device                                    Start      End Sectors  Size Type
./flatcar_production_akamai_image.bin1     4096  2101247 2097152    1G EFI System
./flatcar_production_akamai_image.bin2  2101248  2105343    4096    2M BIOS boot
./flatcar_production_akamai_image.bin3  2105344  6299647 4194304    2G unknown
./flatcar_production_akamai_image.bin4  6299648 10493951 4194304    2G unknown
./flatcar_production_akamai_image.bin6 10493952 12591103 2097152    1G Linux filesystem
./flatcar_production_akamai_image.bin7 12591104 12722175  131072   64M unknown
./flatcar_production_akamai_image.bin9 12722176 16375807 3653632  1.7G unknown

It's not ideal, but we can't lower more.

This was working fine until Linode enforced the size limit (i.e 6Gb
uncompressed image).

rootfs should expand to the available disk size anyway.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is minimal, consistent with existing layout fallback behavior, and directly aligns with the stated goal of reducing initial image size.

Pull request overview

Adjusts the Akamai (Linode) VM image build settings to reduce the initial root filesystem size, addressing Linode’s enforced uncompressed image size limit while still relying on runtime expansion.

Changes:

  • Stop selecting the vm disk layout for Akamai images so the default base layout is used (smaller initial ROOT partition).
File summaries
File Description
build_library/vm_image_util.sh Removes the Akamai-specific DISK_LAYOUT=vm override so Akamai images fall back to the smaller default disk layout.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings September 11, 2026 07:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The Akamai image conversion has a critical layout incompatibility that must be resolved before approval.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (1)

build_library/vm_image_util.sh:233

  • The PR description still leaves the changelog requirement unchecked, but this is a user-facing Linode/Akamai image-size bug fix. Please add a concise entry under changelog/bugfixes/ before merging so the fix is included in release notes.
IMG_akamai_DISK_LAYOUT=akamai
  • Files reviewed: 2/2 changed files
  • Comments generated: 2
  • Review effort level: Lite

Comment thread build_library/disk_layout.json Outdated
"9":{
"label":"ROOT",
"fs_label":"ROOT",
"blocks":"131072"
Comment thread build_library/vm_image_util.sh Outdated

## akamai (Linode)
IMG_akamai_DISK_LAYOUT=vm
IMG_akamai_DISK_LAYOUT=akamai

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Add the required changelog entry before merging.

Review details

Suppressed comments (1)

build_library/vm_image_util.sh:233

  • This changes the published Akamai/Linode image layout and fixes a user-facing image-size regression, so it needs a changelog entry. The repository requires bug-fix/change entries under changelog/ for this kind of change (changelog/README.md:1-13), but the PR currently adds none; please add one before merging.
IMG_akamai_OEM_SYSEXT=akamai
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@tormath1
tormath1 marked this pull request as ready for review September 11, 2026 10:29
@tormath1
tormath1 requested a review from a team as a code owner September 11, 2026 10:29
@tormath1
tormath1 merged commit 5667d9e into main Sep 11, 2026
22 of 25 checks passed
@tormath1
tormath1 deleted the tormath1/linode branch September 11, 2026 12:20
@github-project-automation github-project-automation Bot moved this from ✅ Testing / in Review to Implemented in Flatcar tactical, release planning, and roadmap Sep 11, 2026
@tormath1

Copy link
Copy Markdown
Contributor Author

Cherry-picked to:

  • flatcar-4790
  • flatcar-4757
  • flatcar-4593

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants