You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "squakez (via GitHub)" <gi...@apache.org> on 2023/06/23 14:23:29 UTC

[GitHub] [camel-k] squakez opened a new issue, #4511: Updating a property will cause Integration to rebuild

squakez opened a new issue, #4511:
URL: https://github.com/apache/camel-k/issues/4511

   Running an application and once this is running updating it with a property, it is triggering a rebuild, when in reality this should not happen. I've run some investigation and it seems that this behavior started from Camel K version 1.10.3. We need to find out and fix this to avoid the rebuild on property replacement.
   
   ```
   $ ./kamel run /tmp/test.yaml 
   Integration "test" created
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get it -w
   NAME   PHASE          KIT                        REPLICAS
   test   Building Kit   kit-ciaqiid0s9ls7380vo60   
   test   Deploying      kit-ciaqiid0s9ls7380vo60   
   test   Running        kit-ciaqiid0s9ls7380vo60   0
   test   Running        kit-ciaqiid0s9ls7380vo60   1
   test   Running        kit-ciaqiid0s9ls7380vo60   1
   ^Csquake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ ./kamel run /tmp/test.yaml -p a=1
   Integration "test" updated
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get it
   NAME   PHASE          KIT                        REPLICAS
   test   Building Kit   kit-ciaqj6l0s9ls7380vo6g   
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get it
   NAME   PHASE          KIT                        REPLICAS
   test   Building Kit   kit-ciaqj6l0s9ls7380vo6g   
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get ik
   NAME                       PHASE   TYPE       IMAGE
   kit-ciaqiid0s9ls7380vo60   Ready   platform   10.110.149.72/default/camel-k-kit-ciaqiid0s9ls7380vo60@sha256:2cb38b1f07f067fa149ebead3dd55cc62d297d7e135de637b95e429aaee2bee7
   kit-ciaqj6l0s9ls7380vo6g   Ready   platform   10.110.149.72/default/camel-k-kit-ciaqj6l0s9ls7380vo6g@sha256:60f906c120e9e08e1c360c341913552e5d377351f5d357cb8518d558433e7bba
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get builds
   NAME                       PHASE       AGE    STARTED   DURATION        ATTEMPTS
   kit-ciaqiid0s9ls7380vo60   Succeeded   105s   105s      47.866880309s   
   kit-ciaqj6l0s9ls7380vo6g   Succeeded   24s    24s       8.482967548s    
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ ./kamel run /tmp/test.yaml -p a=3
   Integration "test" updated
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get it
   NAME   PHASE          KIT                        REPLICAS
   test   Building Kit   kit-ciaqlf50s9ls7380vo70   
   squake:~/Downloads/camel-k-client-1.10.3-linux-64bit $ k get builds
   NAME                       PHASE       AGE     STARTED   DURATION        ATTEMPTS
   kit-ciaqiid0s9ls7380vo60   Succeeded   6m17s   6m17s     47.866880309s   
   kit-ciaqj6l0s9ls7380vo6g   Succeeded   4m56s   4m56s     8.482967548s    
   kit-ciaqlf50s9ls7380vo70   Running     6s      6s 
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-k] squakez closed issue #4511: Updating a property will cause Integration to rebuild

Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez closed issue #4511: Updating a property will cause Integration to rebuild
URL: https://github.com/apache/camel-k/issues/4511


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-k] squakez commented on issue #4511: Updating a property will cause Integration to rebuild

Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on issue #4511:
URL: https://github.com/apache/camel-k/issues/4511#issuecomment-1604365937

   The problem is introduced by https://github.com/apache/camel-k/issues/3758 - we must understand if it makes sense to rebuild when a camel property change or not.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org