Skip to content

ci: AUT-1408 replace cicd PAT with GitHub App tokens - #280

Open
svcnemo-autobot wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:ci/use-github-app-auth
Open

ci: AUT-1408 replace cicd PAT with GitHub App tokens#280
svcnemo-autobot wants to merge 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:ci/use-github-app-auth

Conversation

@svcnemo-autobot

Copy link
Copy Markdown
Contributor

Background

cicd-main used the long-lived secrets.PAT credential to publish fake coverage statuses.

What changed

Mint a short-lived GitHub App token from vars.BOT_ID and secrets.BOT_KEY, then pass it to actions/github-script.

Details

flowchart LR
  ID[BOT_ID] --> Token[Mint app token]
  Key[BOT_KEY] --> Token
  Token --> Coverage[Publish coverage status]
Loading

The token action is pinned to immutable v3.2.0.

Tested

  • pre-commit run --all-files --show-diff-on-failure --color=never
  • Workflow YAML parsing and token-order assertions
  • Verified 1 token step and zero secrets.PAT references
  • git diff --check

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from a team as a code owner August 7, 2026 22:56
@copy-pr-bot

copy-pr-bot Bot commented Aug 7, 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.

@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Replaces the long-lived PAT used to publish skipped-coverage statuses with a short-lived GitHub App installation token.

  • Adds an immutable-pinned actions/create-github-app-token step using BOT_ID and BOT_KEY.
  • Passes the generated token to actions/github-script while preserving the existing status target and payload.

Confidence Score: 5/5

The PR appears safe to merge, with the generated token targeting the same repository and commit-status operation as the credential it replaces.

The workflow preserves the existing status publication call and introduces the token-generation step before its only consumer; no concrete reachable failure or repository-rule violation was identified.

Important Files Changed

Filename Overview
.github/workflows/cicd-main.yml Replaces the PAT supplied to the fake-coverage status step with a short-lived GitHub App token scoped to the current repository; no actionable defect was established.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    BotId[vars.BOT_ID] --> Mint[Create GitHub App token]
    BotKey[secrets.BOT_KEY] --> Mint
    Mint --> Script[GitHub Script]
    Script --> Status[Publish codecov/patch status]
Loading

Reviews (1): Last reviewed commit: "ci: AUT-1408 replace cicd PAT with GitHu..." | Re-trigger Greptile

@svcnvidia-nemo-ci

Copy link
Copy Markdown

/ok to test 262c762

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.

2 participants