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

opendefensecloud / artifact-conduit / 19510623857

19 Nov 2025 05:33PM UTC coverage: 54.645% (-1.4%) from 56.007%
19510623857

push

github

jastBytes
Handle different log text length

7 of 13 new or added lines in 1 file covered. (53.85%)

5 existing lines in 1 file now uncovered.

300 of 549 relevant lines covered (54.64%)

11.14 hits per line

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

82.25
/pkg/controller/order_controller.go
1
// Copyright 2025 BWI GmbH and Artifact Conduit contributors
2
// SPDX-License-Identifier: Apache-2.0
3

4
package controller
5

6
import (
7
        "context"
8
        "crypto/sha256"
9
        "encoding/hex"
10
        "encoding/json"
11
        "fmt"
12
        "slices"
13
        "strconv"
14
        "strings"
15

16
        arcv1alpha1 "go.opendefense.cloud/arc/api/arc/v1alpha1"
17
        corev1 "k8s.io/api/core/v1"
18
        apierrors "k8s.io/apimachinery/pkg/api/errors"
19
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
20
        "k8s.io/apimachinery/pkg/fields"
21
        "k8s.io/apimachinery/pkg/runtime"
22
        "k8s.io/apimachinery/pkg/types"
23
        ctrl "sigs.k8s.io/controller-runtime"
24
        "sigs.k8s.io/controller-runtime/pkg/builder"
25
        "sigs.k8s.io/controller-runtime/pkg/client"
26
        "sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
27
        "sigs.k8s.io/controller-runtime/pkg/handler"
28
        "sigs.k8s.io/controller-runtime/pkg/predicate"
29
        "sigs.k8s.io/controller-runtime/pkg/reconcile"
30
)
31

32
const (
33
        orderFinalizer          = "arc.bwi.de/order-finalizer"
34
        workflowConfigSecretKey = "config.json"
35
)
36

37
// OrderReconciler reconciles a Order object
38
type OrderReconciler struct {
39
        client.Client
40
        Scheme *runtime.Scheme
41
}
42

43
type desiredAW struct {
44
        index       int
45
        objectMeta  metav1.ObjectMeta
46
        artifact    *arcv1alpha1.OrderArtifact
47
        srcEndpoint *arcv1alpha1.Endpoint
48
        dstEndpoint *arcv1alpha1.Endpoint
49
        srcSecret   *corev1.Secret
50
        dstSecret   *corev1.Secret
51
}
52

53
//+kubebuilder:rbac:groups=arc.bwi.de,resources=endpoints,verbs=get;list;watch
54
//+kubebuilder:rbac:groups=arc.bwi.de,resources=artifactworkflows,verbs=get;list;watch;create;update;patch;delete
55
//+kubebuilder:rbac:groups=arc.bwi.de,resources=orders,verbs=get;list;watch;create;update;patch;delete
56
//+kubebuilder:rbac:groups=arc.bwi.de,resources=orders/status,verbs=get;update;patch
57
//+kubebuilder:rbac:groups=arc.bwi.de,resources=orders/finalizers,verbs=update
58
//+kubebuilder:rbac:groups="",resources=secrets,verbs=get;list;watch;create;update;patch;delete
59

