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

opendefensecloud / solution-arsenal / 27187611499

09 Jun 2026 06:14AM UTC coverage: 74.285% (+0.4%) from 73.917%
27187611499

push

github

web-flow
feat: check for codegen changes in ci (#571)

## What
<!-- One sentence summary -->
Relates to #557

## Checklist
- [x] ~Tests added/updated~ n/a
- [x] No breaking changes (or upgrade path documented above)
- [x] Readable commit history (squashed and cleaned up as desired)
- [x] AI code review considered and comments resolved

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **New Features**
  * Introduced automated code generation verification in CI/CD pipeline.

* **Chores**
* Enhanced Go version management in CI workflows using dedicated action.
* Extended code generation process to include Helm Chart reference
documentation.
* Updated Helm chart documentation reflecting latest etcd dependency
versions.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

2623 of 3531 relevant lines covered (74.28%)

39.0 hits per line

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

74.3
/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 {
157✔
98
                if apierrors.IsNotFound(err) {
6✔
99
                        return ctrl.Result{}, nil
3✔
100
                }
3✔
101

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

105
        // Handle deletion
106
        if !target.DeletionTimestamp.IsZero() {
152✔
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) {
180✔
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 {
31✔
144
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to add finalizer to Target")
1✔
145
                }
1✔
146

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

150
        // Resolve render registry — supports cross-namespace via ReferenceGrant
151
        registryNamespace := target.Namespace
120✔
152
        if target.Spec.RenderRegistryNamespace != "" {
120✔
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 {
120✔
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{}
120✔
174
        if err := r.Get(ctx, client.ObjectKey{
120✔
175
                Name:      target.Spec.RenderRegistryRef.Name,
120✔
176
                Namespace: registryNamespace,
120✔
177
        }, registry); err != nil {
140✔
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 {
102✔
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",
98✔
200
                "Registry resolved: "+registry.Name); condErr != nil {
98✔
201
                return ctrl.Result{}, condErr
×
202
        }
×
203

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

212
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to build pull secrets lookup from RegistryBindings")
20✔
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{}
78✔
219
        if err := r.List(ctx, bindingList,
78✔
220
                client.InNamespace(target.Namespace),
78✔
221
                client.MatchingFields{
78✔
222
                        indexReleaseBindingTargetName:      target.Name,
78✔
223
                        indexReleaseBindingTargetNamespace: "",
78✔
224
                },
78✔
225
        ); err != nil {
78✔
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)
78✔
231
        if crossNsErr != nil {
78✔
232
                return ctrl.Result{}, errLogAndWrap(log, crossNsErr, "failed to collect cross-namespace ReleaseBindings")
×
233
        }
×
234
        bindingList.Items = append(bindingList.Items, crossNsBindings...)
78✔
235

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

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

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

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

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

67✔
257
        pendingDeps := false
67✔
258

67✔
259
        for _, binding := range bindingList.Items {
153✔
260
                rel := &solarv1alpha1.Release{}
86✔
261
                if err := r.Get(ctx, client.ObjectKey{
86✔
262
                        Name:      binding.Spec.ReleaseRef.Name,
86✔
263
                        Namespace: binding.Namespace,
86✔
264
                }, rel); err != nil {
86✔
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{}
86✔
276
                cvNamespace := rel.Namespace
86✔
277
                if rel.Spec.ComponentVersionNamespace != "" {
86✔
278
                        cvNamespace = rel.Spec.ComponentVersionNamespace
×
279
                }
×
280

281
                if cvNamespace != rel.Namespace {
86✔
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{
86✔
302
                        Name:      rel.Spec.ComponentVersionRef.Name,
86✔
303
                        Namespace: cvNamespace,
86✔
304
                }, cv); err != nil {
86✔
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())
86✔
316
                releases = append(releases, releaseInfo{
86✔
317
                        bindingKey: binding.Namespace + "/" + binding.Name,
86✔
318
                        name:       rel.Name,
86✔
319
                        release:    rel,
86✔
320
                        cv:         cv,
86✔
321
                        rtName:     rtName,
86✔
322
                })
86✔
323
        }
324

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

332
        if len(releases) == 0 && !pendingDeps {
67✔
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
67✔
344

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

77✔
349
                switch {
77✔
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:
53✔
379
                        // RenderTask exists — check for spec drift (e.g. pull secrets
53✔
380
                        // changed after a RegistryBinding was created/updated).
53✔
381
                        desiredSpec, specErr := r.computeReleaseRenderTaskSpec(ri.release, ri.cv, registry, target, pullSecretsByHost)
53✔
382
                        if specErr != nil {
53✔
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) {
55✔
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) {
67✔
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 != "" {
88✔
425
                        releases[i].chartURL = rt.Status.ChartURL
21✔
426
                } else {
67✔
427
                        allRendered = false
46✔
428
                }
46✔
429
        }
430

431
        if pendingDeps {
57✔
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 {
103✔
441
                if condErr := r.setCondition(ctx, target, ConditionTypeReleasesRendered, metav1.ConditionFalse, "Pending",
46✔
442
                        "Waiting for release RenderTasks to complete"); condErr != nil {
50✔
443
                        return ctrl.Result{}, condErr
4✔
444
                }
4✔
445

446
                return ctrl.Result{RequeueAfter: 30 * time.Second}, nil
42✔
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) {
11✔
479
                        bootstrapVersion++
1✔
480
                        needsNewBootstrap = true
1✔
481
                }
1✔
482
        }
483

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

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

2✔
499
                if err := r.Create(ctx, bootstrapRT); err != nil {
2✔
500
                        if !apierrors.IsAlreadyExists(err) {
×
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 {
×
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 {
3✔
515
                        target.Status.BootstrapVersion = bootstrapVersion
1✔
516
                        if err := r.Status().Update(ctx, target); err != nil {
1✔
517
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to update Target bootstrapVersion")
×
518
                        }
×
519
                }
520
        }
521

522
        // Update target status from bootstrap RenderTask
523
        if apimeta.IsStatusConditionTrue(bootstrapRT.Status.Conditions, ConditionTypeJobFailed) {
11✔
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) {
16✔
533
                if condErr := r.setCondition(ctx, target, ConditionTypeBootstrapReady, metav1.ConditionTrue, "Ready",
5✔
534
                        "Bootstrap rendered successfully: "+bootstrapRT.Status.ChartURL); condErr != nil {
5✔
535
                        return ctrl.Result{}, condErr
×
536
                }
×
537

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

547
                return ctrl.Result{}, nil
5✔
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 {
301✔
555
        changed := apimeta.SetStatusCondition(&target.Status.Conditions, metav1.Condition{
301✔
556
                Type:               condType,
301✔
557
                Status:             status,
301✔
558
                ObservedGeneration: target.Generation,
301✔
559
                Reason:             reason,
301✔
560
                Message:            message,
301✔
561
        })
301✔
562
        if changed {
382✔
563
                if err := r.Status().Update(ctx, target); err != nil {
86✔
564
                        return fmt.Errorf("failed to update Target status condition %s: %w", condType, err)
5✔
565
                }
5✔
566
        }
567

568
        return nil
296✔
569
}
570

571
func (r *TargetReconciler) setResolvedCondition(ctx context.Context, target *solarv1alpha1.Target, skipped []string) error {
67✔
572
        if len(skipped) == 0 {
125✔
573
                return r.setCondition(ctx, target, ConditionTypeReleasesResolved, metav1.ConditionTrue, "NoConflicts", "")
58✔
574
        }
58✔
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) {
75✔
584
        if len(releases) == 0 {
76✔
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{}
74✔
591

74✔
592
        for _, ri := range releases {
172✔
593
                uniqueName := ri.release.Spec.UniqueName
98✔
594
                if uniqueName == "" {
104✔
595
                        uniqueName = ri.cv.Spec.ComponentRef.Name
6✔
596
                }
6✔
597

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

601
        var accepted []releaseInfo
74✔
602

74✔
603
        var skipped []string
74✔
604

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

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

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

619
        sort.Strings(uniqueNames)
74✔
620

74✔
621
        for _, uniqueName := range uniqueNames {
163✔
622
                group := namedGroups[uniqueName]
89✔
623
                sort.Slice(group, func(i, j int) bool { return byPriority(group[i], group[j]) })
98✔
624

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

89✔
627
                for _, loser := range group[1:] {
98✔
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]) })
89✔
639

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

74✔
642
        for _, ri := range accepted {
163✔
643
                // Parse ri's own anti-affinity selector once; bail early on invalid selector.
89✔
644
                var riSelector labels.Selector
89✔
645
                if ri.release.Spec.AntiAffinity != nil {
96✔
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 := ""
88✔
662
                for _, other := range resolved {
103✔
663
                        if riSelector != nil && riSelector.Matches(labels.Set(other.release.Labels)) {
19✔
664
                                conflict = other.bindingKey
4✔
665
                                break
4✔
666
                        }
667

668
                        if other.release.Spec.AntiAffinity != nil {
12✔
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 != "" {
93✔
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 {
88✔
683
                        resolved = append(resolved, ri)
83✔
684
                }
83✔
685
        }
686

687
        return resolved, skipped
74✔
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 {
5✔
694
        log := ctrl.LoggerFrom(ctx)
5✔
695

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

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

710
                if _, current := currentRTNames[rt.Name]; current {
27✔
711
                        continue
13✔
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
5✔
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) {
77✔
748
        chartName := fmt.Sprintf("release-%s", rel.Name)
77✔
749
        repo := fmt.Sprintf("%s/%s/%s", target.Namespace, rel.Namespace, chartName)
77✔
750

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

756
        resolvedResources, err := resolveResources(cv.Spec.Resources, pullSecretsByHost, r.RegistryBindingStrict)
77✔
757
        if err != nil {
87✔
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))
67✔
766

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

12✔
802
        for _, ri := range releases {
31✔
803
                ref, err := ociname.ParseReference(ri.chartURL)
19✔
804
                if err != nil {
19✔
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())
19✔
809
                if err != nil {
19✔
810
                        return solarv1alpha1.BootstrapInput{}, err
×
811
                }
×
812

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

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

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

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

837
        sort.Strings(releaseNames)
2✔
838

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

2✔
843
        return solarv1alpha1.RenderTaskSpec{
2✔
844
                RendererConfig: solarv1alpha1.RendererConfig{
2✔
845
                        Type: solarv1alpha1.RendererConfigTypeBootstrap,
2✔
846
                        BootstrapConfig: solarv1alpha1.BootstrapConfig{
2✔
847
                                Chart: solarv1alpha1.ChartConfig{
2✔
848
                                        Name:        chartName,
2✔
849
                                        Description: fmt.Sprintf("Bootstrap of %v", releaseNames),
2✔
850
                                        Version:     tag,
2✔
851
                                        AppVersion:  tag,
2✔
852
                                },
2✔
853
                                Input: input,
2✔
854
                        },
2✔
855
                },
2✔
856
                Repository:     repo,
2✔
857
                Tag:            tag,
2✔
858
                BaseURL:        registry.Spec.Hostname,
2✔
859
                PushSecretRef:  registry.Spec.SolarSecretRef,
2✔
860
                OwnerName:      target.Name,
2✔
861
                OwnerNamespace: target.Namespace,
2✔
862
                OwnerKind:      "Target",
2✔
863
        }, nil
2✔
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 {
28✔
959
                grant := &grantList.Items[i]
1✔
960
                if !grantsRegistryResource(grant) {
2✔
961
                        continue
1✔
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) {
98✔
993
        rbList := &solarv1alpha1.RegistryBindingList{}
98✔
994
        if err := r.List(ctx, rbList,
98✔
995
                client.InNamespace(target.Namespace),
98✔
996
                client.MatchingFields{indexRegistryBindingTargetName: target.Name},
98✔
997
        ); err != nil {
98✔
998
                return nil, err
×
999
        }
×
1000

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

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

98✔
1008
        for _, rb := range rbList.Items {
136✔
1009
                reg := &solarv1alpha1.Registry{}
38✔
1010
                if err := r.Get(ctx, client.ObjectKey{
38✔
1011
                        Name:      rb.Spec.RegistryRef.Name,
38✔
1012
                        Namespace: rb.Namespace,
38✔
1013
                }, reg); err != nil {
48✔
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)
28✔
1019
                if prev, ok := lookup[host]; ok && prev.pullSecret != reg.Spec.TargetPullSecretName {
38✔
1020
                        return nil, fmt.Errorf("conflicting RegistryBindings for host %q: RegistryBinding %s (pull secret %q) vs RegistryBinding %s (pull secret %q)",
10✔
1021
                                host, prev.bindingName, prev.pullSecret, rb.Name, reg.Spec.TargetPullSecretName)
10✔
1022
                }
10✔
1023

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

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

1032
        return result, nil
78✔
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 {
14✔
1150
        for _, t := range grant.Spec.To {
28✔
1151
                if t.Kind == "Registry" && t.Group == solarGroup {
14✔
1152
                        return true
×
1153
                }
×
1154
        }
1155

1156
        return false
14✔
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) {
78✔
1185
        grantList := &solarv1alpha1.ReferenceGrantList{}
78✔
1186
        if err := r.List(ctx, grantList, client.InNamespace(target.Namespace)); err != nil {
78✔
1187
                return nil, err
×
1188
        }
×
1189

1190
        seen := make(map[string]struct{})
78✔
1191
        var result []solarv1alpha1.ReleaseBinding
78✔
1192
        for i := range grantList.Items {
89✔
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
78✔
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 {
93✔
1247
                targetNs := rb.Namespace
3✔
1248
                if rb.Spec.TargetNamespace != "" {
4✔
1249
                        targetNs = rb.Spec.TargetNamespace
1✔
1250
                }
1✔
1251

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

1257
                seen[key] = struct{}{}
3✔
1258
                requests = append(requests, reconcile.Request{
3✔
1259
                        NamespacedName: types.NamespacedName{
3✔
1260
                                Name:      rb.Spec.TargetRef.Name,
3✔
1261
                                Namespace: targetNs,
3✔
1262
                        },
3✔
1263
                })
3✔
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