Skip to content

Record node status snapshots before runs - #209

Merged
yoshifuminakamura merged 1 commit into
developfrom
feature/node-status-snapshot
Sep 15, 2026
Merged

yoshifuminakamura merged 1 commit into
developfrom
feature/node-status-snapshot

Conversation

@yoshifuminakamura

Copy link
Copy Markdown
Collaborator

Summary:

  • record a scheduler-neutral node status snapshot before benchmark runs
  • store the full snapshot as a measurement artifact while keeping Result JSON to summary, hash, and artifact reference
  • show the snapshot only on the console detail surface; keep it hidden from the public surface

Validation:

  • .venv/bin/python -m pytest result_server/tests/test_result_detail_template.py result_server/tests/test_measurement_artifacts.py result_server/tests/test_result_padata_route.py result_server/tests/test_public_result_routes.py result_server/tests/test_api_routes.py
  • bash scripts/tests/test_node_status_snapshot.sh
  • bash scripts/tests/test_process_and_send_results.sh
  • bash scripts/tests/test_result_common_json_contract.sh
  • bash scripts/tests/run_profile_data_shell_tests.sh common
  • bash -n for touched shell scripts
  • shellcheck -S error for touched shell scripts
  • git diff --check

Review note:

  • checked the PR diff for sensitive or environment-specific values; the public surface does not expose this artifact.

Signed-off-by: yoshifuminakamura <nakamura@riken.jp>
@yoshifuminakamura
yoshifuminakamura force-pushed the feature/node-status-snapshot branch from 22d541c to c04c847 Compare September 14, 2026 23:59
@yoshifuminakamura
yoshifuminakamura merged commit 70f4f78 into develop Sep 15, 2026
10 of 11 checks passed
@yoshifuminakamura
yoshifuminakamura deleted the feature/node-status-snapshot branch September 15, 2026 00:56
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