fix: [release/v0.5] Set warmup salt to true as default

[arekay-nv]

What does this PR do?

Sets warmup salt as true by default. This avoids confusion when warmup is enabled but the intent is not to pre-populate the cache with dataset entries. If that is indeed the intent, then explicitly disabling salt can be used.

Type of change

• Bug fix
• New feature
• Documentation update
• Refactor/cleanup

Related issues

Testing

• Tests added/updated
• All tests pass locally
• Manual testing completed

Checklist

• Code follows project style
• Pre-commit hooks pass
• Documentation updated (if needed)

[github-actions]

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

[gemini-code-assist]

Code Review

This pull request updates the default value of the salt parameter in the warmup configuration from False to True in the schema definition and across the concurrency, offline, and online configuration templates. There are no review comments, so I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[nvzhihanj]

Can you push to main as well?

[nvzhihanj]

@attafosu FYI

[arekay-nv]

Review Council — Multi-AI Code Review

Reviewed by: Codex + Claude | Depth: quick (auto-selected: <200 lines)

Found 1 issue.

[arekay-nv]

Review Council — Multi-AI Code Review

Reviewed by: Codex + Claude | Depth: quick (auto-selected: <200 lines, 1 commit)

Found 1 issue — flagged independently by both reviewers (high confidence).

#	File	Line	Severity	Category	Reviewer(s)	Summary
1	src/inference_endpoint/config/schema.py	513	high	testing	Both	test_defaults asserts cfg.salt is False — will fail after this default change

Details: tests/unit/commands/test_benchmark.py:417 has assert cfg.salt is False which now fails (WarmupConfig().salt returns True). Update to assert cfg.salt is True. Test failure confirmed locally.

Codex note: The --no-warmup-salt CLI negation flag is automatically generated by cyclopts and confirmed present, so users who want the old behavior can opt out.

[arekay-nv]

closing in favor of #365 on main.