You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/02/09 14:21:40 UTC
[GitHub] [camel-k-runtime] squakez opened a new issue #625: Duplicated loadUserProperties() method call
squakez opened a new issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625
While working on #593 I realized that for some reason we are calling the `loadUserProperties()` [method](https://github.com/apache/camel-k-runtime/blob/master/camel-k-core/support/src/main/java/org/apache/camel/k/support/RuntimeSupport.java#L339) twice, see log trace:
```
[1] 2021-02-09 13:48:23,002 INFO [org.apa.cam.k.sup.RuntimeSupport] (main) Cannot transform /etc/camel/conf.d/_configmaps/testcm/..2021_02_09_13_48_20.491051359/data.zip into UTF-8 text, skipping.
[1] 2021-02-09 13:48:23,003 INFO [org.apa.cam.k.sup.RuntimeSupport] (main) Cannot transform /etc/camel/conf.d/_secrets/testsecret/..2021_02_09_13_48_20.786131203/resources-data.zip into UTF-8 text, skipping.
[1] 2021-02-09 13:48:25,886 INFO [org.apa.cam.k.sup.RuntimeSupport] (main) Cannot transform /etc/camel/conf.d/_configmaps/testcm/..2021_02_09_13_48_20.491051359/data.zip into UTF-8 text, skipping.
[1] 2021-02-09 13:48:25,888 INFO [org.apa.cam.k.sup.RuntimeSupport] (main) Cannot transform /etc/camel/conf.d/_secrets/testsecret/..2021_02_09_13_48_20.786131203/resources-data.zip into UTF-8 text, skipping.
```
We should realize why this is happening and above all check the reason why it takes a few seconds between the 2 calls.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez closed issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez closed issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] lburgazzoli commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-776041057
I'm not sure about what we can do as it depends on quarkus mechanics.
If the class is loaded by the same class-loader we may see if we can impleent s singleton, otherwise we need to ask the quarkus team
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-853859372
I've just tested the behavior of this issue with the operator built from `camel-main` (containing Quarkus 2.x). I can confirm that the issue will be fixed in next release.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez closed issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez closed issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] lburgazzoli commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-847039914
@squakez the issue has been fixed on quarkus, mind checking if it is still an issue when migrationg to quarkus 2.x ?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-853859372
I've just tested the behavior of this issue with the operator built from `camel-main` (containing Quarkus 2.x). I can confirm that the issue will be fixed in next release.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-776031101
According to this [issue](https://github.com/quarkusio/quarkus/issues/8145#issuecomment-604178886), it seems that is the expected behavior to have the call twice (one at static init time, the other one at runtime). I think we should avoid that, as it delay quite a bit the startup phase, so we should find a way to do that once. @lburgazzoli what do you think?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-847115918
Yep, I'll give a look.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k-runtime] squakez commented on issue #625: Duplicated loadUserProperties() method call
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #625:
URL: https://github.com/apache/camel-k-runtime/issues/625#issuecomment-775975363
I am having a look at this.
cc @lburgazzoli
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org