Skip to content

CI: control-plane workflow (cross-repo connection in CI) - #44

Merged
Lothnic merged 2 commits into
mainfrom
ci-workflows
Aug 3, 2026
Merged

CI: control-plane workflow (cross-repo connection in CI)#44
Lothnic merged 2 commits into
mainfrom
ci-workflows

Conversation

@Lothnic

@Lothnic Lothnic commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Adds .github/workflows/control.yml alongside the existing gateway CI (untouched).

What

A dedicated control-plane job that:

  • installs .[control,dev] and checks out + installs meridian-node,
  • lint + type-checks meridian_control,
  • runs pytest tests/control — which includes the real node ↔ control cross-repo tests (the compatibility target) over a live socket with mTLS enforced,
  • runs scripts/verify_connection.py (the node↔control↔gateway numbers report).

This wires the cross-repo compatibility target into CI, so the connection is verified on every push/PR.

🤖 Generated with Claude Code

Adds a dedicated control-plane job: installs the [control] extra, checks out
meridian-node and installs it, then lint/type-checks meridian_control and runs
tests/control — which includes the real node<->control cross-repo tests (the
compatibility target) and the verify_connection.py numbers report. Additive;
does not touch the existing gateway CI.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 3, 2026 19:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The node repo is private, so the default token can't check it out. Make the
checkout optional (continue-on-error + optional MERIDIAN_NODE_TOKEN secret);
when it's unavailable the cross-repo tests skip via importorskip and the rest of
the control suite still runs. Drop the verify_connection step from CI (it needs
the node agent; it stays a local/on-demand tool).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Lothnic
Lothnic merged commit c7762a4 into main Aug 3, 2026
6 checks passed
@Lothnic
Lothnic deleted the ci-workflows branch August 3, 2026 20:00
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