Skip to content

Commit 3f5ee67

Browse files
committed
Merge remote-tracking branch 'origin/main' into feat/b4-g-native-qualification-9144
Signed-off-by: Aaron Erickson <aerickson@nvidia.com> # Conflicts: # .agents/skills/nemoclaw-maintainer-e2e/SKILL.md
2 parents f78d002 + 5275c73 commit 3f5ee67

95 files changed

Lines changed: 4867 additions & 2097 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.agents/skills/nemoclaw-maintainer-cut-release-tag/SKILL.md

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ When the entry is waived, show the recorded waiver reason in the plan presentati
126126

127127
For the plan's full `origin/main` SHA, require a completed, successful `Release qualification` check from a pre-tag `.github/workflows/e2e.yaml` run.
128128
The workflow planner derives the required jobs from the workflow's E2E metadata.
129-
By default, the check requires every release-required E2E execution result, including `Exact staging Brev Launchable`, to succeed.
129+
By default, the check requires every release-required execution result, including `Publish staging Brev Launchable image`, to succeed.
130130
A repository administrator may waive one or more release-required E2E execution jobs for a documented release exception.
131131
The waiver requires a comma-separated `release_qualification_waived_jobs` list and a `release_qualification_waiver_reason`.
132132
The reason must begin with an ASCII letter or digit and contain 10-500 characters chosen from ASCII letters, digits, spaces, and `.,:;/_()'-`.
@@ -168,6 +168,20 @@ Before showing the confirmation prompt, present the candidate SHA, workflow URL,
168168
For a waived run, also present the waived jobs, their outcomes, the waiver reason, and both recorded actor identities.
169169
No release-note-only delta exception is currently defined.
170170

