Skip to content

fix(ci): AUT-1086 authenticate docs Hub requests - #3354

Open
svcnemo-autobot wants to merge 2 commits into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:fix/ci-authenticate-docs-hub
Open

fix(ci): AUT-1086 authenticate docs Hub requests#3354
svcnemo-autobot wants to merge 2 commits into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:fix/ci-authenticate-docs-hub

Conversation

@svcnemo-autobot

@svcnemo-autobot svcnemo-autobot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Background

Main run 30125214449 failed in Docs_Tests with Hugging Face HTTP 429. Credentials must never reach PR-controlled code.

What changed

  • Authenticate Hub requests only on trusted non-PR refs.
  • Run PR-ref docs tests without a token and with HF_HUB_OFFLINE=1.

Details

cicd-main.yml passes an empty HF_TOKEN for pull-request/* and marks those composite-action runs as fork PRs. The existing action then enables offline Hub mode. Main and scheduled jobs retain authenticated access.

Tested

  • Workflow YAML parsing and git diff --check.
  • Full NeMo-RL pre-commit gate with recursive submodules.
  • Verified PR refs get no token and enable offline mode; trusted refs retain token support.

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from a team as a code owner July 26, 2026 23:27
@svcnemo-autobot
svcnemo-autobot requested a review from ko3n1g July 26, 2026 23:27
@copy-pr-bot

copy-pr-bot Bot commented Jul 26, 2026

Copy link
Copy Markdown

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the CI Relating to CI label Jul 26, 2026
Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot svcnemo-autobot added CI:docs Run doctest labels Jul 26, 2026
@svcnemo-autobot

Copy link
Copy Markdown
Contributor Author

/ok to test cb7cf86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:docs Run doctest CI Relating to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant