Skip to content

Commit e6d0ea9

Browse files
committed
Remove unneeded unpack stage helper method
Signed-off-by: Brett Tofel <[email protected]>
1 parent e558a63 commit e6d0ea9

File tree

2 files changed

+1
-11
lines changed

2 files changed

+1
-11
lines changed

internal/controllers/clusterextension_controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@ func (r *ClusterExtensionReconciler) reconcile(ctx context.Context, ext *ocv1alp
277277

278278
switch unpackResult.State {
279279
case rukpaksource.StatePending:
280-
setStatusInstallFalseUnpackFailed(ext, unpackResult.Message)
280+
setStatusUnpackFailed(ext, unpackResult.Message)
281281
ensureAllConditionsWithReason(ext, ocv1alpha1.ReasonUnpackFailed, "unpack pending")
282282
return ctrl.Result{}, nil
283283
case rukpaksource.StateUnpacked:

internal/controllers/common_controller.go

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -107,16 +107,6 @@ func setStatusUnpackFailed(ext *ocv1alpha1.ClusterExtension, message string) {
107107
})
108108
}
109109

110-
func setStatusInstallFalseUnpackFailed(ext *ocv1alpha1.ClusterExtension, message string) {
111-
apimeta.SetStatusCondition(&ext.Status.Conditions, metav1.Condition{
112-
Type: ocv1alpha1.TypeUnpacked,
113-
Status: metav1.ConditionFalse,
114-
Reason: ocv1alpha1.ReasonUnpackFailed,
115-
Message: message,
116-
ObservedGeneration: ext.GetGeneration(),
117-
})
118-
}
119-
120110
func setStatusUnpacked(ext *ocv1alpha1.ClusterExtension, message string) {
121111
apimeta.SetStatusCondition(&ext.Status.Conditions, metav1.Condition{
122112
Type: ocv1alpha1.TypeUnpacked,

0 commit comments

Comments
 (0)