Skip to content

Conversation

@t0mmylam
Copy link
Collaborator

@t0mmylam t0mmylam commented Sep 3, 2025

Agents/steps need runtime context (agent version, package info). Previously node metadata CMs had only labels/annotations.

- Implementation:

  • Add packages.json to each node’s “metadata” ConfigMap:
    • agentVersion (parsed from operator AgentImage)
    • packages: name, version, image, optional agentImageOverride/interrupt/configInterrupts
  • Refresh/backfill via ValidateNodeConfigmaps
  • Add AgentVersion() helper on SkyhookOperatorOptions.
  • Introduce SkyhookMetadata intermediary (decoupled from CRD shape).
  • Marshal the stable metadata instead of writing CR internals directly.

- Tests:

  • Envtest: ensure skyhook namespace exists.
  • Chainsaw: assert packages.json exists with agentVersion, package names/versions.

@t0mmylam t0mmylam merged commit 1985500 into main Sep 4, 2025
9 of 11 checks passed
@t0mmylam t0mmylam deleted the add-package-config-to-config-map branch September 4, 2025 15:58
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