Skip to content

Commit e64f0bd

Browse files
committed
Fix nnf_cmd exit code and update submodule pointers
Preserve the exit code of release-all.sh through the pipe in nnf_cmd by returning PIPESTATUS[0]. Previously, all invocations exited 0 because the pipe's exit code was that of 'cat', not release-all.sh. Also advance submodule pointers to their current master HEADs: - lustre-fs-operator: 7ebec7d -> c59379b (PR #125 update-vendor) - nnf-dm: 42a3d3a4 -> c959e3b5 (PR #360 update-vendor + others) - nnf-integration-test: b967cc0 -> 3e52df1 (PR #160 update-vendor + #159) - nnf-sos: 8749851d -> bcb00b92 (PR #604 update-vendor + others) Also restore the ONE-command-per-turn rule to release.agent.md and remove trailing blank lines from RELEASE-PLAN.md. Signed-off-by: Anthony Floeder <anthony.floeder@hpe.com>
1 parent b8fe628 commit e64f0bd

7 files changed

Lines changed: 6 additions & 6 deletions

File tree

.github/agents/release.agent.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ Before starting any release work, read these files completely:
1818
## Core Rules
1919

2020
- **ALWAYS wait for explicit user approval before proceeding to the next step.** After every step, state what you did, whether it succeeded, and the evidence. **WAIT for the user.**
21+
- **ONE command per turn.** Run the command for ONE repo, show the full output, state PASS or FAIL with evidence, then STOP. Do not run the next repo's command until the user explicitly says to proceed. This applies even when the plan shows a `for` loop — execute the loop body manually, one repo per turn.
2122
- **ALWAYS run commands in a terminal** so the user can see them.
2223
- **ALWAYS append `2>&1 | cat`** to every `release-all.sh` and `gh` command — no exceptions.
2324
- **ONE repo at a time.** Execute one phase per repo sequentially. **NEVER parallelize** across repos.

nnf-dm

Submodule nnf-dm updated 135 files

nnf-integration-test

tools/release-all/RELEASE-PLAN.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -270,8 +270,6 @@ The `-i` flag displays image version changes inline. Use `-d` to display the ful
270270

271271
Check each repo for correct tag, release notes, and artifacts (`manifests.tar` + `manifests-kind.tar` on `nnf-deploy`).
272272

273-
274-
275273
---
276274

277275
### Relevant files

tools/release-all/setup-release-env.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ nnf_cmd() {
8181
local args=()
8282
[[ "$repo" == "nnf_sos" ]] && args+=("-M")
8383
./release-all.sh -B "$RELEASE_TYPE" -P "$phase" -R "$repo" "${args[@]}" 2>&1 | cat
84+
return "${PIPESTATUS[0]}"
8485
}
8586

8687
# --- Helper: create-pr + capture PR number -----------------------------------

0 commit comments

Comments
 (0)