Skip to content

Add apply --bash script rendering#176

Draft
ishandhanani wants to merge 2 commits into
mainfrom
idhanani/srtctl-apply-bash
Draft

Add apply --bash script rendering#176
ishandhanani wants to merge 2 commits into
mainfrom
idhanani/srtctl-apply-bash

Conversation

@ishandhanani
Copy link
Copy Markdown
Collaborator

@ishandhanani ishandhanani commented May 27, 2026

Summary

  • Add srtctl apply -f <config> --bash to render a standalone lifecycle sbatch/bash script without submitting.
  • Add a small tested srtctl.render.lifecycle layer plus embedded bash runtime functions for cleanup traps, optional tachometer telemetry, HTTP/worker readiness, and benchmark command execution.
  • Split standalone scripts into a generated manual server config plus the original benchmark config: the script starts do_sweep in manual mode, waits for expected workers, then runs srtctl.cli.run_benchmark against the original config.
  • Document the command and telemetry controls.

Validation

  • bash -n on an emitted srtctl apply --bash script
  • uv run pytest tests/test_lifecycle_render.py tests/test_submit_cli.py tests/test_apply_json.py tests/test_apply_mock.py -q
  • make check

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.

1 participant