Skip to content
This repository was archived by the owner on Jun 7, 2026. It is now read-only.

Bump default memory to 1G and storage timeouts to 15m#47

Merged
tomusdrw merged 1 commit into
mainfrom
td-perf-storage-timeout-mem
May 8, 2026
Merged

Bump default memory to 1G and storage timeouts to 15m#47
tomusdrw merged 1 commit into
mainfrom
td-perf-storage-timeout-mem

Conversation

@tomusdrw

@tomusdrw tomusdrw commented May 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Raise docker_memory default in reusable-picofuzz.yml from 512m to 1024m. The 15 performance workflows that don't override (boka, graymatter, jamixir, jampy, jampy-recompiler, jamzig, jamzilla, jamzilla-int, new-jamneration, pyjamaz, tessera, tsjam, turbojam, typeberry, vinwolf) get bumped to 1G; the 8 that already set ≥2048m keep their larger values.
  • Extend timeout-minutes (job hard cap) and TIMEOUT_MINUTES (env var read by getTimeoutMs) to 15 for the storage and storage_light suites in both the minifuzz and picofuzz jobs. Other suites keep inputs.timeout_minutes (default 10).

Test plan

  • Trigger one performance workflow that does not override docker_memory (e.g. typeberry-performance) via workflow_dispatch and confirm TARGET_MEMORY=1024m in the docker run logs.
  • Confirm storage / storage_light matrix legs report the 15m timeout in the GHA UI, while fallback / safrole / forks / no_forks legs still show 10m.
  • Confirm a workflow that explicitly sets a higher memory (e.g. javajam-performance: 8192m) is unchanged.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Optimized test execution with dynamic timeout configuration for resource-intensive test suites.
    • Increased Docker memory allocation to 1024m for improved test performance.

Raise reusable-picofuzz docker_memory default from 512m to 1024m, and
extend the timeout to 15 minutes for storage and storage_light suites
in both minifuzz and picofuzz jobs (other suites stay at 10m default).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 8, 2026

Copy link
Copy Markdown

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e7710b3b-7f4a-4b09-9995-31172426af20

📥 Commits

Reviewing files that changed from the base of the PR and between 438b76d and 590b46c.

📒 Files selected for processing (1)
  • .github/workflows/reusable-picofuzz.yml

📝 Walkthrough

Walkthrough

This PR updates the reusable picofuzz CI workflow with two changes: Docker memory default is increased to 1024m, and both minifuzz and picofuzz jobs now apply conditional 15-minute timeouts specifically for storage test suites while other suites retain the input timeout value. The timeout condition is also propagated to job step environment variables.

Changes

Picofuzz Workflow Configuration

Layer / File(s) Summary
Workflow Inputs
.github/workflows/reusable-picofuzz.yml
inputs.docker_memory.default increased from 512m to 1024m.
Job Timeout Configuration
.github/workflows/reusable-picofuzz.yml
minifuzz and picofuzz job timeout-minutes set to 15 when test matrix is storage or storage_light, else inputs.timeout_minutes.
Container Environment Variables
.github/workflows/reusable-picofuzz.yml
TIMEOUT_MINUTES environment variable in both job steps updated with matching conditional logic for storage vs. other test suites.

Possibly related PRs

Poem

🐰 A rabbit hops through CI flows,
With timeouts tuned for storage rows,
Memory doubled, logic clean,
The fastest fuzzing ever seen! ✨

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the two main changes: bumping default Docker memory from 512m to 1024m (1G) and extending storage test timeouts to 15 minutes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch td-perf-storage-timeout-mem

Comment @coderabbitai help to get the list of available commands and usage tips.

@tomusdrw tomusdrw merged commit 8575482 into main May 8, 2026
13 checks passed
@tomusdrw tomusdrw deleted the td-perf-storage-timeout-mem branch May 8, 2026 20:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant