Skip to content

docs: add ncx-infra-controller-rest repo reference to site-setup guide#582

Open
sabbanis wants to merge 1 commit intoNVIDIA:mainfrom
sabbanis:fix/510-add-rest-repo-reference-to-site-setup
Open

docs: add ncx-infra-controller-rest repo reference to site-setup guide#582
sabbanis wants to merge 1 commit intoNVIDIA:mainfrom
sabbanis:fix/510-add-rest-repo-reference-to-site-setup

Conversation

@sabbanis
Copy link

Summary

Fixes #510

The site-setup guide listed the NICo REST components (cloud-api, cloud-workflow, cloud-cert-manager, elektra-site-agent) without ever linking to their open-source repository. Readers had no way to know where to source these components.

  • Added a sentence in the NICo Components section pointing to ncx-infra-controller-rest before the REST component list
  • Added an inline link in Order of Operations step 4 so readers see the reference at the point of action

Test plan

🤖 Generated with Claude Code

The NICo Components section and Order of Operations step 4 listed the
REST components (cloud-api, cloud-workflow, cloud-cert-manager,
elektra-site-agent) without linking to their open-source repository.
Added explicit references to https://github.com/NVIDIA/ncx-infra-controller-rest
so readers know where to source these components.

Fixes NVIDIA#510

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sabbanis sabbanis requested a review from a team as a code owner March 16, 2026 21:44
@copy-pr-bot
Copy link

copy-pr-bot bot commented Mar 16, 2026

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.

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.

[DOC]: Wrong/missing reference to BMM-Rest components

2 participants