Skip to content

chore: remove TCOR CRD manifest (Decision G violation)#18

Merged
ontave merged 5 commits into
mainfrom
session/22-cleanup
May 5, 2026
Merged

chore: remove TCOR CRD manifest (Decision G violation)#18
ontave merged 5 commits into
mainfrom
session/22-cleanup

Conversation

@ontave
Copy link
Copy Markdown
Contributor

@ontave ontave commented May 5, 2026

Summary

  • Removes wrapper/config/crd/infrastructure.ontai.dev_infrastructuretalosclusteroperationresults.yaml
  • seam-core is the sole CRD manifest owner under Decision G
  • embed.go already declares an empty FS; no code change required
  • go build ./... passes clean

Test plan

  • go build ./... in wrapper passes
  • No embed.go reference to deleted file

ontave added 5 commits May 2, 2026 21:42
PackExecution reconciler was watching RunnerConfig under the old
runner.ontai.dev/v1alpha1 group, which no longer exists after Phase 2B
migrated all cross-operator CRDs to infrastructure.ontai.dev (Decision G).
The missing CRD caused controller-runtime to fail the EventSource watch
every 10s, blocking the PE informer cache from syncing and preventing
pack-deploy Jobs from being submitted after PE creation.

Updated to InfrastructureRunnerConfig under infrastructure.ontai.dev/v1alpha1.
INV-010, Decision G.
Governor directive (session/21): CODEBASE.md eliminated from all repos.
The graphify knowledge graph at ~/ontai/graphify-out/graph.json is the
sole authoritative source for codebase understanding. See root CONTEXT.md
and CLAUDE.md for the Graphify Source of Truth Protocol.
seam-core is the sole owner of InfrastructureTalosClusterOperationResult
CRD definitions and manifests under infrastructure.ontai.dev. Wrapper
imports the Go type from seam-core's module and must not ship the CRD
YAML independently. Deploying from wrapper would install the same CRD
from two controllers, risking schema divergence.

embed.go already declares an empty FS; no embed reference updated.
@ontave ontave merged commit f48b7d4 into main May 5, 2026
2 checks passed
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.

1 participant