• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

opendefensecloud / solution-arsenal / 26951845025

04 Jun 2026 12:31PM UTC coverage: 74.493% (+0.6%) from 73.917%
26951845025

Pull #574

github

web-flow
Merge 2a8b4cbb5 into 6383ec498
Pull Request #574: chore: refactor for backoff v5

25 of 43 new or added lines in 3 files covered. (58.14%)

5 existing lines in 3 files now uncovered.

2646 of 3552 relevant lines covered (74.49%)

38.86 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

73.73
/pkg/controller/target_controller.go
1
// Copyright 2026 BWI GmbH and Solution Arsenal contributors
2
// SPDX-License-Identifier: Apache-2.0
3

4
package controller
5

6
import (
7
        "context"
8
        "errors"
9
        "fmt"
10
        "net/url"
11
        "slices"
12
        "sort"
13
        "strings"
14
        "time"
15

16
        ociname "github.com/google/go-containerregistry/pkg/name"
17
        corev1 "k8s.io/api/core/v1"
18
        apiequality "k8s.io/apimachinery/pkg/api/equality"
19
        apierrors "k8s.io/apimachinery/pkg/api/errors"
20
        apimeta "k8s.io/apimachinery/pkg/api/meta"
21
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
22
        "k8s.io/apimachinery/pkg/labels"
23
        "k8s.io/apimachinery/pkg/runtime"
24
        "k8s.io/apimachinery/pkg/types"
25
        "k8s.io/client-go/tools/events"
26
        ctrl "sigs.k8s.io/controller-runtime"
27
        "sigs.k8s.io/controller-runtime/pkg/builder"
28
        "sigs.k8s.io/controller-runtime/pkg/client"
29
        "sigs.k8s.io/controller-runtime/pkg/handler"
30
        "sigs.k8s.io/controller-runtime/pkg/reconcile"
31

32
        solarv1alpha1 "go.opendefense.cloud/solar/api/solar/v1alpha1"
33
)
34

35
const (
36
        targetFinalizer = "solar.opendefense.cloud/target-finalizer"
37

38
        ConditionTypeRegistryResolved = "RegistryResolved"
39
        ConditionTypeReleasesResolved = "ReleasesResolved"
40
        ConditionTypeReleasesRendered = "ReleasesRendered"
41
        ConditionTypeBootstrapReady   = "BootstrapReady"
42
)
43

44
var ErrReleaseNotRenderedYet = errors.New("release is not rendered yet")
45

46
type releaseInfo struct {
47
        // bindingKey is "<namespace>/<name>" of the originating ReleaseBinding, used as a
48
        // deterministic tiebreaker when two releases share the same priority.
49
        bindingKey string
50
        name       string
51
        release    *solarv1alpha1.Release
52
        cv         *solarv1alpha1.ComponentVersion
53
        rtName     string
54
        chartURL   string
55
}
56

57
type TargetReconciler struct {
58
        client.Client
59
        Scheme   *runtime.Scheme
60
        Recorder events.EventRecorder
61
        // WatchNamespace restricts reconciliation to this namespace.
62
        // Should be empty in production (watches all namespaces).
63
        // Intended for use in integration tests only.
64
        WatchNamespace string
65
        // RegistryBindingStrict enables strict registry binding mode.
66
        // When true, rendering fails if a resource's registry host has no
67
        // matching RegistryBinding. When false (default/relaxed), unmatched
68
        // hosts are treated as anonymous pull (no secretRef rendered).
69
        RegistryBindingStrict bool
70
}
71

72
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=targets,verbs=get;list;watch;create;update;patch;delete
73
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=targets/status,verbs=get;update;patch
74
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=targets/finalizers,verbs=update
75
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=registries,verbs=get;list;watch
76
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=releasebindings,verbs=get;list;watch
77
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=registrybindings,verbs=get;list;watch
78
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=releases,verbs=get;list;watch
79
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=componentversions,verbs=get;list;watch
80
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=referencegrants,verbs=get;list;watch
81
//+kubebuilder:rbac:groups=solar.opendefense.cloud,resources=rendertasks,verbs=get;list;watch;create;update;patch;delete
82
//+kubebuilder:rbac:groups=events.k8s.io,resources=events,verbs=create;patch
83

84
// Reconcile collects ReleaseBindings, resolves the render registry, creates per-release
85
// RenderTasks (with dedup), and creates a per-target bootstrap RenderTask.
86
func (r *TargetReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
234✔
87
        log := ctrl.LoggerFrom(ctx)
234✔
88

234✔
89
        log.V(1).Info("Target is being reconciled", "req", req)
234✔
90

234✔
91
        if r.WatchNamespace != "" && req.Namespace != r.WatchNamespace {
314✔
92
                return ctrl.Result{}, nil
80✔
93
        }
80✔
94

95
        // Fetch target
96
        target := &solarv1alpha1.Target{}
154✔
97
        if err := r.Get(ctx, req.NamespacedName, target); err != nil {
158✔
98
                if apierrors.IsNotFound(err) {
8✔
99
                        return ctrl.Result{}, nil
4✔
100
                }
4✔
101

102
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get object")
×
103
        }
104

105
        // Handle deletion
106
        if !target.DeletionTimestamp.IsZero() {
151✔
107
                log.V(1).Info("Target is being deleted")
1✔
108
                r.Recorder.Eventf(target, nil, corev1.EventTypeWarning, "Deleting", "Reconcile", "Target is being deleted, cleaning up RenderTasks")
1✔
109

1✔
110
                // Delete owned RenderTasks
1✔
111
                if err := r.deleteOwnedRenderTasks(ctx, target); err != nil {
1✔
112
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to delete owned RenderTasks")
×
113
                }
×
114

115
                // Remove finalizer
116
                if slices.Contains(target.Finalizers, targetFinalizer) {
2✔
117
                        latest := &solarv1alpha1.Target{}
1✔
118
                        if err := r.Get(ctx, req.NamespacedName, latest); err != nil {
1✔
119
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get latest Target for finalizer removal")
×
120
                        }
×
121

122
                        original := latest.DeepCopy()
1✔
123
                        latest.Finalizers = slices.DeleteFunc(latest.Finalizers, func(s string) bool {
2✔
124
                                return s == targetFinalizer
1✔
125
                        })
1✔
126
                        if err := r.Patch(ctx, latest, client.MergeFrom(original)); err != nil {
1✔
127
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to remove finalizer from Target")
×
128
                        }
×
129
                }
130

131
                return ctrl.Result{}, nil
1✔
132
        }
133

134
        // Set finalizer if not set
135
        if !slices.Contains(target.Finalizers, targetFinalizer) {
179✔
136
                latest := &solarv1alpha1.Target{}
30✔
137
                if err := r.Get(ctx, req.NamespacedName, latest); err != nil {
30✔
138
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to get latest Target for finalizer addition")
×
139
                }
×
140

141
                original := latest.DeepCopy()
30✔
142
                latest.Finalizers = append(latest.Finalizers, targetFinalizer)
30✔
143
                if err := r.Patch(ctx, latest, client.MergeFrom(original)); err != nil {
30✔
144
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to add finalizer to Target")
×
145
                }
×
146

147
                return ctrl.Result{}, nil
30✔
148
        }
149

150
        // Resolve render registry — supports cross-namespace via ReferenceGrant
151
        registryNamespace := target.Namespace
119✔
152
        if target.Spec.RenderRegistryNamespace != "" {
119✔
153
                registryNamespace = target.Spec.RenderRegistryNamespace
×
154
        }
×
155

156
        // If the registry lives in a different namespace, verify a ReferenceGrant permits it
157
        // before attempting to fetch the object.
158
        if registryNamespace != target.Namespace {
119✔
159
                granted, err := r.registryGranted(ctx, registryNamespace, target.Namespace)
×
160
                if err != nil {
×
161
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to check ReferenceGrant for Registry")
×
162
                }
×
163
                if !granted {
×
164
                        if condErr := r.setCondition(ctx, target, ConditionTypeRegistryResolved, metav1.ConditionFalse, "NotGranted",
×
165
                                "No ReferenceGrant allows access to Registry "+target.Spec.RenderRegistryRef.Name+" in namespace "+registryNamespace); condErr != nil {
×
166
                                return ctrl.Result{}, condErr
×
167
                        }
×
168

169
                        return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
×
170
                }
171
        }
172

173
        registry := &solarv1alpha1.Registry{}
119✔
174
        if err := r.Get(ctx, client.ObjectKey{
119✔
175
                Name:      target.Spec.RenderRegistryRef.Name,
119✔
176
                Namespace: registryNamespace,
119✔
177
        }, registry); err != nil {
139✔
178
                if apierrors.IsNotFound(err) {
40✔
179
                        if condErr := r.setCondition(ctx, target, ConditionTypeRegistryResolved, metav1.ConditionFalse, "NotFound",
20✔
180
                                "Registry not found: "+target.Spec.RenderRegistryRef.Name); condErr != nil {
20✔
181
                                return ctrl.Result{}, condErr
×
182
                        }
×
183

184
                        return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
20✔
185
                }
186

187
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get Registry")
×
188
        }
189

190
        if registry.Spec.SolarSecretRef == nil {
101✔
191
                if condErr := r.setCondition(ctx, target, ConditionTypeRegistryResolved, metav1.ConditionFalse, "MissingSolarSecretRef",
2✔
192
                        "Registry does not have SolarSecretRef set, required for rendering"); condErr != nil {
2✔
193
                        return ctrl.Result{}, condErr
×
194
                }
×
195

196
                return ctrl.Result{}, nil
2✔
197
        }
198

199
        if condErr := r.setCondition(ctx, target, ConditionTypeRegistryResolved, metav1.ConditionTrue, "Resolved",
97✔
200
                "Registry resolved: "+registry.Name); condErr != nil {
97✔
201
                return ctrl.Result{}, condErr
×
202
        }
×
203

204
        // Build hostname→targetPullSecretName lookup from RegistryBindings for this target.
205
        pullSecretsByHost, err := r.buildPullSecretsLookup(ctx, target)
97✔
206
        if err != nil {
118✔
207
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "RegistryBindingConflict",
21✔
208
                        err.Error()); condErr != nil {
21✔
UNCOV
209
                        return ctrl.Result{}, condErr
×
UNCOV
210
                }
×
211

212
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to build pull secrets lookup from RegistryBindings")
21✔
213
        }
