Fix recreating BucketAccess with Update method

kvaps · kvaps · commit 1829a9ebfe13 · 2024-08-01T17:40:10.000+02:00
Signed-off-by: Andrei Kvapil &lt;kvapss@gmail.com&gt;
diff --git a/pkg/imported-sidecar/pkg/bucket/bucket_controller.go b/pkg/imported-sidecar/pkg/bucket/bucket_controller.go
@@ -297,6 +297,12 @@ func (b *BucketListener) Delete(ctx context.Context, inputBucket *v1alpha1.Bucke
 				return err
 			}
 		}
+	} else {
+		// Trigger the Add logic to ensure that the BucketAccess is properly reconciled
+		err := bal.Add(ctx, bucketAccess)
+		if err != nil {
+			return bal.recordError(bucketAccess, v1.EventTypeWarning, events.FailedGrantAccess, err)
+		}
 	}
 
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -297,6 +297,12 @@ func (b BucketListener) Delete(ctx context.Context, inputBucket v1alpha1.Bucke`
`297`	`297`	`return err`
`298`	`298`	`}`
`299`	`299`	`}`
	`300`	`+ } else {`
	`301`	`+ // Trigger the Add logic to ensure that the BucketAccess is properly reconciled`
	`302`	`+ err := bal.Add(ctx, bucketAccess)`
	`303`	`+ if err != nil {`
	`304`	`+ return bal.recordError(bucketAccess, v1.EventTypeWarning, events.FailedGrantAccess, err)`
	`305`	`+ }`
`300`	`306`	`}`
`301`	`307`
`302`	`308`	`return nil`