Skip to content

feat(argo-workflows): configure enrollment post hooks via ConfigMap - #2602

Merged
cardoe merged 2 commits into
mainfrom
enroll-server-post-hooks
Oct 28, 2026
Merged

cardoe merged 2 commits into
mainfrom
enroll-server-post-hooks

Conversation

@cardoe

@cardoe cardoe commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Configure post-enrollment WorkflowTemplate hooks independently for enroll-server, enroll-fw, and enroll-netdev using per-workflow ConfigMaps. Invoke each configured run template directly as part of the parent workflow instead of creating child Workflow resources.

Add node UUID output support to the firewall and network-device enrollment commands so every hook receives the same node_uuid input. Document the ConfigMap discovery label and the reusable hook contract, and remove the obsolete lint stub.

Validation:

  • uv run pytest tests/test_enroll_netdev.py tests/test_enroll_fw.py tests/test_netdev_reconciler.py
  • targeted pre-commit hooks, including Argo WorkflowTemplate validation, YAML, Markdown, Ruff, formatting, and Pyright

@cardoe
cardoe force-pushed the enroll-server-post-hooks branch 4 times, most recently from a6c0ffe to 5e9c981 Compare October 28, 2026 17:07
@cardoe
cardoe force-pushed the enroll-server-post-hooks branch from 5e9c981 to 8771cc6 Compare October 28, 2026 17:36
@cardoe cardoe changed the title feat(argo-workflows): configure post-enroll templates via ConfigMap feat(argo-workflows): configure enrollment post hooks via ConfigMap Oct 28, 2026
@github-actions

Copy link
Copy Markdown

Python test results

497 tests   497 ✅  3s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit 5880001.

@cardoe
cardoe added this pull request to the merge queue Oct 28, 2026
Merged via the queue into main with commit e85aba1 Oct 28, 2026
73 checks passed
@cardoe
cardoe deleted the enroll-server-post-hooks branch October 28, 2026 17:55
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