214

215
        // Collect ReleaseBindings for this target — same namespace first, then cross-namespace via ReferenceGrants.
216
        // Filter on targetNamespace="" to exclude cross-namespace bindings (targetNamespace set) that share the
217
        // target name but point to a target in a different namespace.
218
        bindingList := &solarv1alpha1.ReleaseBindingList{}
76✔
219
        if err := r.List(ctx, bindingList,
76✔
220
                client.InNamespace(target.Namespace),
76✔
221
                client.MatchingFields{
76✔
222
                        indexReleaseBindingTargetName:      target.Name,
76✔
223
                        indexReleaseBindingTargetNamespace: "",
76✔
224
                },
76✔
225
        ); err != nil {
76✔
226
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to list ReleaseBindings")
×
227
        }
×
228

229
        // Collect cross-namespace ReleaseBindings authorized by ReferenceGrants in target's namespace.
230
        crossNsBindings, crossNsErr := r.collectCrossNamespaceReleaseBindings(ctx, target)
76✔
231
        if crossNsErr != nil {
76✔
232
                return ctrl.Result{}, errLogAndWrap(log, crossNsErr, "failed to collect cross-namespace ReleaseBindings")
×
233
        }
×
234
        bindingList.Items = append(bindingList.Items, crossNsBindings...)
76✔
235

76✔
236
        // FIXME: collect cross-namespace RegistryBindings here once ADR-010 is finalized and
76✔
237
        // RegistryBinding collection is wired into the rendering pipeline.
76✔
238

76✔
239
        if len(bindingList.Items) == 0 {
86✔
240
                log.V(1).Info("No ReleaseBindings found for target")
10✔
241
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "NoReleaseBindings",
10✔
242
                        "No ReleaseBindings found for this target"); condErr != nil {
10✔
243
                        return ctrl.Result{}, condErr
×
244
                }
×
245

246
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesResolved, metav1.ConditionFalse, "NoReleaseBindings",
10✔
247
                        "No ReleaseBindings found for this target"); condErr != nil {
10✔
248
                        return ctrl.Result{}, condErr
×
249
                }
×
250

251
                return ctrl.Result{}, nil
10✔
252
        }
253

254
        // For each bound release, ensure a per-release RenderTask exists
255
        var releases []releaseInfo
66✔
256

66✔
257
        pendingDeps := false
66✔
258

66✔
259
        for _, binding := range bindingList.Items {
149✔
260
                rel := &solarv1alpha1.Release{}
83✔
261
                if err := r.Get(ctx, client.ObjectKey{
83✔
262
                        Name:      binding.Spec.ReleaseRef.Name,
83✔
263
                        Namespace: binding.Namespace,
83✔
264
                }, rel); err != nil {
83✔
265
                        if apierrors.IsNotFound(err) {
×
266
                                log.V(1).Info("Release not found", "release", binding.Spec.ReleaseRef.Name)
×
267
                                pendingDeps = true
×
268

×
269
                                continue
×
270
                        }
271

272
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to get Release")
×
273
                }
274

275
                cv := &solarv1alpha1.ComponentVersion{}
83✔
276
                cvNamespace := rel.Namespace
83✔
277
                if rel.Spec.ComponentVersionNamespace != "" {
83✔
278
                        cvNamespace = rel.Spec.ComponentVersionNamespace
×
279
                }
×
280

281
                if cvNamespace != rel.Namespace {
83✔
282
                        granted := false
×
283
                        grantList := &solarv1alpha1.ReferenceGrantList{}
×
284
                        if err := r.List(ctx, grantList, client.InNamespace(cvNamespace)); err != nil {
×
285
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to check ReferenceGrant for cross-namespace ComponentVersion")
×
286
                        }
×
287
                        for i := range grantList.Items {
×
288
                                if grantPermitsComponentVersionAccess(&grantList.Items[i], rel.Namespace) {
×
289
                                        granted = true
×
290
                                        break
×
291
                                }
292
                        }
293
                        if !granted {
×
294
                                log.V(1).Info("ComponentVersion access not granted", "cv", rel.Spec.ComponentVersionRef.Name, "namespace", cvNamespace)
×
295
                                pendingDeps = true
×
296

×
297
                                continue
×
298
                        }
299
                }
300

301
                if err := r.Get(ctx, client.ObjectKey{
83✔
302
                        Name:      rel.Spec.ComponentVersionRef.Name,
83✔
303
                        Namespace: cvNamespace,
83✔
304
                }, cv); err != nil {
83✔
305
                        if apierrors.IsNotFound(err) {
×
306
                                log.V(1).Info("ComponentVersion not found", "cv", rel.Spec.ComponentVersionRef.Name)
×
307
                                pendingDeps = true
×
308

×
309
                                continue
×
310
                        }
311

312
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to get ComponentVersion")
×
313
                }
314

315
                rtName := releaseRenderTaskName(rel.Namespace, rel.Name, target.Name, rel.GetGeneration())
83✔
316
                releases = append(releases, releaseInfo{
83✔
317
                        bindingKey: binding.Namespace + "/" + binding.Name,
83✔
318
                        name:       rel.Name,
83✔
319
                        release:    rel,
83✔
320
                        cv:         cv,
83✔
321
                        rtName:     rtName,
83✔
322
                })
83✔
323
        }
324

325
        // Resolve conflicts: deduplicate by uniqueName (priority wins) and apply anti-affinity rules.
326
        var skipped []string
66✔
327
        releases, skipped = resolveReleaseConflicts(releases)
66✔
328
        if condErr := r.setResolvedCondition(ctx, target, skipped); condErr != nil {
66✔
329
                return ctrl.Result{}, condErr
×
330
        }
×
331

332
        if len(releases) == 0 && !pendingDeps {
66✔
333
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "AllReleaseBindingsFiltered",
×
334
                        "All ReleaseBindings were filtered out by the release resolver (uniqueName conflicts or anti-affinity rules)"); condErr != nil {
×
335
                        return ctrl.Result{}, condErr
×
336
                }
