Skip to content

Commit 0956e60

Browse files
committed
fix conditions
1 parent 1be332d commit 0956e60

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

controllers/ibmpowervsimage_controller.go

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ import (
3737
clusterv1beta1 "sigs.k8s.io/cluster-api/api/core/v1beta1" //nolint:staticcheck
3838
clusterv1 "sigs.k8s.io/cluster-api/api/core/v1beta2"
3939
clusterv1util "sigs.k8s.io/cluster-api/util"
40-
"sigs.k8s.io/cluster-api/util/deprecated/v1beta1/conditions"
4140
v1beta1conditions "sigs.k8s.io/cluster-api/util/deprecated/v1beta1/conditions" //nolint:staticcheck
4241
v1beta2conditions "sigs.k8s.io/cluster-api/util/deprecated/v1beta1/conditions/v1beta2"
4342
v1beta1patch "sigs.k8s.io/cluster-api/util/deprecated/v1beta1/patch"
@@ -99,6 +98,10 @@ func (r *IBMPowerVSImageReconciler) Reconcile(ctx context.Context, req ctrl.Requ
9998
scopeParams.Zone = cluster.Spec.Zone
10099
}
101100

101+
// if isPaused, requeue, err := paused.EnsurePausedCondition(ctx, r.Client, cluster, ibmPowerVSImage); err != nil || isPaused || requeue {
102+
// return ctrl.Result{}, err
103+
// }
104+
102105
// Create the scope
103106
imageScope, err := scope.NewPowerVSImageScope(scopeParams)
104107
if err != nil {
@@ -350,15 +353,15 @@ func patchIBMPowerVSImage(ctx context.Context, patchHelper *v1beta1patch.Helper,
350353
}
351354

352355
// always update the readyCondition.
353-
conditions.SetSummary(ibmPowerVSImage,
354-
conditions.WithConditions(
355-
infrav1.ImageReadyCondition,
356+
v1beta1conditions.SetSummary(ibmPowerVSImage,
357+
v1beta1conditions.WithConditions(
358+
infrav1.InstanceReadyCondition,
356359
),
357360
)
358361

359362
if err := v1beta2conditions.SetSummaryCondition(ibmPowerVSImage, ibmPowerVSImage, infrav1.IBMPowerVSImageReadyV1Beta2Condition,
360363
v1beta2conditions.ForConditionTypes{
361-
infrav1.IBMPowerVSImageReadyV1Beta2Condition,
364+
infrav1.IBMPowerVSMachineInstanceReadyV1Beta2Condition,
362365
},
363366
// Using a custom merge strategy to override reasons applied during merge.
364367
v1beta2conditions.CustomMergeStrategy{

0 commit comments

Comments
 (0)