60
// Reconcile moves the current state of the cluster closer to the desired state
61
func (r *OrderReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
34✔
62
        log := ctrl.LoggerFrom(ctx)
34✔
63

34✔
64
        // Fetch the Order instance
34✔
65
        order := &arcv1alpha1.Order{}
34✔
66
        if err := r.Get(ctx, req.NamespacedName, order); err != nil {
36✔
67
                if apierrors.IsNotFound(err) {
4✔
68
                        // Object not found, return. Created objects are automatically garbage collected.
2✔
69
                        return ctrl.Result{}, nil
2✔
70
                }
2✔
71
                return ctrl.Result{}, errLogAndWrap(log, err, "failed to get object")
×
72
        }
73

74
        // Handle deletion: cleanup fragments, then remove finalizer
75
        if !order.DeletionTimestamp.IsZero() {
35✔
76
                log.V(1).Info("Order is being deleted")
3✔
77
                if len(order.Status.ArtifactWorkflows) > 0 {
5✔
78
                        for sha := range order.Status.ArtifactWorkflows {
6✔
79
                                // Remove Secret and ArtifactWorkflow
4✔
80
                                aw := &arcv1alpha1.ArtifactWorkflow{
4✔
81
                                        ObjectMeta: awObjectMeta(order, sha),
4✔
82
                                }
4✔
83
                                _ = r.Delete(ctx, aw) // Ignore errors
4✔
84
                                delete(order.Status.ArtifactWorkflows, sha)
4✔
85
                        }
4✔
86
                        if err := r.Status().Update(ctx, order); err != nil {
3✔
87
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to update order status")
1✔
88
                        }
1✔
89
                        log.V(1).Info("Order artifact workflows cleaned up")
1✔
90
                        // Requeue until all fragments are gone
1✔
91
                        return ctrl.Result{Requeue: true}, nil
1✔
92
                }
93
                // All fragments are gone, remove finalizer
94
                if slices.Contains(order.Finalizers, orderFinalizer) {
2✔
95
                        log.V(1).Info("No artifact workflows, removing finalizer from Order")
1✔
96
                        order.Finalizers = slices.DeleteFunc(order.Finalizers, func(f string) bool {
2✔
97
                                return f == orderFinalizer
1✔
98
                        })
1✔
99
                        if err := r.Update(ctx, order); err != nil {
1✔
100
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to remove finalizer")
×
101
                        }
×
102
                }
103
                return ctrl.Result{}, nil
1✔
104
        }
105

106
        // Add finalizer if not present and not deleting
107
        if order.DeletionTimestamp.IsZero() {
58✔
108
                if !slices.Contains(order.Finalizers, orderFinalizer) {
36✔
109
                        log.V(1).Info("Adding finalizer to Order")
7✔
110
                        order.Finalizers = append(order.Finalizers, orderFinalizer)
7✔
111
                        if err := r.Update(ctx, order); err != nil {
7✔
112
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to add finalizer")
×
113
                        }
×
114
                        // Return without requeue; the Update event will trigger reconciliation again
115
                        return ctrl.Result{}, nil
7✔
116
                }
117
        }
118

119
        // Before we compare to our status, let's fetch all necessary information
120
        // to compute desired state:
121
        desiredAWs := map[string]desiredAW{}
22✔
122
        for i, artifact := range order.Spec.Artifacts {
61✔
123
                // TODO: When a endpoint or secret fetch fails, we stop the reconciliation of the whole order.
39✔
124
                //       Should we instead not fail but skip invalid artifacts?
39✔
125
                log := log.WithValues("artifactIndex", i)
39✔
126

39✔
127
                // We need the referenced src- and dst-endpoints for the artifact
39✔
128
                srcRefName := artifact.SrcRef.Name
39✔
129
                if srcRefName == "" {
46✔
130
                        srcRefName = order.Spec.Defaults.SrcRef.Name
7✔
131
                }
7✔
132
                dstRefName := artifact.DstRef.Name
39✔
133
                if dstRefName == "" {
49✔
134
                        dstRefName = order.Spec.Defaults.DstRef.Name
10✔
135
                }
10✔
136
                srcEndpoint := &arcv1alpha1.Endpoint{}
39✔
137
                if err := r.Get(ctx, namespacedName(order.Namespace, srcRefName), srcEndpoint); err != nil {
39✔
138
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to fetch endpoint for source")
×
139
                }
×
140
                dstEndpoint := &arcv1alpha1.Endpoint{}
39✔
141
                if err := r.Get(ctx, namespacedName(order.Namespace, dstRefName), dstEndpoint); err != nil {
39✔
142
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to fetch endpoint for destination")
×
143
                }
×
144

145
                // Next, we need the secret contents
146
                srcSecret := &corev1.Secret{}
39✔
147
                if srcEndpoint.Spec.SecretRef.Name != "" {
75✔
148
                        if err := r.Get(ctx, namespacedName(order.Namespace, srcEndpoint.Spec.SecretRef.Name), srcSecret); err != nil {
36✔
149
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to fetch secret for source")
×
150
                        }
×
151
                }
152

153
                dstSecret := &corev1.Secret{}
39✔
154
                if srcEndpoint.Spec.SecretRef.Name != "" {
75✔
155
                        if err := r.Get(ctx, namespacedName(order.Namespace, dstEndpoint.Spec.SecretRef.Name), dstSecret); err != nil {
36✔
156
                                return ctrl.Result{}, errLogAndWrap(log, err, "failed to fetch secret for destination")
×
157
                        }
×
158
                }
159

160
                // Create a hash based on all related data for idempotency and compute the workflow name
161
                h := sha256.New()
39✔
162
                data := []any{
39✔
163
                        order.Namespace,
39✔
164
                        artifact.Type, artifact.Spec.Raw,
39✔
165
                        srcEndpoint.Name, srcEndpoint.Generation,
39✔
166
                        dstEndpoint.Name, dstEndpoint.Generation,
39✔
167
                        srcSecret.Name, srcSecret.Generation,
39✔
168
                        dstSecret.Name, dstSecret.Generation,
39✔
169
                }
39✔
170
                jsonData, err := json.Marshal(data)
39✔
171
                if err != nil {
39✔
172
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to marshal fragment data")
×
173
                }
×
174
                h.Write(jsonData)
39✔
175
                sha := hex.EncodeToString(h.Sum(nil))[:16]
39✔
176

39✔
177
                // We gave all the information to further process this artifact workflow.
39✔
178
                // Let's store it to compare it to the current status!
39✔
179
                desiredAWs[sha] = desiredAW{
39✔
180
                        index:       i,
39✔
181
                        objectMeta:  awObjectMeta(order, sha),
39✔
182
                        artifact:    &artifact,
39✔
183
                        srcEndpoint: srcEndpoint,
39✔
184
                        dstEndpoint: dstEndpoint,
39✔
185
                        srcSecret:   srcSecret,
39✔
186
                        dstSecret:   dstSecret,
39✔
187
                }
39✔
188
        }
189

190
        // List missing fragments
191
        createAWs := []string{}
22✔
192
        for sha := range desiredAWs {
61✔
193
                _, exists := order.Status.ArtifactWorkflows[sha]
39✔
194
                if exists {
64✔
195
                        continue
25✔
196
                }
197
                createAWs = append(createAWs, sha)
14✔
198
        }
199

200
        // Make sure status is initialized
201
        if order.Status.ArtifactWorkflows == nil {
29✔
202
                order.Status.ArtifactWorkflows = map[string]arcv1alpha1.OrderArtifactWorkflowStatus{}
7✔
203
        }
7✔
204

205
        // Find obsolete fragments
206
        deleteAWs := []string{}
22✔
207
        for sha := range order.Status.ArtifactWorkflows {
48✔
208
                _, exists := desiredAWs[sha]
26✔
209
                if exists {
51✔
210
                        continue
25✔
211
                }
212
                deleteAWs = append(deleteAWs, sha)
1✔
213
        }
214

215
        // Create missing fragments
216
        for _, sha := range createAWs {
36✔
217
                daw := desiredAWs[sha]
14✔
218
                aw, err := r.hydrateArtifactWorkflow(&daw)
14✔
219
                if err != nil {
14✔
220
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to hydrate artifact workflow")
×
221
                }
×
222

223
                // Set owner references
224
                if err := controllerutil.SetControllerReference(order, aw, r.Scheme); err != nil {
14✔
225
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to set controller reference")
×
226
                }
×
227

228
                // Create artifact workflow
229
                if err := r.Create(ctx, aw); err != nil {
14✔
UNCOV
230
                        if apierrors.IsAlreadyExists(err) {
×
UNCOV
231
                                // Already created by a previous reconcile — that's fine
×
UNCOV
232
                                continue
×
233
                        }
234
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to create artifact workflow")
×
235
                }
236

237
                // Update status
238
                order.Status.ArtifactWorkflows[sha] = arcv1alpha1.OrderArtifactWorkflowStatus{
14✔
239
                        ArtifactIndex: daw.index,
14✔
240
                        Phase:         arcv1alpha1.WorkflowUnknown,
14✔
241
                }
14✔
242
        }
243

244
        // Delete obsolete fragments
245
        for _, sha := range deleteAWs {
23✔
246
                // Does not exist anymore, let's clean up!
1✔
247
                if err := r.Delete(ctx, &arcv1alpha1.ArtifactWorkflow{
1✔
248
                        ObjectMeta: awObjectMeta(order, sha),
1✔
249
                }); client.IgnoreNotFound(err) != nil {
1✔
250
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to delete artifact workflow")
×
251
                }
×
252

253
                // Update status
254
                delete(order.Status.ArtifactWorkflows, sha)
1✔
255
        }
256

257
        // Update status
258
        if len(createAWs) > 0 || len(deleteAWs) > 0 {
31✔
259
                log.V(1).Info("Updating Order.Status")
9✔
260
                if err := r.Status().Update(ctx, order); err != nil {
9✔
UNCOV
261
                        return ctrl.Result{}, errLogAndWrap(log, err, "failed to update status")
×
UNCOV
262
                }
×
263
        }
264

265
        return ctrl.Result{}, nil
22✔
266
}
267

