Skip to content

Air gapped setup#64

Open
r2dedios wants to merge 6 commits into
RHEcosystemAppEng:mainfrom
r2dedios:air-gapped-setup
Open

Air gapped setup#64
r2dedios wants to merge 6 commits into
RHEcosystemAppEng:mainfrom
r2dedios:air-gapped-setup

Conversation

@r2dedios
Copy link
Copy Markdown
Contributor

@r2dedios r2dedios commented Apr 22, 2026

Summary

Pack(s) affected

  • rh-sre
  • rh-developer
  • ocp-admin
  • rh-virt
  • rh-ai-engineer
  • Other / repo-wide

Change type

  • New skill
  • New agent
  • New pack
  • Update existing skill / agent
  • MCP server config (mcps.json)
  • Docs / README
  • CI / tooling

CLAUDE.md compliance

  • Agents orchestrate skills; no direct MCP/tool calls in agents
  • Skills are single-purpose task executors
  • Skills encapsulate all tool access (MCP tools invoked only inside skills)
  • Document consultation: file is read with the Read tool, then declared to the user
  • No credentials hardcoded; env vars used via ${VAR} references
  • Human-in-the-loop confirmation added for any destructive or critical operations

Validation

  • make validate passes locally
  • New/changed skills have valid YAML frontmatter (name, description)
  • New/changed agents have valid YAML frontmatter (name, description)

Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
…CP+Opencode in air gapped environments

Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
@r2dedios r2dedios requested a review from dmartinol April 22, 2026 08:47
@r2dedios r2dedios self-assigned this Apr 22, 2026
@r2dedios r2dedios added the documentation Improvements or additions to documentation label Apr 22, 2026
Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
Copy link
Copy Markdown
Collaborator

@dmartinol dmartinol left a comment

Choose a reason for hiding this comment

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

My 2 cents:
Just add a small AIRGAPPED.md file to the main README with the findings in terms of validated environments, like: RHOAI version and configuration, validated model with ad-hoc notes if required (e.g., any functional or system issues detected?), main system specification (GPUs, etc) and move all the other files to a dedicated repo

@r2dedios r2dedios added the invalid This doesn't seem right label May 11, 2026
@r2dedios
Copy link
Copy Markdown
Contributor Author

Moving this PR and its content to a separate repository

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

Labels

documentation Improvements or additions to documentation invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants