camel-k ignores changes to traits configured using annotations

[lburgazzoli]

Assuming that you have a binding like the one below were annotations are used to configure traits:

apiVersion: camel.apache.org/v1alpha1
kind: KameletBinding
metadata:
  name: timer2log
  annotations:
    trait.camel.apache.org/container.image: acme/bad:1
    trait.camel.apache.org/jvm.enabled: "false"
    trait.camel.apache.org/kamelets.enabled: "false"
spec:
  integration:
    configuration:
      - type: configmap
        value: timer2log-config
  source:
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: timer-source
    properties:
      message: "foo"
  sink:
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: log-sink

In this specific case as example, the container image that must be used to run the Integration is set set using the trait.camel.apache.org/container.image: acme/bad:1 annotation.
If you update the resource and you change only the container image, then the reconcile loop is not triggered because the kamelet binding controller take into account only the phase or the generation (which does not change when only the matadata is amended ).

Since annotation can be used to configure traits,, they should be taken into account.

[squakez]

Duplicate of #2648

[tadayosi]

Btw, while I'm not sure if it's directly related to the issue, I find another bug with kameletbindings with traits, where the traits defined on a kameletbinding is always wiped out on an integration. It's being fixed with this pull req: #3478

Specifically, this is what I try to fix there: 127460a