Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion api/v1alpha3/gittarget_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ type GitTargetSpec struct {
// GitTargetPlacementSpec declares where NEW resources are written when no document
// for their identity exists yet in Git — one exact-type map plus a fallback
// default template (Option B2 of
// docs/spec/gittarget-new-file-placement-rules.md). There is
// docs/layout/new-file-placement-rules.md). There is
// deliberately no separate "sensitive" placement block: sensitivity is a
// write-safety classification the controller owns (encrypt the content, keep the
// path identity-complete, never append or co-mingle), not a second placement
Expand Down
42 changes: 35 additions & 7 deletions docs/INDEX.md

Large diffs are not rendered by default.

13 changes: 8 additions & 5 deletions docs/TODO.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,14 @@ This file is meant to track the smaller current backlog, not historical notes.
Secrets and CozyStack `tenantsecrets`; resources with sensitive fields under shapes such as
`spec.credentials` need an explicit field policy or full-file encryption decision.

- [ ] Revisit output layout. **Now designed and postponed, not open-ended.** The answer is to declare
what the folder *is* rather than a path template: `GitTarget.spec.layout`, designed in
[gittarget-layout-model.md](design/gittarget-layout-model.md) and filed as
[#293](https://github.com/ConfigButler/gitops-reverser/issues/293), sequenced with the rest of the
breaking `GitTarget` work as [#294](https://github.com/ConfigButler/gitops-reverser/issues/294).
- [ ] Revisit output layout. **Now designed and postponed, not open-ended, and the answer has
reversed since [#293](https://github.com/ConfigButler/gitops-reverser/issues/293) was filed.** The
path template **stays**; what it could not express becomes two additive members of
`spec.placement`, `serializeNamespace` and `kustomizeRoot`. [layout/model.md](layout/model.md) carries the reversal
and [layout/implementation-plan.md](layout/implementation-plan.md) the order. The placement work
is no longer breaking, so it no longer needs
[#294](https://github.com/ConfigButler/gitops-reverser/issues/294); the issues still describe the
discriminated union and want updating.
Deliberately **not** in 0.41.0, which already carries the new attribution model and the
sibling-inference removal. Multiple resources per file is bundle support, which exists for
match-first today and is a separate question from where a *new* file goes.
Expand Down
2 changes: 1 addition & 1 deletion docs/architecture.md
Original file line number Diff line number Diff line change
Expand Up @@ -1139,7 +1139,7 @@ Placement runs **only for a resource with no existing document** in the target.
manifest identity rather than by path, instead of recomputing placement. So a change to how new files are
placed never moves a file already in Git. A new resource is placed by the first of these that applies
([internal/manifestanalyzer/placement.go](../internal/manifestanalyzer/placement.go),
[design](spec/gittarget-new-file-placement-rules.md)):
[design](layout/new-file-placement-rules.md)):

1. **Declared policy (`spec.placement`).** A `GitTarget` can declare a `byType` map (exact
`[group/]version/resource` → path template) plus a `default` template, rendered from a small
Expand Down
274 changes: 0 additions & 274 deletions docs/design/gittarget-api-wave.md

This file was deleted.

Loading