Skip to content

patch(0.10.0) update refs to actions - #5006

Merged
nv-dmendoza merged 2 commits into
NVIDIA:release/v0.10.0from
nv-dmendoza:backport-4895-0-10-0
Aug 14, 2026
Merged

patch(0.10.0) update refs to actions#5006
nv-dmendoza merged 2 commits into
NVIDIA:release/v0.10.0from
nv-dmendoza:backport-4895-0-10-0

Conversation

@nv-dmendoza

Copy link
Copy Markdown
Collaborator

Backports #4895

The repo NVIDIA/dsx-github-actions was transferred to dsx-ai-factory/dsx-github-actions. GitHub Actions resolves action/workflow references differently depending on where they appear in a workflow file:

  • Job-level reusable workflow calls (jobs.<id>.uses:) are resolved at parse time. If the target can't be found, GitHub rejects the entire workflow file before any jobs run — this is what caused the hard failure in ci.yaml ([run 31631294473](https://github.com/NVIDIA/infra-controller/actions/runs/31631294473)).
  • Step-level action references (steps[*].uses:) are resolved at runtime. GitHub still follows HTTP redirects for these, so workflows like rest-ci.yml that only had step-level references continued to trigger and pass despite the stale org name.

Because step-level references silently kept working via redirect, the breakage was not uniform — only workflows with job-level reusable workflow calls hard-failed. This PR updates all references regardless of kind for consistency and to avoid relying on redirect behavior that GitHub can revoke at any time.

Related issues

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

The repo `NVIDIA/dsx-github-actions` was transferred to
`dsx-ai-factory/dsx-github-actions`. GitHub Actions resolves
action/workflow references differently depending on where they appear in
a workflow file:

- **Job-level reusable workflow calls** (`jobs.<id>.uses:`) are resolved
at parse time. If the target can't be found, GitHub rejects the entire
workflow file before any jobs run — this is what caused the hard failure
in `ci.yaml` ([[run
31631294473](https://github.com/NVIDIA/infra-controller/actions/runs/31631294473)](https://github.com/NVIDIA/infra-controller/actions/runs/31631294473)).
- **Step-level action references** (`steps[*].uses:`) are resolved at
runtime. GitHub still follows HTTP redirects for these, so workflows
like `rest-ci.yml` that only had step-level references continued to
trigger and pass despite the stale org name.

Because step-level references silently kept working via redirect, the
breakage was not uniform — only workflows with job-level reusable
workflow calls hard-failed. This PR updates all references regardless of
kind for consistency and to avoid relying on redirect behavior that
GitHub can revoke at any time.

- [ ] **Add** - New feature or capability
- [ ] **Change** - Changes in existing functionality
- [ ] **Fix** - Bug fixes
- [ ] **Remove** - Removed features or deprecated functionality
- [x] **Internal** - Internal changes (refactoring, tests, docs, etc.)

- [ ] **This PR contains breaking changes**

- [ ] Unit tests added/updated
- [ ] Integration tests added/updated
- [ ] Manual testing performed
- [ ] No testing required (docs, internal refactor, etc.)
@nv-dmendoza
nv-dmendoza requested a review from a team August 14, 2026 19:18
@nv-dmendoza
nv-dmendoza requested review from a team and lauragra-y as code owners August 14, 2026 19:18
@thossain-nv thossain-nv added the rest-api Add this label when an issue or PR concerns NICo REST API label Aug 14, 2026 — with ChatGPT Codex Connector
@nv-dmendoza
nv-dmendoza merged commit 03948a1 into NVIDIA:release/v0.10.0 Aug 14, 2026
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rest-api Add this label when an issue or PR concerns NICo REST API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants