Skip to content

Commit

Permalink
Fix NodeConfig DaemonSet prunning
Browse files Browse the repository at this point in the history
Wrong function was passed to controllerhelpers.Prune resulting in
sync failing due to failure in deletion.
  • Loading branch information
zimnx committed Jun 13, 2023
1 parent 203d34d commit daa8d3b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/controller/nodeconfig/sync_daemonsets.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (

scyllav1alpha1 "github.com/scylladb/scylla-operator/pkg/api/scylla/v1alpha1"
"github.com/scylladb/scylla-operator/pkg/controllerhelpers"
"github.com/scylladb/scylla-operator/pkg/naming"
"github.com/scylladb/scylla-operator/pkg/resourceapply"
appsv1 "k8s.io/api/apps/v1"
)
Expand All @@ -30,7 +31,7 @@ func (ncc *Controller) syncDaemonSet(
requiredDaemonSets,
daemonSets,
&controllerhelpers.PruneControlFuncs{
DeleteFunc: ncc.scyllaClient.NodeConfigs().Delete,
DeleteFunc: ncc.kubeClient.AppsV1().DaemonSets(naming.ScyllaOperatorNodeTuningNamespace).Delete,
},
ncc.eventRecorder)
if err != nil {
Expand Down

0 comments on commit daa8d3b

Please sign in to comment.