You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by as...@apache.org on 2019/11/28 07:41:09 UTC

[camel-k] 09/13: chore: Remove conflict checking as patch is now used instead of update

This is an automated email from the ASF dual-hosted git repository.

astefanutti pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-k.git

commit a3c3cf48556e03c54101c0c9b1ecde5a344f1766
Author: Antonin Stefanutti <an...@stefanutti.fr>
AuthorDate: Tue Nov 26 15:37:31 2019 +0100

    chore: Remove conflict checking as patch is now used instead of update
---
 pkg/controller/integration/integration_controller.go | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/pkg/controller/integration/integration_controller.go b/pkg/controller/integration/integration_controller.go
index 3e28398..16849e8 100644
--- a/pkg/controller/integration/integration_controller.go
+++ b/pkg/controller/integration/integration_controller.go
@@ -19,8 +19,6 @@ package integration
 import (
 	"context"
 
-	"github.com/pkg/errors"
-
 	appsv1 "k8s.io/api/apps/v1"
 	k8serrors "k8s.io/apimachinery/pkg/api/errors"
 	"k8s.io/apimachinery/pkg/runtime"
@@ -280,16 +278,6 @@ func (r *ReconcileIntegration) Reconcile(request reconcile.Request) (reconcile.R
 
 			newTarget, err := a.Handle(ctx, target)
 			if err != nil {
-				// Some traits, like the deployment and knative service ones,
-				// update owned resources in the running phase, so it's better
-				// handling update conflicts gracefully, consistently with the
-				// primary integration update requests.
-				if cause := errors.Cause(err); k8serrors.IsConflict(cause) {
-					log.Error(cause, "conflict")
-					return reconcile.Result{
-						Requeue: true,
-					}, nil
-				}
 				return reconcile.Result{}, err
 			}