×
337

338
                return ctrl.Result{}, nil
×
339
        }
340

341
        // Create per-release RenderTasks (one per target+release pair).
342
        // The renderer job handles dedup by skipping if the chart already exists in the registry.
343
        allRendered := true
66✔
344

66✔
345
        for i, ri := range releases {
140✔
346
                rt := &solarv1alpha1.RenderTask{}
74✔
347
                err := r.Get(ctx, client.ObjectKey{Name: ri.rtName, Namespace: target.Namespace}, rt)
74✔
348

74✔
349
                switch {
74✔
350
                case apierrors.IsNotFound(err):
24✔
351
                        spec, specErr := r.computeReleaseRenderTaskSpec(ri.release, ri.cv, registry, target, pullSecretsByHost)
24✔
352
                        if specErr != nil {
34✔
353
                                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "MissingRegistryBinding",
10✔
354
                                        specErr.Error()); condErr != nil {
10✔
355
                                        return ctrl.Result{}, condErr
×
356
                                }
×
357

358
                                return ctrl.Result{}, errLogAndWrap(log, specErr, "failed to compute release RenderTask spec")
10✔
359
                        }
360

361
                        rt = &solarv1alpha1.RenderTask{
14✔
362
                                ObjectMeta: metav1.ObjectMeta{
14✔
363
                                        Name:      ri.rtName,
14✔
364
                                        Namespace: target.Namespace,
14✔
365
                                },
14✔
366
                                Spec: spec,
14✔
367
                        }
14✔
368

14✔
369
                        if err := r.Create(ctx, rt); err != nil {
14✔
370
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to create release RenderTask")
×
371
                        }
×
372

373
                        log.V(1).Info("Created release RenderTask", "release", ri.name, "renderTask", ri.rtName)
14✔
374
                        r.Recorder.Eventf(target, nil, corev1.EventTypeNormal, "Created", "Create",
14✔
375
                                "Created release RenderTask %s for release %s", ri.rtName, ri.name)
14✔
376
                case err != nil:
×
377
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to get release RenderTask")
×
378
                default:
50✔
379
                        // RenderTask exists — check for spec drift (e.g. pull secrets
50✔
380
                        // changed after a RegistryBinding was created/updated).
50✔
381
                        desiredSpec, specErr := r.computeReleaseRenderTaskSpec(ri.release, ri.cv, registry, target, pullSecretsByHost)
50✔
382
                        if specErr != nil {
50✔
383
                                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "MissingRegistryBinding",
×
384
                                        specErr.Error()); condErr != nil {
×
385
                                        return ctrl.Result{}, condErr
×
386
                                }
×
387

388
                                return ctrl.Result{}, errLogAndWrap(log, specErr, "failed to compute release RenderTask spec for comparison")
×
389
                        }
390

391
                        if !apiequality.Semantic.DeepEqual(rt.Spec, desiredSpec) {
52✔
392
                                if err := r.Delete(ctx, rt); err != nil {
2✔
393
                                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to delete stale release RenderTask")
×
394
                                }
×
395

396
                                rt = &solarv1alpha1.RenderTask{
2✔
397
                                        ObjectMeta: metav1.ObjectMeta{
2✔
398
                                                Name:      ri.rtName,
2✔
399
                                                Namespace: target.Namespace,
2✔
400
                                        },
2✔
401
                                        Spec: desiredSpec,
2✔
402
                                }
2✔
403

2✔
404
                                if err := r.Create(ctx, rt); err != nil {
2✔
405
                                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to recreate release RenderTask")
×
406
                                }
×
407

408
                                log.V(1).Info("Recreated release RenderTask (spec drift)", "release", ri.name, "renderTask", ri.rtName)
2✔
409
                                r.Recorder.Eventf(target, nil, corev1.EventTypeNormal, "Updated", "Update",
2✔
410
                                        "Recreated release RenderTask %s for release %s (spec drift)", ri.rtName, ri.name)
2✔
411
                        }
412
                }
413

414
                // Check if release RenderTask is complete
415
                if apimeta.IsStatusConditionTrue(rt.Status.Conditions, ConditionTypeJobFailed) {
64✔
416
                        if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "ReleaseFailed",
×
417
                                fmt.Sprintf("Release %s rendering failed", ri.name)); condErr != nil {
×
418
                                return ctrl.Result{}, condErr
×
419
                        }
×
420

421
                        return ctrl.Result{}, nil
×
422
                }
423

424
                if apimeta.IsStatusConditionTrue(rt.Status.Conditions, ConditionTypeJobSucceeded) && rt.Status.ChartURL != "" {
83✔
425
                        releases[i].chartURL = rt.Status.ChartURL
19✔
426
                } else {
64✔
427
                        allRendered = false
45✔
428
                }
45✔
429
        }
430

431
        if pendingDeps {
56✔
432
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "MissingDependencies",
×
433
                        "One or more bound Releases or ComponentVersions not found"); condErr != nil {
×
434
                        return ctrl.Result{}, condErr
×
435
                }
×
436

437
                return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
×
438
        }
439

440
        if !allRendered {
101✔
441
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "Pending",
45✔
442
                        "Waiting for release RenderTasks to complete"); condErr != nil {
46✔
443
                        return ctrl.Result{}, condErr
1✔
444
                }
1✔
445

446
                return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
44✔
447
        }
448

449
        if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionTrue, "AllRendered",
11✔
450
                "All releases rendered successfully"); condErr != nil {
11✔
451
                return ctrl.Result{}, condErr
×
452
        }
×
453

454
        // Determine if a new bootstrap render is needed by checking whether the
455
        // current bootstrapVersion's RenderTask still matches the desired release set.
456
        bootstrapVersion := target.Status.BootstrapVersion
11✔
457
        bootstrapRTName := targetRenderTaskName(target.Name, bootstrapVersion)
11✔
458
        bootstrapRT := &solarv1alpha1.RenderTask{}
11✔
459
        err = r.Get(ctx, client.ObjectKey{Name: bootstrapRTName, Namespace: target.Namespace}, bootstrapRT)
11✔
460

11✔
461
        needsNewBootstrap := false
11✔
462

11✔
463
        switch {
11✔
464
        case apierrors.IsNotFound(err):
1✔
465
                // No RenderTask for the current version yet — create one
1✔
466
                needsNewBootstrap = true
1✔
467
        case err != nil:
×
468
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get bootstrap RenderTask")
×
469
        default:
10✔
470
                // RenderTask exists — check if the desired bootstrap input changed
10✔
471
                // (release set, resolved refs/tags, or userdata)
10✔
472
                desiredInput, inputErr := buildBootstrapInput(target, releases, registry.Spec.TargetPullSecretName)
10✔
473
                if inputErr != nil {
10✔
474
                        return ctrl.Result{}, errLogAndWrap(log, inputErr, "failed to build desired bootstrap input for comparison")
×
475
                }
×
476

477
                existingInput := bootstrapRT.Spec.RendererConfig.BootstrapConfig.Input
10✔
478
                if !apiequality.Semantic.DeepEqual(desiredInput, existingInput) {
12✔
479
                        bootstrapVersion++
2✔
480
                        needsNewBootstrap = true
2✔
481
                }
2✔
482
        }
483

