Skip to content

Commit

Permalink
fix: don't require app.kubernetes.io/instance:che label on k8s obje…
Browse files Browse the repository at this point in the history
…ct to trigger a reconcile (#1204)

* fix: don't require 'app.kubernetes.io/instance:che' label

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
  • Loading branch information
tolusha authored Nov 30, 2021
1 parent 1e9fa6a commit 474b0c8
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions controllers/che/cheobj_verifier.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,12 +57,6 @@ func IsTrustedBundleConfigMap(cl client.Client, watchNamespace string, obj clien
// ignore not matched labels
return false, ctrl.Request{}
}

// Check for instance label
if value, exists := obj.GetLabels()[deploy.KubernetesInstanceLabelKey]; !exists || value != deploy.DefaultCheFlavor(checluster) {
// Ignore config map with missing instance label
return false, ctrl.Request{}
}
}

return true, ctrl.Request{
Expand Down Expand Up @@ -100,11 +94,6 @@ func IsEclipseCheRelatedObj(cl client.Client, watchNamespace string, obj client.
return false, ctrl.Request{}
}

// Check for instance label
if value, exists := obj.GetLabels()[deploy.KubernetesInstanceLabelKey]; !exists || value != deploy.DefaultCheFlavor(checluster) {
return false, ctrl.Request{}
}

return true, ctrl.Request{
NamespacedName: types.NamespacedName{
Namespace: checluster.Namespace,
Expand Down

0 comments on commit 474b0c8

Please sign in to comment.