Skip to content

benchmarking: put the glutton data dir on the rootfs - #3647

Merged
Max Smythe (maxsmythe) merged 1 commit into
agent-substrate:mainfrom
chw120:glutton-data-dir-rootfs
Nov 5, 2026
Merged

Max Smythe (maxsmythe) merged 1 commit into
agent-substrate:mainfrom
chw120:glutton-data-dir-rootfs

Conversation

@chw120

Copy link
Copy Markdown
Collaborator

The glutton template stored WriteDisk files under /tmp, which is a tmpfs on both sandbox classes (#3533), so they counted as memory and no benchmark wrote to the rootfs upper layer. Use /var/lib/glutton with no volume mounted instead, so agentsession's files land in the rootfs upper layer and in its snapshot.

On gVisor, agentsession now writes to the rootfs for the first time, so its numbers will change. On microVM, its files go back to the host-backed rootfs upper layer, as before #3533.

AI use: AI-assisted, I reviewed and tested it.

  • Tests pass
  • Docs updated

The glutton template stored WriteDisk files under /tmp, which is a tmpfs
on both sandbox classes, so they counted as memory and no benchmark wrote
to the rootfs upper layer. Use /var/lib/glutton with no volume mounted
instead, so agentsession's files land in the rootfs upper layer and in
its snapshot.

On gVisor, agentsession now writes to the rootfs for the first time, so
its numbers will change.

Signed-off-by: Chenyi Wang <[email protected]>

@maxsmythe Max Smythe (maxsmythe) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@maxsmythe

Copy link
Copy Markdown
Collaborator

Merging as benchmark-only PR

@maxsmythe
Max Smythe (maxsmythe) added this pull request to the merge queue Nov 5, 2026
Merged via the queue into agent-substrate:main with commit fc69ac3 Nov 5, 2026
7 checks passed
@chw120
Chenyi Wang (chw120) deleted the glutton-data-dir-rootfs branch November 5, 2026 02:29
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.

2 participants