Skip to content

Add amp prefix to follow the same pattern as in the other blueprints#86

Merged
gab-arrobo merged 1 commit intoopennetworkinglab:mainfrom
gab-arrobo:add-amp-prefix
Apr 16, 2026
Merged

Add amp prefix to follow the same pattern as in the other blueprints#86
gab-arrobo merged 1 commit intoopennetworkinglab:mainfrom
gab-arrobo:add-amp-prefix

Conversation

@gab-arrobo
Copy link
Copy Markdown
Contributor

No description provided.

@gab-arrobo gab-arrobo requested review from a team as code owners April 16, 2026 06:47
Copilot AI review requested due to automatic review settings April 16, 2026 06:47
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

Updates OnRamp documentation to use the AMP-prefixed Make targets (e.g., amp-roc-*, amp-monitor-*) to match the naming pattern used by other blueprints and targets.

Changes:

  • Replaced ROC Make targets with amp-roc-* in ROC-related docs.
  • Replaced Monitoring Make targets with amp-monitor-* in inspection/reference docs.
  • Updated blueprint documentation to reference amp-roc-load instead of roc-load.

Reviewed changes

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

File Description
onramp/roc.rst Updates ROC setup commands to amp-roc-* targets.
onramp/ref.rst Updates reference table entries to amp-roc-* / amp-monitor-* targets.
onramp/inspect.rst Updates inspect workflow commands to amp-roc-* / amp-monitor-* targets.
onramp/blueprints.rst Updates blueprint docs to reference amp-roc-load and AMP ROC install/load commands.

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

Comment thread onramp/blueprints.rst Outdated
Comment thread onramp/blueprints.rst Outdated
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
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

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


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

@gab-arrobo gab-arrobo merged commit 6e6ad59 into opennetworkinglab:main Apr 16, 2026
6 checks passed
@gab-arrobo gab-arrobo deleted the add-amp-prefix branch April 16, 2026 07:15
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