484
        if needsNewBootstrap {
14✔
485
                spec, specErr := r.computeBootstrapRenderTaskSpec(target, releases, registry, bootstrapVersion)
3✔
486
                if specErr != nil {
3✔
487
                        return ctrl.Result{}, errLogAndWrap(log, specErr, "failed to compute bootstrap RenderTask spec")
×
488
                }
×
489

490
                bootstrapRTName = targetRenderTaskName(target.Name, bootstrapVersion)
3✔
491
                bootstrapRT = &solarv1alpha1.RenderTask{
3✔
492
                        ObjectMeta: metav1.ObjectMeta{
3✔
493
                                Name:      bootstrapRTName,
3✔
494
                                Namespace: target.Namespace,
3✔
495
                        },
3✔
496
                        Spec: spec,
3✔
497
                }
3✔
498

3✔
499
                if err := r.Create(ctx, bootstrapRT); err != nil {
4✔
500
                        if !apierrors.IsAlreadyExists(err) {
1✔
501
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to create bootstrap RenderTask")
×
502
                        }
×
503

504
                        if err := r.Get(ctx, client.ObjectKey{Name: bootstrapRTName, Namespace: target.Namespace}, bootstrapRT); err != nil {
1✔
505
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get existing bootstrap RenderTask")
×
506
                        }
×
507
                } else {
2✔
508
                        log.V(1).Info("Created bootstrap RenderTask", "renderTask", bootstrapRTName, "bootstrapVersion", bootstrapVersion)
2✔
509
                        r.Recorder.Eventf(target, nil, corev1.EventTypeNormal, "Created", "Create",
2✔
510
                                "Created bootstrap RenderTask %s (version %d)", bootstrapRTName, bootstrapVersion)
2✔
511
                }
2✔
512

513
                // Persist the new bootstrapVersion in status
514
                if bootstrapVersion != target.Status.BootstrapVersion {
5✔
515
                        target.Status.BootstrapVersion = bootstrapVersion
2✔
516
                        if err := r.Status().Update(ctx, target); err != nil {
3✔
517
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to update Target bootstrapVersion")
1✔
518
                        }
1✔
519
                }
520
        }
521

522
        // Update target status from bootstrap RenderTask
523
        if apimeta.IsStatusConditionTrue(bootstrapRT.Status.Conditions, ConditionTypeJobFailed) {
10✔
524
                if condErr := r.setCondition(ctx, target, ConditionTypeBootstrapReady, metav1.ConditionFalse, "Failed",
×
525
                        "Bootstrap rendering failed"); condErr != nil {
×
526
                        return ctrl.Result{}, condErr
×
527
                }
×
528

529
                return ctrl.Result{}, nil
×
530
        }
531

532
        if apimeta.IsStatusConditionTrue(bootstrapRT.Status.Conditions, ConditionTypeJobSucceeded) {
14✔
533
                if condErr := r.setCondition(ctx, target, ConditionTypeBootstrapReady, metav1.ConditionTrue, "Ready",
4✔
534
                        "Bootstrap rendered successfully: "+bootstrapRT.Status.ChartURL); condErr != nil {
4✔
535
                        return ctrl.Result{}, condErr
×
536
                }
×
537

538
                // Clean up stale RenderTasks owned by this target (old versions)
539
                currentRTNames := map[string]struct{}{bootstrapRTName: {}}
4✔
540
                for _, ri := range releases {
10✔
541
                        currentRTNames[ri.rtName] = struct{}{}
6✔
542
                }
6✔
543
                if err := r.deleteStaleRenderTasks(ctx, target, currentRTNames); err != nil {
4✔
544
                        log.Error(err, "failed to clean up stale RenderTasks")
×
545
                }
×
546

547
                return ctrl.Result{}, nil
4✔
548
        }
549

550
        // Still running
551
        return ctrl.Result{}, nil
6✔
552
}
553

554
func (r *TargetReconciler) setCondition(ctx context.Context, target *solarv1alpha1.Target, condType string, status metav1.ConditionStatus, reason, message string) error {
296✔
555
        changed := apimeta.SetStatusCondition(&target.Status.Conditions, metav1.Condition{
296✔
556
                Type:               condType,
296✔
557
                Status:             status,
296✔
558
                ObservedGeneration: target.Generation,
296✔
559
                Reason:             reason,
296✔
560
                Message:            message,
296✔
561
        })
296✔
562
        if changed {
379✔
563
                if err := r.Status().Update(ctx, target); err != nil {
84✔
564
                        return fmt.Errorf("failed to update Target status condition %s: %w", condType, err)
1✔
565
                }
1✔
566
        }
567

568
        return nil
295✔
569
}
570

571
func (r *TargetReconciler) setResolvedCondition(ctx context.Context, target *solarv1alpha1.Target, skipped []string) error {
66✔
572
        if len(skipped) == 0 {
123✔
573
                return r.setCondition(ctx, target, ConditionTypeReleasesResolved, metav1.ConditionTrue, "NoConflicts", "")
57✔
574
        }
57✔
575

576
        return r.setCondition(ctx, target, ConditionTypeReleasesResolved, metav1.ConditionTrue, "Resolved", strings.Join(skipped, "; "))
9✔
577
}
578

579
// resolveReleaseConflicts deduplicates releases by uniqueName (keeping the highest-priority
580
// binding) and filters releases that violate anti-affinity rules of already-accepted releases.
581
// Releases without a uniqueName are deduplicated using the parent Component name from the CV.
582
// It returns the accepted releases and a slice of human-readable filter messages.
583
func resolveReleaseConflicts(releases []releaseInfo) ([]releaseInfo, []string) {
74✔
584
        if len(releases) == 0 {
75✔
585
                return releases, nil
1✔
586
        }
1✔
587

588
        // Step A: uniqueName deduplication.
589
        // When UniqueName is empty, fall back to the parent Component name from the CV.
590
        namedGroups := map[string][]releaseInfo{}
73✔
591

73✔
592
        for _, ri := range releases {
168✔
593
                uniqueName := ri.release.Spec.UniqueName
95✔
594
                if uniqueName == "" {
101✔
595
                        uniqueName = ri.cv.Spec.ComponentRef.Name
6✔
596
                }
6✔
597

598
                namedGroups[uniqueName] = append(namedGroups[uniqueName], ri)
95✔
599
        }
600

601
        var accepted []releaseInfo
73✔
602

73✔
603
        var skipped []string
73✔
604

73✔
605
        // byPriority sorts releases with highest priority first; bindingKey breaks ties.
73✔
606
        byPriority := func(a, b releaseInfo) bool {
95✔
607
                if a.release.Spec.Priority != b.release.Spec.Priority {
32✔
608
                        return a.release.Spec.Priority > b.release.Spec.Priority
10✔
609
                }
10✔
610

611
                return a.bindingKey < b.bindingKey
12✔
612
        }
613

614
        uniqueNames := make([]string, 0, len(namedGroups))
73✔
615
        for k := range namedGroups {
159✔
616
                uniqueNames = append(uniqueNames, k)
86✔
617
        }
86✔
618

619
        sort.Strings(uniqueNames)
73✔
620

73✔
621
        for _, uniqueName := range uniqueNames {
159✔
622
                group := namedGroups[uniqueName]
86✔
623
                sort.Slice(group, func(i, j int) bool { return byPriority(group[i], group[j]) })
95✔
624

625
                accepted = append(accepted, group[0])
86✔
626

86✔
627
                for _, loser := range group[1:] {
95✔
628
                        skipped = append(skipped, fmt.Sprintf(
9✔
629
                                "binding %s filtered: uniqueName %q conflict, lower priority than %s",
9✔
630
                                loser.bindingKey, uniqueName, group[0].bindingKey,
9✔
631
                        ))
9✔
632
                }
9✔
633
        }
634

635
        // Step B: anti-affinity evaluation.
636
        // Walk in deterministic order (priority desc, bindingKey asc); accept each release only
637
        // if its AntiAffinity selector does not match any already-accepted release's labels.
638
        sort.Slice(accepted, func(i, j int) bool { return byPriority(accepted[i], accepted[j]) })
86✔
639

640
        resolved := make([]releaseInfo, 0, len(accepted))
73✔
641

73✔
642
        for _, ri := range accepted {
159✔
643
                // Parse ri's own anti-affinity selector once; bail early on invalid selector.
86✔
644
                var riSelector labels.Selector
86✔
645
                if ri.release.Spec.AntiAffinity != nil {
93✔
646
                        sel, err := metav1.LabelSelectorAsSelector(ri.release.Spec.AntiAffinity)
7✔
647
                        if err != nil {
8✔
648
                                skipped = append(skipped, fmt.Sprintf(
1✔
649
                                        "binding %s filtered: invalid antiAffinity selector: %v",
1✔
650
                                        ri.bindingKey, err,
1✔
651
                                ))
1✔
652

1✔
653
                                continue
1✔
654
                        }
655

656
                        riSelector = sel
6✔
657
                }
658

659
                // Check both directions: ri's anti-affinity against already-resolved labels,
660
                // and already-resolved anti-affinities against ri's labels.
661
                conflict := ""
85✔
662
                for _, other := range resolved {
98✔
663
                        if riSelector != nil && riSelector.Matches(labels.Set(other.release.Labels)) {
17✔
664
                                conflict = other.bindingKey
4✔
665
                                break
4✔
666
                        }
667

668
                        if other.release.Spec.AntiAffinity != nil {
10✔
669
                                otherSel, err := metav1.LabelSelectorAsSelector(other.release.Spec.AntiAffinity)
1✔
670
                                if err == nil && otherSel.Matches(labels.Set(ri.release.Labels)) {
2✔
671
                                        conflict = other.bindingKey
1✔
672
                                        break
1✔
673
                                }
674
                        }
675
                }
676

677
                if conflict != "" {
90✔
678
                        skipped = append(skipped, fmt.Sprintf(
5✔
679
                                "binding %s filtered: anti-affinity conflict with %s",
5✔
680
                                ri.bindingKey, conflict,
5✔
681
                        ))
5✔
682
                } else {
85✔
683
                        resolved = append(resolved, ri)
80✔
684
                }
80✔
685
        }
686

687
        return resolved, skipped
73✔
688
}
689