268
func (r *OrderReconciler) hydrateArtifactWorkflow(daw *desiredAW) (*arcv1alpha1.ArtifactWorkflow, error) {
14✔
269
        params, err := dawToParameters(daw)
14✔
270
        if err != nil {
14✔
271
                return nil, err
×
272
        }
×
273

274
        // Next we create the ArtifactWorkflow instance
275
        aw := &arcv1alpha1.ArtifactWorkflow{
14✔
276
                ObjectMeta: daw.objectMeta,
14✔
277
                Spec: arcv1alpha1.ArtifactWorkflowSpec{
14✔
278
                        Type:         daw.artifact.Type,
14✔
279
                        Parameters:   params,
14✔
280
                        SrcSecretRef: daw.srcEndpoint.Spec.SecretRef,
14✔
281
                        DstSecretRef: daw.dstEndpoint.Spec.SecretRef,
14✔
282
                },
14✔
283
        }
14✔
284

14✔
285
        return aw, nil
14✔
286
}
287

288
// generateReconcileRequestsForEndpoint generates reconcile requests for all Endpoints referenced by an Order
289
func (r *OrderReconciler) generateReconcileRequestsForEndpoint(ctx context.Context, endpoint client.Object) []reconcile.Request {
23✔
290
        resourcesReferencingEndpoint := &arcv1alpha1.OrderList{}
23✔
291
        listOps := &client.ListOptions{
23✔
292
                FieldSelector: fields.SelectorFromSet(fields.Set{".spec.srcRef.name": endpoint.GetName(), ".spec.dstRef.name": endpoint.GetName()}),
23✔
293
                Namespace:     endpoint.GetNamespace(),
23✔
294
        }
23✔
295
        err := r.List(ctx, resourcesReferencingEndpoint, listOps)
23✔
296
        if err != nil {
46✔
297
                return []reconcile.Request{}
23✔
298
        }
23✔
299

300
        requests := make([]reconcile.Request, len(resourcesReferencingEndpoint.Items))
×
301
        for i, item := range resourcesReferencingEndpoint.Items {
×
302
                log := ctrl.LoggerFrom(ctx)
×
303
                log.V(1).Info("Generating reconcile request for resource because referenced endpoint has changed...")
×
304
                requests[i] = reconcile.Request{
×
305
                        NamespacedName: types.NamespacedName{
×
306
                                Name:      item.GetName(),
×
307
                                Namespace: item.GetNamespace(),
×
308
                        },
×
309
                }
×
310
        }
×
311
        return requests
×
312
}
313

