Skip to content

Handle caught late cache probe exits - #132

Merged
yoshifuminakamura merged 1 commit into
developfrom
feature/late-restore-caught-probe
Aug 26, 2026
Merged

Handle caught late cache probe exits#132
yoshifuminakamura merged 1 commit into
developfrom
feature/late-restore-caught-probe

Conversation

@yoshifuminakamura

Copy link
Copy Markdown
Collaborator

Summary

  • treat a late restore probe status file as authoritative even when build.sh catches the build tool exit and returns a different nonzero code
  • preserve fallback behavior by rerunning build.sh normally if restore validation still misses
  • extend the autotools-style cache test to wrap the final make in an if/exit path like GENESIS Fugaku

Safety

  • common cache wrapper/test-only change
  • diff scan found no token/secret-like values

Tests

  • bash scripts/tests/test_build_cache.sh
  • bash scripts/tests/test_build_environment_snapshot.sh
  • shellcheck -S error scripts/build_with_cache.sh scripts/build_tool_wrappers/bk_build_tool_wrapper.sh scripts/collect_environment_snapshot.sh scripts/tests/test_build_cache.sh
  • bash -n scripts/build_with_cache.sh scripts/build_tool_wrappers/bk_build_tool_wrapper.sh scripts/collect_environment_snapshot.sh scripts/tests/test_build_cache.sh
  • /home/nakamura/fugakunext/venv/bin/python -m pytest result_server/tests/test_environment_snapshot_ci_scripts.py

Signed-off-by: yoshifuminakamura <nakamura@riken.jp>
@yoshifuminakamura
yoshifuminakamura merged commit 0acbaa7 into develop Aug 26, 2026
7 checks passed
@yoshifuminakamura
yoshifuminakamura deleted the feature/late-restore-caught-probe branch August 26, 2026 01:02
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