Skip to content

microvm: drop the kata-config asset - #1704

Merged
Benjamin Elder (BenTheElder) merged 1 commit into
agent-substrate:mainfrom
hdp617:remove-clh-config
Sep 22, 2026
Merged

Benjamin Elder (BenTheElder) merged 1 commit into
agent-substrate:mainfrom
hdp617:remove-clh-config

Conversation

@hdp617

Copy link
Copy Markdown
Contributor

Today, ateom fetches the kata-config asset (configuration-clh.toml) to read only 3 values default_memory, default_vcpus and kernel_params. The first 2 are redundant because (1) the values never change and (2) ateom has the same defaults. Only kernel_params is relevant for the kata-agent; its value also changed once in the Kata project history.

ateom now owns all three values, which also allows us to tune them specifically for Substrate.

Verified e2e with a GKE cluster.

Fixes #1693

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@hdp617

Copy link
Copy Markdown
Contributor Author

@bowei Bowei Du (bowei) added kind/cleanup Small fixes that are not bugs, for example a typo in a code comment area/microVM labels Sep 17, 2026
@BenTheElder

Copy link
Copy Markdown
Collaborator

This appears to be failing tests ..

@hdp617

Copy link
Copy Markdown
Contributor Author

It's a flake; I don't have permission to rerun.

Comment thread hack/install-microvm-deps.sh Resolved
Today, `ateom` fetches the `kata-config` asset (configuration-clh.toml) to read only 3 values `default_memory`, `default_vcpus` and `kernel_params`. The first 2 are redundant because (1) the values never change and (2) ateom has the same defaults. Only `kernel_params` is relevant for the `kata-agent`; its value also changed once in the Kata project history.

`ateom` now owns all three values, which also allows us to tune them specifically for Substrate.
@hdp617

Copy link
Copy Markdown
Contributor Author

Looks like a flake.

@BenTheElder
Benjamin Elder (BenTheElder) added this pull request to the merge queue Sep 22, 2026
Merged via the queue into agent-substrate:main with commit 8062bba Sep 22, 2026
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/microVM kind/cleanup Small fixes that are not bugs, for example a typo in a code comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drop the kata-config asset from the microvm SandboxConfig

3 participants