314
// SetupWithManager sets up the controller with the Manager.
315
func (r *OrderReconciler) SetupWithManager(mgr ctrl.Manager) error {
1✔
316
        return ctrl.NewControllerManagedBy(mgr).
1✔
317
                For(&arcv1alpha1.Order{}).
1✔
318
                Watches(
1✔
319
                        &arcv1alpha1.Endpoint{},
1✔
320
                        handler.EnqueueRequestsFromMapFunc(r.generateReconcileRequestsForEndpoint),
1✔
321
                        builder.WithPredicates(predicate.ResourceVersionChangedPredicate{}),
1✔
322
                ).
1✔
323
                Owns(&arcv1alpha1.ArtifactWorkflow{}).
1✔
324
                Complete(r)
1✔
325
}
1✔
326

327
func namespacedName(namespace, name string) types.NamespacedName {
150✔
328
        return types.NamespacedName{
150✔
329
                Namespace: namespace,
150✔
330
                Name:      name,
150✔
331
        }
150✔
332
}
150✔
333

334
func awName(order *arcv1alpha1.Order, sha string) string {
44✔
335
        return fmt.Sprintf("%s-%s", order.Name, sha)
44✔
336
}
44✔
337

338
func awObjectMeta(order *arcv1alpha1.Order, sha string) metav1.ObjectMeta {
44✔
339
        return metav1.ObjectMeta{
44✔
340
                Namespace: order.Namespace,
44✔
341
                Name:      awName(order, sha),
44✔
342
        }
44✔
343
}
44✔
344

