Skip to content

Commit 12c9066

Browse files
committed
fix: transmit context on delete resource
1 parent 0aa66ed commit 12c9066

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

controllers/account/controllers/namespace_controller.go

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -380,8 +380,8 @@ func (r *NamespaceReconciler) SuspendUserResource(ctx context.Context, namespace
380380
return errors2.Join(errs...)
381381
}
382382

383-
func (r *NamespaceReconciler) DeleteUserResource(_ context.Context, namespace string) error {
384-
err := deleteResource(r.dynamicClient, "backup", namespace)
383+
func (r *NamespaceReconciler) DeleteUserResource(ctx context.Context, namespace string) error {
384+
err := deleteResource(ctx, r.dynamicClient, "backup", namespace)
385385
if err != nil {
386386
return err
387387
}
@@ -394,7 +394,7 @@ func (r *NamespaceReconciler) DeleteUserResource(_ context.Context, namespace st
394394
errChan := make(chan error, len(deleteResources))
395395
for _, rs := range deleteResources {
396396
go func(resource string) {
397-
errChan <- deleteResource(r.dynamicClient, resource, namespace)
397+
errChan <- deleteResource(ctx, r.dynamicClient, resource, namespace)
398398
}(rs)
399399
}
400400
for range deleteResources {
@@ -2225,8 +2225,11 @@ func (r *NamespaceReconciler) resumeCertificates(ctx context.Context, namespace
22252225
return nil
22262226
}
22272227

2228-
func deleteResource(dynamicClient dynamic.Interface, resource, namespace string) error {
2229-
ctx := context.Background()
2228+
func deleteResource(
2229+
ctx context.Context,
2230+
dynamicClient dynamic.Interface,
2231+
resource, namespace string,
2232+
) error {
22302233
deletePolicy := v12.DeletePropagationForeground
22312234
var gvr schema.GroupVersionResource
22322235
switch resource {

0 commit comments

Comments
 (0)