690
// deleteStaleRenderTasks removes RenderTasks owned by this target that are no
691
// longer needed. Any owned RenderTask whose name is not in currentRTNames is
692
// deleted. This covers both old bootstrap versions and old release generations.
693
func (r *TargetReconciler) deleteStaleRenderTasks(ctx context.Context, target *solarv1alpha1.Target, currentRTNames map[string]struct{}) error {
4✔
694
        log := ctrl.LoggerFrom(ctx)
4✔
695

4✔
696
        rtList := &solarv1alpha1.RenderTaskList{}
4✔
697
        if err := r.List(ctx, rtList,
4✔
698
                client.InNamespace(target.Namespace),
4✔
699
                client.MatchingFields{indexOwnerKind: "Target"},
4✔
700
        ); err != nil {
4✔
701
                return err
×
702
        }
×
703

704
        for i := range rtList.Items {
15✔
705
                rt := &rtList.Items[i]
11✔
706
                if rt.Spec.OwnerName != target.Name || rt.Spec.OwnerNamespace != target.Namespace {
11✔
707
                        continue
×
708
                }
709

710
                if _, current := currentRTNames[rt.Name]; current {
21✔
711
                        continue
10✔
712
                }
713

714
                log.V(1).Info("Deleting stale RenderTask", "renderTask", rt.Name)
1✔
715
                if err := r.Delete(ctx, rt, client.PropagationPolicy(metav1.DeletePropagationBackground)); client.IgnoreNotFound(err) != nil {
1✔
716
                        return err
×
717
                }
×
718

719
                r.Recorder.Eventf(target, nil, corev1.EventTypeNormal, "Deleted", "Delete",
1✔
720
                        "Deleted stale RenderTask %s", rt.Name)
1✔
721
        }
722

723
        return nil
4✔
724
}
725

726
func (r *TargetReconciler) deleteOwnedRenderTasks(ctx context.Context, target *solarv1alpha1.Target) error {
1✔
727
        rtList := &solarv1alpha1.RenderTaskList{}
1✔
728
        if err := r.List(ctx, rtList,
1✔
729
                client.InNamespace(target.Namespace),
1✔
730
                client.MatchingFields{indexOwnerKind: "Target"},
1✔
731
        ); err != nil {
1✔
732
                return err
×
733
        }
×
734

735
        for i := range rtList.Items {
1✔
736
                rt := &rtList.Items[i]
×
737
                if rt.Spec.OwnerName == target.Name && rt.Spec.OwnerNamespace == target.Namespace {
×
738
                        if err := r.Delete(ctx, rt, client.PropagationPolicy(metav1.DeletePropagationBackground)); client.IgnoreNotFound(err) != nil {
×
739
                                return err
×
740
                        }
×
741
                }
742
        }
743

744
        return nil
1✔
745
}
746

747
func (r *TargetReconciler) computeReleaseRenderTaskSpec(rel *solarv1alpha1.Release, cv *solarv1alpha1.ComponentVersion, registry *solarv1alpha1.Registry, target *solarv1alpha1.Target, pullSecretsByHost map[string]string) (solarv1alpha1.RenderTaskSpec, error) {
74✔
748
        chartName := fmt.Sprintf("release-%s", rel.Name)
74✔
749
        repo := fmt.Sprintf("%s/%s/%s", target.Namespace, rel.Namespace, chartName)
74✔
750

74✔
751
        var targetNamespace string
74✔
752
        if rel.Spec.TargetNamespace != nil {
148✔
753
                targetNamespace = *rel.Spec.TargetNamespace
74✔
754
        }
74✔
755

756
        resolvedResources, err := resolveResources(cv.Spec.Resources, pullSecretsByHost, r.RegistryBindingStrict)
74✔
757
        if err != nil {
84✔
758
                return solarv1alpha1.RenderTaskSpec{}, fmt.Errorf("release %s: %w", rel.Name, err)
10✔
759
        }
10✔
760

761
        // Include a hash of pull-secret names in the tag so that charts whose
762
        // content differs only in secretRef get unique OCI tags. Without this,
763
        // the renderer's exists-check skips re-pushing after a spec-drift
764
        // recreation (e.g. RegistryBinding created after the first render).
765
        tag := fmt.Sprintf("v0.0.%d-%s", rel.GetGeneration(), pullSecretsTag(resolvedResources))
64✔
766

64✔
767
        return solarv1alpha1.RenderTaskSpec{
64✔
768
                RendererConfig: solarv1alpha1.RendererConfig{
64✔
769
                        Type: solarv1alpha1.RendererConfigTypeRelease,
64✔
770
                        ReleaseConfig: solarv1alpha1.ReleaseConfig{
64✔
771
                                Chart: solarv1alpha1.ChartConfig{
64✔
772
                                        Name:        chartName,
64✔
773
                                        Description: fmt.Sprintf("Release of %s", rel.Spec.ComponentVersionRef.Name),
64✔
774
                                        Version:     tag,
64✔
775
                                        AppVersion:  tag,
64✔
776
                                },
64✔
777
                                Input: solarv1alpha1.ReleaseInput{
64✔
778
                                        Component:  solarv1alpha1.ReleaseComponent{Name: cv.Spec.ComponentRef.Name},
64✔
779
                                        Resources:  resolvedResources,
64✔
780
                                        Entrypoint: cv.Spec.Entrypoint,
64✔
781
                                },
64✔
782
                                Values:          rel.Spec.Values,
64✔
783
                                TargetNamespace: targetNamespace,
64✔
784
                        },
64✔
785
                },
64✔
786
                Repository:     repo,
64✔
787
                Tag:            tag,
64✔
788
                BaseURL:        registry.Spec.Hostname,
64✔
789
                PushSecretRef:  registry.Spec.SolarSecretRef,
64✔
790
                FailedJobTTL:   rel.Spec.FailedJobTTL,
64✔
791
                OwnerName:      target.Name,
64✔
792
                OwnerNamespace: target.Namespace,
64✔
793
                OwnerKind:      "Target",
64✔
794
        }, nil
64✔
795
}
796