171+
After image publication succeeds, present this advisory manual validation:
172+
173+
- State that the image-publication job built and published the candidate image to the staging family used by the [NemoClaw staging Launchable](https://brev.nvidia.com/launchable/deploy/now?launchableID=env-3GdbIjswX4fs3VJ6cYRHr5zoQXo).
174+
- Encourage the maintainer to deploy one instance and hand its Brev environment URL to a Codex session that invokes `nemoclaw-maintainer-validate-launchable`.
175+
- Require the manual validation to compare the deployed concrete image with `launchable-image.json`; do not assume that the mutable family still points to the candidate.
176+
- State that browser-control capability is required for Codex to click and verify the web interface.
177+
- State that a securely supplied inference credential is required to complete hosted and sandbox inference validation. Never ask the maintainer to paste the credential into chat.
178+
- Record the manual result as `complete pass`, `partially blocked`, `failed`, or `not run` when the maintainer provides it.
179+
180+
This manual validation is advisory while the automated Launchable path is blocked by issue #8924.
181+
Its absence, partial result, or failure does not block the signing preflight, confirmation prompt, or release tag.
182+
Do not describe successful image publication as successful Launchable, runtime, or inference validation.
183+
Apply the temporary policy in [Pre-Tag E2E Evidence](../nemoclaw-maintainer-policies/references/release-train.md#temporary-staging-launchable-qualification-policy): NemoClaw maintainers own it while #8924 remains open, the successful exact image-publication job and artifact remain required release evidence under normal Actions retention, and the full automated lane returns only after a checksum-pinned Brev release passes deployment through verified cleanup on trusted `main`.
184+
171185
Run the release script's signing preflight before asking for confirmation:
172186

173187
```bash
@@ -322,10 +336,11 @@ If the Announcement is valid, return its URL with the release artifacts and mark
322336

323337
- Plan generation fails: fix the named precondition, then regenerate the plan.
324338
- Planned changelog entry is missing or malformed: stop before plan generation and run the pre-tag `nemoclaw-contributor-update-docs` workflow. Use post-release recovery only when the tag already exists.
325-
- Full-mode E2E waits in the Launchable concurrency queue: keep the run pending until the earlier Launchable E2E job finishes.
339+
- Full-mode E2E waits in the Launchable concurrency queue: keep the run pending until the earlier Launchable image-publication job finishes.
326340
- Full-mode E2E ran for another SHA: reject the run and dispatch full mode for the plan candidate SHA.
327341
- No qualifying `Release qualification` exists: inspect the GitHub result and run pre-tag E2E for the planned SHA only when no qualifying run already exists. Use a job waiver only with explicit repository administrator authorization. Do not release until the release script accepts the canonical check.
328-
- Launchable E2E or cleanup fails: inspect the diagnostic artifacts, correct the failure, and rerun the affected E2E work. Do not infer Launchable success from another workflow result.
342+
- Launchable image publication fails: inspect `launchable-image.json` and the producer run, correct the failure, and rerun the affected work. Do not infer image publication from manual Launchable validation.
343+
- Advisory Launchable validation is blocked or fails: record the exact partial result and continue the release flow. Do not convert the result into a release gate or an automated E2E pass.
329344
- `origin/main` moved after plan generation: regenerate the plan and ask for the new confirmation phrase.
330345
- Remote semver tag already exists: stop; do not retag unless the maintainer explicitly starts protected-tag remediation.
331346
- Signing preflight fails: fix the reported Git signer or signing-key failure. Run the preflight again before requesting confirmation.

.agents/skills/nemoclaw-maintainer-e2e/SKILL.md

Lines changed: 30 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: nemoclaw-maintainer-e2e
3-
description: Dispatches and verifies trusted or administrator-authorized GitHub Actions E2E for NemoClaw maintainers, including manual PR E2E for the current PR head commit. Use for requests such as run E2E for PR #123, run native runtime qualification, run the E2E suite, run the Launchable E2E, run the full E2E suite, deploy pre-release full E2E, run pre-tag full E2E, or run release-candidate E2E.
3+
description: Dispatches and verifies trusted or administrator-authorized GitHub Actions E2E for NemoClaw maintainers, including manual PR E2E for the latest PR commit and staging Launchable image publication. Use for requests such as run E2E for PR #123, run native runtime qualification, run the E2E suite, publish the Launchable image, run the Launchable E2E, run the full E2E suite, deploy pre-release full E2E, run pre-tag full E2E, or run release-candidate E2E.
44
---
55

66
<!-- SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. -->
@@ -39,14 +39,14 @@ The workflow does not rotate or revoke these API keys or messaging credentials.
3939
Live targets can create external resources.
4040
After a failure, inspect the artifacts and remove resources that target cleanup did not remove.
4141

42-
`Exact staging Brev Launchable` reads these credentials from repository Actions secrets:
42+
`Publish staging Brev Launchable image` reads this credential from repository Actions secrets:
4343

44-
- `BREV_API_KEY` authenticates the Brev CLI for workspace operations in the organization identified by `BREV_ORG_ID`.
4544
- `NEMOCLAW_IMAGE_DISPATCH_TOKEN` is exposed as `GH_TOKEN` only to the trusted host script. It grants Actions read/write access to `brevdev/nemoclaw-image`, which the script uses to dispatch the image workflow, inspect its run, and download its handoff artifact.
46-
- `NVIDIA_INFERENCE_API_KEY` is exported into the Brev guest for the full E2E process. Code in the baked candidate checkout can read and use it.
4745

48-
These credentials remain valid until they expire or an administrator revokes them in their issuing services. If cleanup fails, remove the recorded Brev workspace. Rotate or revoke each credential to remove later access.
49-
This Brev credential boundary applies only to trusted Launchable or full manual dispatches against `main`. It does not apply to `main` pushes or manual PR runs.
46+
This credential remains valid until it expires or an administrator revokes it in GitHub. Rotate or revoke it to remove later access.
47+
The job does not receive `BREV_API_KEY`, `BREV_ORG_ID`, or `NVIDIA_INFERENCE_API_KEY`.
48+
It does not install or authenticate the Brev CLI, create a workspace, or run inference.
49+
This image-publication credential boundary applies only to trusted Launchable or full manual dispatches against `main`. It does not apply to `main` pushes or manual PR runs.
5050

5151
For `managed-image-protected-runtime`, the workflow supplies the long-lived `NVIDIA_API_KEY` repository secret only to the trusted qualification step. Trusted host code uses it for NGC login and passes it as `NGC_API_KEY` and `NIM_NGC_API_KEY` to the temporary NIM container. Candidate managed sandboxes receive generated local route tokens instead of this key. The live fixture removes the temporary NIM container only if its exact ID, name, requested image, immutable image ID, cohort owner, and provider kind match the recorded authority. The test fails if evidence is missing or ambiguous, a name is reused, authority drifts, removal is indeterminate, or the exact ID or name remains. A cleanup refusal can leave the container and its API key in place until runner teardown. The final workflow step removes the job's isolated Docker credential directory and fails if that removal does not complete. The workflow does not revoke the NVIDIA API key. Revoke it, or rotate it and disable the old value, in the issuing NVIDIA service. Verify that the exposed key is no longer valid.
5252

@@ -76,7 +76,7 @@ Require a review reason containing 10 to 500 printable characters.
7676
Choose exactly one mode:
7777

7878
- For a PR revision run, leave `E2E_JOBS` empty. The run selects:
79-
- every default-selected free-standing workflow E2E except `Exact staging Brev Launchable`;
79+
- every default-selected free-standing workflow E2E except `Publish staging Brev Launchable image`;
8080
- every shared credential-free test; and
8181
- these controller-selected registry targets: `ubuntu-policy-custom-missing-presets-negative`, `ubuntu-repo-cloud-langchain-deepagents-code`, `ubuntu-repo-cloud-openclaw`, and `ubuntu-repo-docker-post-reboot-recovery`.
8282
The run skips `jetson-nvmap-gpu` unless `allow_jetson_dispatch` is `true`.
@@ -522,20 +522,25 @@ A changed head repository, head SHA, or base SHA invalidates the evidence and re
522522
| Request | Mode | `jobs` | `include_staging_brev_launchable` |
523523
|---|---|---|---|
524524
| “Run the E2E suite” | Ordinary | empty | `false` |
525-
| “Run the Launchable E2E” | Launchable | `staging-brev-launchable` | `false` |
525+
| “Publish the Launchable image” | Launchable image | `staging-brev-launchable` | `false` |
526+
| “Run the Launchable E2E” | Clarify before dispatch | not applicable | not applicable |
526527
| “Run the full E2E suite” | Full | empty | `true` |
527528
| “deploy pre-release full E2E” | Full | empty | `true` |
528529
| “run pre-tag full E2E” | Full | empty | `true` |
529530
| “run release-candidate E2E” | Full | empty | `true` |
530531
| “run pre-tag E2E with an administrator job waiver” | Administrator-waived full | empty | `true` |
531532

532533
A generic E2E request must not authorize the Brev Launchable path.
534+
For “Run the Launchable E2E,” explain that issue #8924 blocks automated deployment, runtime, and inference validation.
535+
Ask whether the maintainer wants image publication or advisory validation through `nemoclaw-maintainer-validate-launchable` against one deployed instance.
536+
Do not dispatch until the maintainer selects one of those operations.
533537
Do not infer full mode from words such as “all” or “complete.”
534-
Ask for clarification only when the request contains conflicting mode phrases.
538+
Ask for clarification when the request uses the legacy Launchable E2E phrase or contains conflicting mode phrases.
535539

536-
Ordinary mode selects every default-selected workflow E2E except `Exact staging Brev Launchable`.
537-
Launchable mode runs only `Exact staging Brev Launchable`.
538-
Full mode adds `Exact staging Brev Launchable` to the default E2E selection in the same workflow run.
540+
Ordinary mode selects every default-selected workflow E2E except `Publish staging Brev Launchable image`.
541+
Launchable image mode runs only `Publish staging Brev Launchable image`.
542+
Full mode adds `Publish staging Brev Launchable image` to the default E2E selection in the same workflow run.
543+
The Launchable image job stops after exact image-publication evidence and does not deploy a workspace or run inference.
539544
Administrator-waived full mode runs the full suite but omits the approved execution jobs from release qualification.
540545
Every waived job still runs.
541546
Use this mode only when a repository administrator explicitly authorizes the job IDs and supplies the reason.
@@ -581,7 +586,7 @@ gh workflow run .github/workflows/e2e.yaml \
581586
-f "correlation_id=${CORRELATION_ID}"
582587
```
583588

584-
For Launchable mode:
589+
For Launchable image mode:
585590

586591
```bash
587592
gh workflow run .github/workflows/e2e.yaml \
@@ -632,8 +637,8 @@ gh workflow run .github/workflows/e2e.yaml \
632637
```
633638

634639
Do not set `jobs=staging-brev-launchable` for full mode.
635-
Empty `jobs` and `targets` select every default-selected workflow E2E except `Exact staging Brev Launchable`.
636-
The `include_staging_brev_launchable` input adds the Launchable E2E job to that same run.
640+
Empty `jobs` and `targets` select every default-selected workflow E2E except `Publish staging Brev Launchable image`.
641+
The `include_staging_brev_launchable` input adds the Launchable image-publication job to that same run.
637642
The trusted `main` workflow verifies that the dispatching and rerunning actors have
638643
repository `maintain` or `admin` permission before the Launchable path's source
639644
checkout. That role check is the authorization.
@@ -659,7 +664,7 @@ empty-selector manual run or enable explicit qualification selection. Set it
659664
only after a repository administrator confirms an online DGX Spark runner in
660665
the authoritative runner inventory.
661666
If GitHub pauses the qualification job for the `approve-dgx-spark-image-qualification` environment, an authorized environment reviewer must approve it before qualification starts.
662-
`Exact staging Brev Launchable` does not require environment approval.
667+
`Publish staging Brev Launchable image` does not require environment approval.
663668

664669
Find the run by its unique title:
665670

@@ -693,7 +698,7 @@ Wait for completion:
693698
gh run watch "$RUN_ID" --repo NVIDIA/NemoClaw
694699
```
695700

696-
Launchable and full modes can wait in the non-cancelling Launchable concurrency queue.
701+
Launchable image and full modes can wait in the non-cancelling Launchable concurrency queue.
697702
Queued, waiting, or accepted dispatch state is not success.
698703
Classify the completed workflow and `Release qualification` job with the checks below.
699704

@@ -715,20 +720,21 @@ Require `run-$RUN_ID.json` to report:
715720
- `head_sha` equal to `CANDIDATE_SHA`;
716721
- `status` equal to `completed`.
717722

718-
For ordinary, Launchable, and unwaived full modes, require `conclusion` equal to `success`.
723+
For ordinary, Launchable image, and unwaived full modes, require `conclusion` equal to `success`.
719724
For administrator-waived full mode, permit `conclusion` equal to `success` or `failure`.
720725
A `failure` conclusion is acceptable only when one completed, successful `Release qualification` job and a valid exact-run waiver artifact with at least one canonical waived job failure both exist.
721726

722-
For Launchable mode, also require `jobs-latest-$RUN_ID.json` to contain one completed, successful
723-
`Exact staging Brev Launchable` job. Return the workflow and job URLs.
727+
For Launchable image mode, also require `jobs-latest-$RUN_ID.json` to contain one completed, successful
728+
`Publish staging Brev Launchable image` job. Return the workflow and job URLs.
729+
Require its artifact to contain `launchable-image.json` for the selected candidate SHA and concrete staging image URI.
724730

725731
For a full run, with or without a job waiver, require `jobs-latest-$RUN_ID.json` to contain one completed, successful
726732
`Release qualification` job. Return its job URL with the workflow URL.
727-
In full mode, that job waits for every default-required E2E result, including `Exact staging Brev Launchable`.
728-
The Launchable job directly verifies the candidate checkout, in-guest full E2E result, and workspace cleanup before it succeeds.
729-
Its `launchable-e2e.json`, `full-e2e.log`, and `cleanup.json` artifacts remain available for diagnosis.
733+
In full mode, that job waits for every default-required result, including `Publish staging Brev Launchable image`.
734+
The Launchable image job verifies only the exact candidate image producer receipt and staging-family publication.
735+
Its `launchable-image.json` artifact records Launchable, runtime, and inference validation as not run.
730736
A skipped, cancelled, queued, or failed `Release qualification` job is not evidence.
731-
A Launchable-only run is not full-mode or pre-tag release evidence.
737+
A Launchable image-only run is not full-mode or pre-tag release evidence.
732738

733739
For administrator-waived full mode, the job waits for every unwaived release-required result.
734740
A waived execution job may fail without failing `Release qualification`.

0 commit comments

Comments
 (0)