Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 18 additions & 3 deletions .agents/skills/nemoclaw-maintainer-cut-release-tag/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ Read the generated `plan.json` and show the maintainer:

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

After image publication succeeds, present this advisory manual validation:

- 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).
- Encourage the maintainer to deploy one instance and hand its Brev environment URL to a Codex session that invokes `nemoclaw-maintainer-validate-launchable`.
- 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.
- State that browser-control capability is required for Codex to click and verify the web interface.
- 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.
- Record the manual result as `complete pass`, `partially blocked`, `failed`, or `not run` when the maintainer provides it.

This manual validation is advisory while the automated Launchable path is blocked by issue #8924.
Its absence, partial result, or failure does not block the signing preflight, confirmation prompt, or release tag.
Do not describe successful image publication as successful Launchable, runtime, or inference validation.
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`.

Run the release script's signing preflight before asking for confirmation:

```bash
Expand Down Expand Up @@ -323,10 +337,11 @@ If the Announcement is valid, return its URL with the release artifacts and mark
- Plan generation fails: fix the named precondition, then regenerate the plan.
- Documentation workflow state is incomplete: return to `nemoclaw-maintainer-evening`, then repeat
Step 1 after the documentation PR merges.
- Full-mode E2E waits in the Launchable concurrency queue: keep the run pending until the earlier Launchable E2E job finishes.
- Full-mode E2E waits in the Launchable concurrency queue: keep the run pending until the earlier Launchable image-publication job finishes.
- Full-mode E2E ran for another SHA: reject the run and dispatch full mode for the plan candidate SHA.
- 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.
- 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.
- 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.
- 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.
- `origin/main` moved after plan generation: regenerate the plan and ask for the new confirmation phrase.
- Remote semver tag already exists: stop; do not retag unless the maintainer explicitly starts protected-tag remediation.
- Signing preflight fails: fix the reported Git signer or signing-key failure. Run the preflight again before requesting confirmation.
Expand Down
57 changes: 30 additions & 27 deletions .agents/skills/nemoclaw-maintainer-e2e/SKILL.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
name: nemoclaw-maintainer-e2e
description: Dispatches and verifies trusted GitHub Actions E2E for NemoClaw maintainers, including manual PR E2E for the latest PR commit. Use for requests such as run E2E for PR #123, 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.
description: Dispatches and verifies trusted 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 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.
---

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

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

- `BREV_API_KEY` authenticates the trusted host-side Brev CLI for workspace operations in the organization identified by `BREV_ORG_ID`. Candidate code does not receive this API key.
- `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.
- `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.

`brev login` writes `BREV_API_KEY` and `BREV_ORG_ID` to `$HOME/.brev/credentials.json` on the GitHub-hosted runner. Later trusted steps and processes in the same job can read that file. The workflow does not delete it explicitly; it remains on the ephemeral runner filesystem until runner teardown discards that filesystem.
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.
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.

The `NEMOCLAW_STAGING_LAUNCHABLE_ID` repository Actions variable selects the standing Launchable. Keep its value equal to the Launchable ID in the default URL owned by [`nemoclaw-maintainer-validate-launchable`](../nemoclaw-maintainer-validate-launchable/SKILL.md).
This credential remains valid until it expires or an administrator revokes it in GitHub. Rotate or revoke it to remove later access.
The job does not receive `BREV_API_KEY`, `BREV_ORG_ID`, or `NVIDIA_INFERENCE_API_KEY`.
It does not install or authenticate the Brev CLI, create a workspace, or run inference.
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.

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.

Expand All @@ -70,7 +67,7 @@ Require a review reason containing 10 to 500 printable characters.
Choose exactly one mode:

- For a PR revision run, leave `E2E_JOBS` empty. The run selects:
- every default-selected free-standing workflow E2E except `Exact staging Brev Launchable`;
- every default-selected free-standing workflow E2E except `Publish staging Brev Launchable image`;
- every shared credential-free test; and
- 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`.
The run skips `jetson-nvmap-gpu` unless `allow_jetson_dispatch` is `true`.
Expand Down Expand Up @@ -161,20 +158,25 @@ A changed head repository, head SHA, or base SHA invalidates the evidence and re
| Request | Mode | `jobs` | `include_staging_brev_launchable` |
|---|---|---|---|
| “Run the E2E suite” | Ordinary | empty | `false` |
| “Run the Launchable E2E” | Launchable | `staging-brev-launchable` | `false` |
| “Publish the Launchable image” | Launchable image | `staging-brev-launchable` | `false` |
| “Run the Launchable E2E” | Clarify before dispatch | not applicable | not applicable |
| “Run the full E2E suite” | Full | empty | `true` |
| “deploy pre-release full E2E” | Full | empty | `true` |
| “run pre-tag full E2E” | Full | empty | `true` |
| “run release-candidate E2E” | Full | empty | `true` |
| “run pre-tag E2E with an administrator job waiver” | Administrator-waived full | empty | `true` |

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

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

For Launchable mode:
For Launchable image mode:

```bash
gh workflow run .github/workflows/e2e.yaml \
Expand Down Expand Up @@ -271,8 +273,8 @@ gh workflow run .github/workflows/e2e.yaml \
```

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

Find the run by its unique title:

Expand Down Expand Up @@ -332,7 +334,7 @@ Wait for completion:
gh run watch "$RUN_ID" --repo NVIDIA/NemoClaw
```

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

Expand All @@ -354,20 +356,21 @@ Require `run-$RUN_ID.json` to report:
- `head_sha` equal to `CANDIDATE_SHA`;
- `status` equal to `completed`.

For ordinary, Launchable, and unwaived full modes, require `conclusion` equal to `success`.
For ordinary, Launchable image, and unwaived full modes, require `conclusion` equal to `success`.
For administrator-waived full mode, permit `conclusion` equal to `success` or `failure`.
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.

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

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

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