Skip to content

Add OCUDU as a submodule#144

Merged
gab-arrobo merged 2 commits intoopennetworkinglab:mainfrom
gab-arrobo:ocudu-submodule
Mar 27, 2026
Merged

Add OCUDU as a submodule#144
gab-arrobo merged 2 commits intoopennetworkinglab:mainfrom
gab-arrobo:ocudu-submodule

Conversation

@gab-arrobo
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Arrobo, Gabriel <[email protected]>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds OCUDU support to the OnRamp repo by introducing an OCUDU submodule and wiring it into the existing blueprint/config and Makefile entrypoints.

Changes:

  • Adds deps/ocudu as a git submodule.
  • Introduces a new OCUDU blueprint (vars/main-ocudu.yml) and documents it in vars/README.md.
  • Adds OCUDU inventory group and Makefile targets/includes to manage OCUDU components.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
vars/main-ocudu.yml New OCUDU-based blueprint configuration (core + ocudu + amp variables).
vars/README.md Adds the OCUDU blueprint entry and documentation link.
hosts.ini Adds an ocudu_nodes inventory group stub.
Makefile Exposes OCUDU make targets and includes the OCUDU submodule Makefile.
.gitmodules Registers deps/ocudu as a submodule.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Arrobo, Gabriel <[email protected]>
Copy link
Copy Markdown
Contributor

@andybavier andybavier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@gab-arrobo gab-arrobo merged commit 0f0943a into opennetworkinglab:main Mar 27, 2026
1 check passed
@gab-arrobo gab-arrobo deleted the ocudu-submodule branch March 27, 2026 19:12
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