refactor(core): retire the gitops/ overlay (addons in eks-gitops, eval/slo in the chart)#35
Merged
Merged
Conversation
…l/slo in the chart) This repo builds the operator (charts/operator + CRDs) and the per-tenant terraform; it is no longer a deploy catalog. The gitops/ overlay — a second ArgoCD source that duplicated operator/agentgateway/kagent + argo and carried the GPU/eval/SLO addons — is removed now that every piece has its home in the single eks-gitops catalog or the operator chart. Deleted the whole gitops/ tree (ApplicationSets, addons, environments, app-project, the 7 persona dashboards, the overlay README). Destinations: - gpu-operator / nvidia-dra-driver / aws-neuron -> eks-gitops accelerators category. - agentgateway / kagent / operator / argo-* -> already in eks-gitops ai-platform + argo-platform. - eval-runtime + operator-slo -> charts/operator behind evalRuntime.* / slo.* toggles. - the 7 persona dashboards -> eks-gitops dashboards/ (GrafanaDashboard CRs). CI / tooling: ci.yaml drops the gitops dry-run job and renders the operator chart with eval/slo toggles on (exercising the Files.Get-wrapped CRs); Taskfile + package.json drop gitops:validate (the ci composite now runs helm:template, special-casing tenant + operator); commitlint drops the gitops scope; renovate drops the four gitops-keyed customManagers (versions live in eks-gitops now, eks-gitops#34); .trivyignore drops the neuron waivers; .changeset drops gitops/ from the skip-list. Docs: ADR 0007 marked Superseded; new ADR 0008 records the fold into the operator chart and why ADR-0007's Argo-CRD blocker no longer applies. README, AGENTS, ARCHITECTURE, CONTRIBUTING, integrating-with-eks-gitops, the three runbooks, eval-gating-flow, adr-0001, and the terraform docs rewritten to the current state. Closes #33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
PR-6 — the final step of consolidating
eks-agent-platform/gitops/into the single eks-gitops catalog + the operator chart (#33). This repo builds the operator (charts/operator+ CRDs) and the per-tenant terraform; it is no longer a deploy catalog.Deletes the whole
gitops/overlay (52 files). Everything already has a home from the earlier PRs:gitops/evalRuntime.*/slo.*toggles (PR-2/3, merged)CI/tooling: dropped the
gitopsdry-run job (the helm job now renders the operator chart with eval/slo toggles on, exercising the.Files.Get-wrapped CRs); removedgitops:validate(Taskfile + package.json) — thecicomposite runshelm:template; dropped thegitopscommitlint scope; removed the four gitops-keyed renovate customManagers (→ eks-gitops#34); removed the neuron trivy waivers;.changesetskip-list.Docs: ADR-0007 marked Superseded; new ADR-0008 records the chart fold + why ADR-0007's Argo-CRD blocker no longer applies. README, AGENTS, ARCHITECTURE, CONTRIBUTING, integrating-with-eks-gitops, the three runbooks, eval-gating-flow, adr-0001, and the terraform docs rewritten to the current state (atemporal).
Verification
helm lint+helm:templaterender all charts (operator with eval/slo toggles);tofu fmtclean; renovate/package JSON + all YAML valid; commitlint loads.gitops/path references survive except in the ADRs (decision records — intentional).The
gitops/overlay was never wired (no eks-gitops bridge pointed at it), so deleting it removes nothing live — safe to merge. PRs 1-4 (the eks-gitops homes) are already merged tomain, so ArgoCD reconciles them independently. If you want extra confidence before merging, confirm the accelerators + operator(eval/slo) Applications come upSynced/Healthyon a hub cluster.Follow-ups tracked: landing-zone#48 (eval IRSA annotations), eks-gitops#33 (KSM CR-state mount + prod alerting), eks-gitops#34 (version trackers).
Closes #33