Skip to content

cp: use GitHub App authentication for code freeze - #311

Merged
ko3n1g merged 2 commits into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:chore/cherrypick-307-to-main
Oct 19, 2026
Merged

ko3n1g merged 2 commits into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:chore/cherrypick-307-to-main

Conversation

@svcnemo-autobot

Copy link
Copy Markdown
Contributor

Background

Copy #307 at pinned source head 5f6a8b897d9b1deaaf2e37aa4b7408cd74b430a1 onto main.

What changed

Switch code-freeze authentication to the GitHub App and update the shared workflow pin.

Details

Both source commits retain cherry-pick provenance. Only .github/workflows/release-freeze.yml differs; its final blob matches the pinned source.

flowchart LR
  Dispatch --> Wrapper[Code-freeze wrapper]
  Wrapper --> Shared[Pinned shared workflow]
  App[GitHub App credentials] --> Shared
Loading

Tested

  • Native changed-file pre-commit completed; Python/type/Markdown hooks skipped by their own filters.
  • Diff whitespace, source identity, signatures, signoffs, and remote head verified.
  • Product/GPU tests excluded: product/dependency inputs unchanged.
  • actionlint unavailable due denied asset access. Required CI and secret-plumbing approval pending.

Signed-off-by: oliver könig <okoenig@nvidia.com>
(cherry picked from commit ad155ef)
Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
(cherry picked from commit 5f6a8b8)
Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from a team as a code owner October 18, 2026 19:33
@copy-pr-bot

copy-pr-bot Bot commented Oct 18, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@svcnvidia-nemo-ci

Copy link
Copy Markdown

/ok to test 206fd76

@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Critical impact] The available changes appear safe to merge; no concrete defect was found.

Summary

Updates code-freeze authentication to pass GitHub App credentials to a newly pinned shared workflow.

  • Code-freeze runs through the GitHub App now.

No actionable issues found in the available code. The pinned shared workflow and live credential setup were not verified. No workflow run was performed.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Dispatch[Manual dispatch] --> Wrapper[Code-freeze workflow]
  Credentials[GitHub App ID and key] --> Wrapper
  Wrapper --> Shared[Pinned shared workflow]
  Environment[public environment input] --> Shared
Loading

Reviews (1) · Last reviewed commit: "ci(code-freeze): pin merged github app w..." · Reviewed by Greptile

@ko3n1g
ko3n1g merged commit d1b3de9 into NVIDIA-NeMo:main Oct 19, 2026
31 checks passed

This branch was successfully deployed

2 active deployments
public — 206fd76b Deployed Oct 18, 2026 by copy-pr-bot[bot] via release / finalize / notify #282
test — 206fd76b Deployed Oct 18, 2026 by copy-pr-bot[bot] via cicd-wait-in-queue #1091
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.

3 participants