345
// TODO: add unit tests
346
func dawToParameters(daw *desiredAW) ([]arcv1alpha1.ArtifactWorkflowParameter, error) {
14✔
347
        params := []arcv1alpha1.ArtifactWorkflowParameter{
14✔
348
                {
14✔
349
                        Name:  paramName("src", "type"),
14✔
350
                        Value: string(daw.srcEndpoint.Spec.Type),
14✔
351
                },
14✔
352
                {
14✔
353
                        Name:  paramName("src", "remoteURL"),
14✔
354
                        Value: daw.srcEndpoint.Spec.RemoteURL,
14✔
355
                },
14✔
356
                {
14✔
357
                        Name:  paramName("dst", "type"),
14✔
358
                        Value: string(daw.dstEndpoint.Spec.Type),
14✔
359
                },
14✔
360
                {
14✔
361
                        Name:  paramName("dst", "remoteURL"),
14✔
362
                        Value: daw.dstEndpoint.Spec.RemoteURL,
14✔
363
                },
14✔
364
                {
14✔
365
                        Name:  "srcSecret",
14✔
366
                        Value: fmt.Sprintf("%v", daw.srcSecret.Name != ""),
14✔
367
                },
14✔
368
                {
14✔
369
                        Name:  "dstSecret",
14✔
370
                        Value: fmt.Sprintf("%v", daw.dstSecret.Name != ""),
14✔
371
                },
14✔
372
        }
14✔
373

14✔
374
        spec := map[string]any{}
14✔
375
        raw := daw.artifact.Spec.Raw
14✔
376
        if len(raw) == 0 {
20✔
377
                raw = []byte("{}")
6✔
378
        }
6✔
379
        if err := json.Unmarshal(raw, &spec); err != nil {
14✔
380
                return nil, err
×
381
        }
×
382
        flattened := map[string]any{}
14✔
383
        flattenMap("spec", spec, flattened)
14✔
384
        for name, value := range flattened {
22✔
385
                params = append(params, arcv1alpha1.ArtifactWorkflowParameter{
8✔
386
                        Name:  name,
8✔
387
                        Value: fmt.Sprintf("%v", value),
8✔
388
                })
8✔
389
        }
8✔
390

391
        return params, nil
14✔
392
}
393

394
// TODO: add unit tests
395
func paramName(prefix, suffix string) string {
56✔
396
        return prefix + strings.ToUpper(suffix[:1]) + suffix[1:]
56✔
397
}
56✔
398

399
// TODO: add unit tests
400
func flattenMap(prefix string, src map[string]any, dst map[string]any) {
14✔
401
        for k, v := range src {
22✔
402
                kt := strings.ToUpper(k[:1]) + k[1:]
8✔
403
                switch child := v.(type) {
8✔
404
                case map[string]any:
×
405
                        flattenMap(prefix+k, child, dst)
×
406
                case []any:
×
407
                        for i, av := range child {
×
408
                                dst[prefix+kt+strconv.Itoa(i)] = av
×
409
                        }
×
410
                default:
8✔
411
                        dst[prefix+kt] = v
8✔
412
                }
413
        }
414
}
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