797
// buildBootstrapInput constructs the desired BootstrapInput from the current
798
// target and resolved releases. Used for both comparison and spec construction.
799
func buildBootstrapInput(target *solarv1alpha1.Target, releases []releaseInfo, renderRegistryPullSecret string) (solarv1alpha1.BootstrapInput, error) {
13✔
800
        resolvedReleases := map[string]solarv1alpha1.ResolvedResourceAccess{}
13✔
801

13✔
802
        for _, ri := range releases {
33✔
803
                ref, err := ociname.ParseReference(ri.chartURL)
20✔
804
                if err != nil {
20✔
805
                        return solarv1alpha1.BootstrapInput{}, fmt.Errorf("failed to parse chartURL %s: %w", ri.chartURL, err)
×
806
                }
×
807

808
                repo, err := url.JoinPath(ref.Context().RegistryStr(), ref.Context().RepositoryStr())
20✔
809
                if err != nil {
20✔
810
                        return solarv1alpha1.BootstrapInput{}, err
×
811
                }
×
812

813
                resolvedReleases[ri.name] = solarv1alpha1.ResolvedResourceAccess{
20✔
814
                        Repository:     strings.TrimPrefix(repo, "oci://"),
20✔
815
                        Tag:            ref.Identifier(),
20✔
816
                        PullSecretName: renderRegistryPullSecret,
20✔
817
                }
20✔
818
        }
819

820
        return solarv1alpha1.BootstrapInput{
13✔
821
                Releases: resolvedReleases,
13✔
822
                Userdata: target.Spec.Userdata,
13✔
823
        }, nil
13✔
824
}
825

826
func (r *TargetReconciler) computeBootstrapRenderTaskSpec(target *solarv1alpha1.Target, releases []releaseInfo, registry *solarv1alpha1.Registry, bootstrapVersion int64) (solarv1alpha1.RenderTaskSpec, error) {
3✔
827
        input, err := buildBootstrapInput(target, releases, registry.Spec.TargetPullSecretName)
3✔
828
        if err != nil {
3✔
829
                return solarv1alpha1.RenderTaskSpec{}, err
×
830
        }
×
831

832
        releaseNames := make([]string, 0, len(releases))
3✔
833
        for _, ri := range releases {
8✔
834
                releaseNames = append(releaseNames, ri.name)
5✔
835
        }
5✔
836

837
        sort.Strings(releaseNames)
3✔
838

3✔
839
        chartName := fmt.Sprintf("bootstrap-%s", target.Name)
3✔
840
        repo := fmt.Sprintf("%s/%s", target.Namespace, chartName)
3✔
841
        tag := fmt.Sprintf("v0.0.%d", bootstrapVersion)
3✔
842

3✔
843
        return solarv1alpha1.RenderTaskSpec{
3✔
844
                RendererConfig: solarv1alpha1.RendererConfig{
3✔
845
                        Type: solarv1alpha1.RendererConfigTypeBootstrap,
3✔
846
                        BootstrapConfig: solarv1alpha1.BootstrapConfig{
3✔
847
                                Chart: solarv1alpha1.ChartConfig{
3✔
848
                                        Name:        chartName,
3✔
849
                                        Description: fmt.Sprintf("Bootstrap of %v", releaseNames),
3✔
850
                                        Version:     tag,
3✔
851
                                        AppVersion:  tag,
3✔
852
                                },
3✔
853
                                Input: input,
3✔
854
                        },
3✔
855
                },
3✔
856
                Repository:     repo,
3✔
857
                Tag:            tag,
3✔
858
                BaseURL:        registry.Spec.Hostname,
3✔
859
                PushSecretRef:  registry.Spec.SolarSecretRef,
3✔
860
                OwnerName:      target.Name,
3✔
861
                OwnerNamespace: target.Namespace,
3✔
862
                OwnerKind:      "Target",
3✔
863
        }, nil
3✔
864
}
865

866
// SetupWithManager sets up the controller with the Manager.
867
func (r *TargetReconciler) SetupWithManager(mgr ctrl.Manager) error {
1✔
868
        return ctrl.NewControllerManagedBy(mgr).
1✔
869
                For(&solarv1alpha1.Target{}).
1✔
870
                Watches(
1✔
871
                        &solarv1alpha1.ReleaseBinding{},
1✔
872
                        handler.EnqueueRequestsFromMapFunc(r.mapReleaseBindingToTarget),
1✔
873
                ).
1✔
874
                Watches(
1✔
875
                        &solarv1alpha1.RenderTask{},
1✔
876
                        handler.EnqueueRequestsFromMapFunc(mapRenderTaskToOwner("Target")),
1✔
877
                        builder.WithPredicates(renderTaskStatusChangePredicate()),
1✔
878
                ).
1✔
879
                Watches(
1✔
880
                        &solarv1alpha1.Registry{},
1✔
881
                        handler.EnqueueRequestsFromMapFunc(r.mapRegistryToTargets),
1✔
882
                ).
1✔
883
                Watches(
1✔
884
                        &solarv1alpha1.RegistryBinding{},
1✔
885
                        handler.EnqueueRequestsFromMapFunc(r.mapRegistryBindingToTarget),
1✔
886
                ).
1✔
887
                Watches(
1✔
888
                        &solarv1alpha1.ReferenceGrant{},
1✔
889
                        handler.EnqueueRequestsFromMapFunc(r.mapReferenceGrantToTargets),
1✔
890
                ).
1✔
891
                Watches(
1✔
892
                        &solarv1alpha1.Release{},
1✔
893
                        handler.EnqueueRequestsFromMapFunc(r.mapReleaseToTargets),
1✔
894
                ).
1✔
895
                Complete(r)
1✔
896
}
1✔
897

898
// registryGranted checks whether a ReferenceGrant in registryNamespace permits
899
// fromNamespace to reference the named registry.
900
func (r *TargetReconciler) registryGranted(ctx context.Context, registryNamespace, fromNamespace string) (bool, error) {
×
901
        grantList := &solarv1alpha1.ReferenceGrantList{}
×
902
        if err := r.List(ctx, grantList, client.InNamespace(registryNamespace)); err != nil {
×
903
                return false, err
×
904
        }
×
905
        for i := range grantList.Items {
×
906
                grant := &grantList.Items[i]
×
907
                if grantPermitsRegistryAccess(grant, fromNamespace) {
×
908
                        return true, nil
×
909
                }
×
910
        }
911

912
        return false, nil
×
913
}
914

915
// grantPermitsRegistryAccess returns true if the ReferenceGrant allows a Target in
916
// fromNamespace to reference Registry resources in the grant's namespace.
917
func grantPermitsRegistryAccess(grant *solarv1alpha1.ReferenceGrant, fromNamespace string) bool {
×
918
        return grantPermits(grant, solarGroup, "Target", fromNamespace, solarGroup, "Registry")
×
919
}
×
920

921
// mapRegistryToTargets maps a Registry event to reconcile requests for all
922
// Targets that reference it — either in the same namespace or cross-namespace.
923
func (r *TargetReconciler) mapRegistryToTargets(ctx context.Context, obj client.Object) []reconcile.Request {
27✔
924
        reg, ok := obj.(*solarv1alpha1.Registry)
27✔
925
        if !ok {
27✔
926
                return nil
×
927
        }
×
928

929
        // Same-namespace targets
930
        targetList := &solarv1alpha1.TargetList{}
27✔
931
        if err := r.List(ctx, targetList, client.InNamespace(reg.Namespace)); err != nil {
27✔
932
                ctrl.LoggerFrom(ctx).Error(err, "failed to list Targets for Registry", "registry", reg.Name)
×
933

×
934
                return nil
×
935
        }
×
936

937
        var requests []reconcile.Request
27✔
938
        for _, t := range targetList.Items {
29✔
939
                if t.Spec.RenderRegistryRef.Name == reg.Name &&
2✔
940
                        (t.Spec.RenderRegistryNamespace == "" || t.Spec.RenderRegistryNamespace == reg.Namespace) {
3✔
941
                        requests = append(requests, reconcile.Request{
1✔
942
                                NamespacedName: types.NamespacedName{
1✔
943
                                        Name:      t.Name,
1✔
944
                                        Namespace: t.Namespace,
1✔
945
                                },
1✔
946
                        })
1✔
947
                }
1✔
948
        }
949

950
        // Cross-namespace targets: find namespaces that have been granted access to
951
        // registries in reg.Namespace, then check their targets.
952
        grantList := &solarv1alpha1.ReferenceGrantList{}
27✔
953
        if err := r.List(ctx, grantList, client.InNamespace(reg.Namespace)); err != nil {
27✔
954
                ctrl.LoggerFrom(ctx).Error(err, "failed to list ReferenceGrants for cross-namespace Registry mapping")
×
955
                return requests
×
956
        }
×
957

958
        for i := range grantList.Items {
27✔
959
                grant := &grantList.Items[i]
×
960
                if !grantsRegistryResource(grant) {
×
961
                        continue
×
962
                }
963
                for _, from := range grant.Spec.From {
×
964
                        if from.Kind != "Target" || from.Group != solarGroup {
×
965
                                continue
×
966
                        }
967
                        crossTargets := &solarv1alpha1.TargetList{}
×
968
                        if err := r.List(ctx, crossTargets, client.InNamespace(from.Namespace)); err != nil {
×
969
                                ctrl.LoggerFrom(ctx).Error(err, "failed to list cross-namespace Targets", "namespace", from.Namespace)
×
970
                                continue
×
971
                        }
972
                        for _, t := range crossTargets.Items {
×
973
                                if t.Spec.RenderRegistryRef.Name == reg.Name && t.Spec.RenderRegistryNamespace == reg.Namespace {
×
974
                                        requests = append(requests, reconcile.Request{
×
975
                                                NamespacedName: types.NamespacedName{
×
976
                                                        Name:      t.Name,
×
977
                                                        Namespace: t.Namespace,
×
978
                                                },
×
979
                                        })
×
980
                                }
×
981
                        }
982
                }
983
        }
984

985
        return requests
27✔
986
}
987

