Remove old TODO

Azure · Jan 5, 2024 · d357654 · d357654
1 parent d53a4fd
commit d357654
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/internal/controllers/cleanup/composition.go b/internal/controllers/cleanup/composition.go
@@ -12,10 +12,6 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
 )
 
-// TODO: Should we limit the max number of resource slices per composition to protect etcd in the face of partial resource slice write failures?
-
-// TODO: Add finalizer on composition to block until all resource slices owned by the composition are fully deleted (reuse builtin?)
-
 type compositionController struct {
 	client client.Client
 }

diff --git a/internal/controllers/cleanup/resourceslice.go b/internal/controllers/cleanup/resourceslice.go
@@ -16,8 +16,6 @@ import (
 
 // TODO: Should we limit the max number of resource slices per composition to protect etcd in the face of partial resource slice write failures?
 
-// TODO: Add finalizer on composition to block until all resource slices owned by the composition are fully deleted (reuse builtin?)
-
 type resourceSliceController struct {
 	client client.Client
 }