Commit 482ab5a
authored
ci: raise agent audit turn limit and preserve logs (#571)
* ci: raise agent audit turn limit and preserve logs
The Friday test-health audit hit the 30-turn cap on its first-ever run
(2026-04-24) and the agent log was discarded with the self-hosted
runner. Heavier recipes need more room, and the next failure should be
diagnosable.
- Raise --max-turns from 30 to 50
- Switch --output-format from text to stream-json so events are emitted
during the run instead of only at process exit; prefix with
stdbuf -oL -eL to line-buffer the pipe
- Upload /tmp/claude-audit-log.txt and /tmp/audit-<suite>.md as an
artifact (if: always(), 14-day retention) using the upload-artifact
SHA already pinned in build-notebooks.yml
Signed-off-by: Andre Manoel <amanoel@nvidia.com>
* ci: disambiguate audit artifact name across run attempts
actions/upload-artifact@v4+ rejects duplicate names within a workflow,
and re-running a failed run reuses the same github.run_id. Append
github.run_attempt so re-runs upload successfully instead of failing at
the exact moment the artifact is most useful.
Found by Codex review of #571.
Signed-off-by: Andre Manoel <amanoel@nvidia.com>
* ci: only upload agent log on failure
Raise the bar for persisting the full verbose stream-json event log:
we only need it when we're actually debugging a failure, and the audit
report itself still lands in the step summary on success. Shrinks the
window where tool inputs, read file contents, or other verbose-stream
detail could end up in a 14-day artifact.
Addresses the minor privacy finding from Codex review of #571.
Signed-off-by: Andre Manoel <amanoel@nvidia.com>
* ci: drop raw agent log from job summary
With --output-format stream-json the previous tail -100 of the agent
log emitted raw NDJSON into the GH Actions UI summary, which is
unreadable. The audit report itself (/tmp/audit-<suite>.md) already
carries the human-readable payload, and the full event stream is
available as an on-failure artifact, so the raw tail was redundant and
worse than nothing for the summary surface.
Also rewords the fallback message to point at the artifact when no
report lands (typically a failure).
Signed-off-by: Andre Manoel <amanoel@nvidia.com>
---------
Signed-off-by: Andre Manoel <amanoel@nvidia.com>1 parent e92c76b commit 482ab5a
1 file changed
Lines changed: 15 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
174 | 174 | | |
175 | 175 | | |
176 | 176 | | |
177 | | - | |
| 177 | + | |
178 | 178 | | |
179 | 179 | | |
180 | | - | |
181 | | - | |
| 180 | + | |
| 181 | + | |
182 | 182 | | |
183 | 183 | | |
184 | 184 | | |
| |||
205 | 205 | | |
206 | 206 | | |
207 | 207 | | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
208 | 219 | | |
209 | 220 | | |
210 | 221 | | |
| |||
216 | 227 | | |
217 | 228 | | |
218 | 229 | | |
219 | | - | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | | - | |
| 230 | + | |
230 | 231 | | |
0 commit comments