988
// buildPullSecretsLookup lists RegistryBindings for the given target, resolves
989
// each bound Registry, and returns a map from registry hostname to
990
// targetPullSecretName. Registries without a targetPullSecretName are included
991
// with an empty string (anonymous pull).
992
func (r *TargetReconciler) buildPullSecretsLookup(ctx context.Context, target *solarv1alpha1.Target) (map[string]string, error) {
97✔
993
        rbList := &solarv1alpha1.RegistryBindingList{}
97✔
994
        if err := r.List(ctx, rbList,
97✔
995
                client.InNamespace(target.Namespace),
97✔
996
                client.MatchingFields{indexRegistryBindingTargetName: target.Name},
97✔
997
        ); err != nil {
97✔
998
                return nil, err
×
999
        }
×
1000

1001
        type hostEntry struct {
97✔
1002
                pullSecret  string
97✔
1003
                bindingName string
97✔
1004
        }
97✔
1005

97✔
1006
        lookup := make(map[string]hostEntry, len(rbList.Items))
97✔
1007

97✔
1008
        for _, rb := range rbList.Items {
137✔
1009
                reg := &solarv1alpha1.Registry{}
40✔
1010
                if err := r.Get(ctx, client.ObjectKey{
40✔
1011
                        Name:      rb.Spec.RegistryRef.Name,
40✔
1012
                        Namespace: rb.Namespace,
40✔
1013
                }, reg); err != nil {
50✔
1014
                        return nil, fmt.Errorf("failed to get Registry %s referenced by RegistryBinding %s: %w",
10✔
1015
                                rb.Spec.RegistryRef.Name, rb.Name, err)
10✔
1016
                }
10✔
1017

1018
                host := strings.ToLower(reg.Spec.Hostname)
30✔
1019
                if prev, ok := lookup[host]; ok && prev.pullSecret != reg.Spec.TargetPullSecretName {
41✔
1020
                        return nil, fmt.Errorf("conflicting RegistryBindings for host %q: RegistryBinding %s (pull secret %q) vs RegistryBinding %s (pull secret %q)",
11✔
1021
                                host, prev.bindingName, prev.pullSecret, rb.Name, reg.Spec.TargetPullSecretName)
11✔
1022
                }
11✔
1023

1024
                lookup[host] = hostEntry{pullSecret: reg.Spec.TargetPullSecretName, bindingName: rb.Name}
19✔
1025
        }
1026

1027
        result := make(map[string]string, len(lookup))
76✔
1028
        for host, entry := range lookup {
84✔
1029
                result[host] = entry.pullSecret
8✔
1030
        }
8✔
1031

1032
        return result, nil
76✔
1033
}
1034

1035
// mapRegistryBindingToTarget maps a RegistryBinding event to a reconcile request
1036
// for the referenced Target.
1037
func (r *TargetReconciler) mapRegistryBindingToTarget(ctx context.Context, obj client.Object) []reconcile.Request {
7✔
1038
        rb, ok := obj.(*solarv1alpha1.RegistryBinding)
7✔
1039
        if !ok {
7✔
1040
                return nil
×
1041
        }
×
1042

1043
        if rb.Spec.TargetRef.Name == "" {
7✔
1044
                return nil
×
1045
        }
×
1046

1047
        return []reconcile.Request{
7✔
1048
                {
7✔
1049
                        NamespacedName: types.NamespacedName{
7✔
1050
                                Name:      rb.Spec.TargetRef.Name,
7✔
1051
                                Namespace: rb.Namespace,
7✔
1052
                        },
7✔
1053
                },
7✔
1054
        }
7✔
1055
}
1056

1057
// mapReferenceGrantToTargets enqueues Targets affected by a ReferenceGrant change
1058
// either because the grant controls Registry access (Target → Registry) or because
1059
// it controls ComponentVersion access (Release → ComponentVersion).
1060
func (r *TargetReconciler) mapReferenceGrantToTargets(ctx context.Context, obj client.Object) []reconcile.Request {
13✔
1061
        grant, ok := obj.(*solarv1alpha1.ReferenceGrant)
13✔
1062
        if !ok {
13✔
1063
                return nil
×
1064
        }
×
1065

1066
        var requests []reconcile.Request
13✔
1067

13✔
1068
        if grantsRegistryResource(grant) {
13✔
1069
                for _, from := range grant.Spec.From {
×
1070
                        if from.Kind != "Target" || from.Group != solarGroup {
×
1071
                                continue
×
1072
                        }
1073
                        targets := &solarv1alpha1.TargetList{}
×
1074
                        if err := r.List(ctx, targets, client.InNamespace(from.Namespace)); err != nil {
×
1075
                                ctrl.LoggerFrom(ctx).Error(err, "failed to list Targets for ReferenceGrant mapping", "namespace", from.Namespace)
×
1076
                                continue
×
1077
                        }
1078
                        for _, t := range targets.Items {
×
1079
                                // Enqueue targets that reference a registry specifically in the grant's namespace
×
1080
                                if t.Spec.RenderRegistryNamespace == grant.Namespace {
×
1081
                                        requests = append(requests, reconcile.Request{
×
1082
                                                NamespacedName: types.NamespacedName{
×
1083
                                                        Name:      t.Name,
×
1084
                                                        Namespace: t.Namespace,
×
1085
                                                },
×
1086
                                        })
×
1087
                                }
×
1088
                        }
1089
                }
1090
        }
1091

1092
        if grantsComponentVersionResource(grant) {
18✔
1093
                seen := map[string]struct{}{}
5✔
1094
                for _, from := range grant.Spec.From {
10✔
1095
                        if from.Kind != "Release" || from.Group != solarGroup {
5✔
1096
                                continue
×
1097
                        }
1098
                        bindings := &solarv1alpha1.ReleaseBindingList{}
5✔
1099
                        if err := r.List(ctx, bindings, client.InNamespace(from.Namespace)); err != nil {
5✔
1100
                                ctrl.LoggerFrom(ctx).Error(err, "failed to list ReleaseBindings for ComponentVersion grant mapping", "namespace", from.Namespace)
×
1101
                                continue
×
1102
                        }
1103
                        for _, rb := range bindings.Items {
6✔
1104
                                if rb.Spec.TargetRef.Name == "" {
1✔
1105
                                        continue
×
1106
                                }
1107
                                targetNs := rb.Namespace
1✔
1108
                                if rb.Spec.TargetNamespace != "" {
2✔
1109
                                        targetNs = rb.Spec.TargetNamespace
1✔
1110
                                }
1✔
1111
                                key := targetNs + "/" + rb.Spec.TargetRef.Name
1✔
1112
                                if _, ok := seen[key]; ok {
1✔
1113
                                        continue
×
1114
                                }
1115
                                seen[key] = struct{}{}
1✔
1116
                                requests = append(requests, reconcile.Request{
1✔
1117
                                        NamespacedName: types.NamespacedName{
1✔
1118
                                                Name:      rb.Spec.TargetRef.Name,
1✔
1119
                                                Namespace: targetNs,
1✔
1120
                                        },
1✔
1121
                                })
1✔
1122
                        }
1123
                }
1124
        }
1125

1126
        if grantsReleaseBindingToTargetResource(grant) {
17✔
1127
                // The grant lives in the Target's namespace and authorizes ReleaseBindings from
4✔
1128
                // other namespaces. Enqueue all Targets in the grant's namespace so they pick up
4✔
1129
                // the new or removed cross-namespace ReleaseBindings.
4✔
1130
                targets := &solarv1alpha1.TargetList{}
4✔
1131
                if err := r.List(ctx, targets, client.InNamespace(grant.Namespace)); err != nil {
4✔
1132
                        ctrl.LoggerFrom(ctx).Error(err, "failed to list Targets for ReleaseBinding grant mapping", "namespace", grant.Namespace)
×
1133
                } else {
4✔
1134
                        for _, t := range targets.Items {
8✔
1135
                                requests = append(requests, reconcile.Request{
4✔
1136
                                        NamespacedName: types.NamespacedName{
4✔
1137
                                                Name:      t.Name,
4✔
1138
                                                Namespace: t.Namespace,
4✔
1139
                                        },
4✔
1140
                                })
4✔
1141
                        }
4✔
1142
                }
1143
        }
1144

1145
        return requests
13✔
1146
}
1147

