Skip to content

fix: cleanup nodecustomdata.yaml which are static paths on VHD#8587

Merged
awesomenix merged 2 commits into
mainfrom
nishp/update/cilium
May 26, 2026
Merged

fix: cleanup nodecustomdata.yaml which are static paths on VHD#8587
awesomenix merged 2 commits into
mainfrom
nishp/update/cilium

Conversation

@awesomenix
Copy link
Copy Markdown
Contributor

Recently we ran into issue where customdata has reached beyond the limit

The length of customdata 88636 for VMSS <vmss name> exceeds maximum length 87380 characters

Cleanup static scripts which are part of nodecustomdata.yaml. This will allow us to rollout scriptless in that region and we wont be using the else case anymore

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces the generated Linux cloud-init CustomData payload size by removing many write_files entries from parts/linux/cloud-init/nodecustomdata.yml, based on the assumption that these artifacts already exist at static paths on the VHD. This is intended to avoid exceeding the VMSS CustomData size limit and to support scriptless CSE rollout.

Changes:

  • Removed a large set of cloud-init write_files entries for scripts/services/config files assumed to be pre-baked on the VHD.
  • Kept only the core provisioning scripts (provision*.sh) and the init custom cloud script in the non-scriptless path.

Comment thread parts/linux/cloud-init/nodecustomdata.yml
Comment thread parts/linux/cloud-init/nodecustomdata.yml
Comment thread parts/linux/cloud-init/nodecustomdata.yml
@awesomenix awesomenix merged commit 95cfdd8 into main May 26, 2026
21 of 39 checks passed
@awesomenix awesomenix deleted the nishp/update/cilium branch May 26, 2026 21:48
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.

4 participants