1148
// grantsRegistryResource returns true if the ReferenceGrant includes Registry in its To list.
1149
func grantsRegistryResource(grant *solarv1alpha1.ReferenceGrant) bool {
13✔
1150
        for _, t := range grant.Spec.To {
26✔
1151
                if t.Kind == "Registry" && t.Group == solarGroup {
13✔
1152
                        return true
×
1153
                }
×
1154
        }
1155

1156
        return false
13✔
1157
}
1158

1159
// grantsReleaseBindingToTargetResource returns true if the ReferenceGrant authorizes
1160
// ReleaseBindings in another namespace to reference Targets in the grant's namespace.
1161
func grantsReleaseBindingToTargetResource(grant *solarv1alpha1.ReferenceGrant) bool {
24✔
1162
        hasReleaseBindingFrom := false
24✔
1163
        for _, f := range grant.Spec.From {
48✔
1164
                if f.Kind == "ReleaseBinding" && f.Group == solarGroup {
39✔
1165
                        hasReleaseBindingFrom = true
15✔
1166
                        break
15✔
1167
                }
1168
        }
1169
        if !hasReleaseBindingFrom {
33✔
1170
                return false
9✔
1171
        }
9✔
1172
        for _, t := range grant.Spec.To {
30✔
1173
                if t.Kind == "Target" && t.Group == solarGroup {
30✔
1174
                        return true
15✔
1175
                }
15✔
1176
        }
1177

1178
        return false
×
1179
}
1180

1181
// collectCrossNamespaceReleaseBindings returns ReleaseBindings from other namespaces
1182
// that reference target via spec.targetRef.name + spec.targetNamespace, authorized by
1183
// a ReferenceGrant in target's namespace.
1184
func (r *TargetReconciler) collectCrossNamespaceReleaseBindings(ctx context.Context, target *solarv1alpha1.Target) ([]solarv1alpha1.ReleaseBinding, error) {
76✔
1185
        grantList := &solarv1alpha1.ReferenceGrantList{}
76✔
1186
        if err := r.List(ctx, grantList, client.InNamespace(target.Namespace)); err != nil {
76✔
1187
                return nil, err
×
1188
        }
×
1189

1190
        seen := make(map[string]struct{})
76✔
1191
        var result []solarv1alpha1.ReleaseBinding
76✔
1192
        for i := range grantList.Items {
87✔
1193
                grant := &grantList.Items[i]
11✔
1194
                if !grantsReleaseBindingToTargetResource(grant) {
11✔
1195
                        continue
×
1196
                }
1197
                for _, from := range grant.Spec.From {
22✔
1198
                        if from.Kind != "ReleaseBinding" || from.Group != solarGroup {
11✔
1199
                                continue
×
1200
                        }
1201
                        crossBindings := &solarv1alpha1.ReleaseBindingList{}
11✔
1202
                        if err := r.List(ctx, crossBindings,
11✔
1203
                                client.InNamespace(from.Namespace),
11✔
1204
                                client.MatchingFields{indexReleaseBindingTargetName: target.Name},
11✔
1205
                        ); err != nil {
11✔
1206
                                return nil, err
×
1207
                        }
×
1208
                        for _, rb := range crossBindings.Items {
19✔
1209
                                if rb.Spec.TargetNamespace != target.Namespace {
8✔
1210
                                        continue
×
1211
                                }
1212
                                key := rb.Namespace + "/" + rb.Name
8✔
1213
                                if _, exists := seen[key]; exists {
9✔
1214
                                        continue
1✔
1215
                                }
1216
                                seen[key] = struct{}{}
7✔
1217
                                result = append(result, rb)
7✔
1218
                        }
1219
                }
1220
        }
1221

1222
        return result, nil
76✔
1223
}
1224

1225
// mapReleaseToTargets maps a Release event to reconcile requests for all
1226
// Targets that are bound to the release via ReleaseBindings.
1227
func (r *TargetReconciler) mapReleaseToTargets(ctx context.Context, obj client.Object) []reconcile.Request {
90✔
1228
        rel, ok := obj.(*solarv1alpha1.Release)
90✔
1229
        if !ok {
90✔
1230
                return nil
×
1231
        }
×
1232

1233
        bindingList := &solarv1alpha1.ReleaseBindingList{}
90✔
1234
        if err := r.List(ctx, bindingList,
90✔
1235
                client.InNamespace(rel.Namespace),
90✔
1236
                client.MatchingFields{indexReleaseBindingReleaseName: rel.Name},
90✔
1237
        ); err != nil {
90✔
1238
                ctrl.LoggerFrom(ctx).Error(err, "failed to list ReleaseBindings for Release", "release", rel.Name)
×
1239

×
1240
                return nil
×
1241
        }
×
1242

1243
        seen := map[string]struct{}{}
90✔
1244
        var requests []reconcile.Request
90✔
1245

90✔
1246
        for _, rb := range bindingList.Items {
92✔
1247
                targetNs := rb.Namespace
2✔
1248
                if rb.Spec.TargetNamespace != "" {
2✔
1249
                        targetNs = rb.Spec.TargetNamespace
×
1250
                }
×
1251

1252
                key := targetNs + "/" + rb.Spec.TargetRef.Name
2✔
1253
                if _, ok := seen[key]; ok {
2✔
1254
                        continue
×
1255
                }
1256

1257
                seen[key] = struct{}{}
2✔
1258
                requests = append(requests, reconcile.Request{
2✔
1259
                        NamespacedName: types.NamespacedName{
2✔
1260
                                Name:      rb.Spec.TargetRef.Name,
2✔
1261
                                Namespace: targetNs,
2✔
1262
                        },
2✔
1263
                })
2✔
1264
        }
1265

1266
        return requests
90✔
1267
}
1268

1269
func (r *TargetReconciler) mapReleaseBindingToTarget(_ context.Context, obj client.Object) []reconcile.Request {
33✔
1270
        rb, ok := obj.(*solarv1alpha1.ReleaseBinding)
33✔
1271
        if !ok || rb.Spec.TargetRef.Name == "" {
33✔
1272
                return nil
×
1273
        }
×
1274

1275
        targetNs := rb.Namespace
33✔
1276
        if rb.Spec.TargetNamespace != "" {
42✔
1277
                targetNs = rb.Spec.TargetNamespace
9✔
1278
        }
9✔
1279

1280
        return []reconcile.Request{
33✔
1281
                {
33✔
1282
                        NamespacedName: types.NamespacedName{
33✔
1283
                                Name:      rb.Spec.TargetRef.Name,
33✔
1284
                                Namespace: targetNs,
33✔
1285
                        },
33✔
1286
                },
33✔
1287
        }
33✔
1288
}
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc