You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "lfabriko (via GitHub)" <gi...@apache.org> on 2023/04/26 10:29:53 UTC
[GitHub] [camel-k] lfabriko opened a new issue, #4283: Add statical check if integration refers to components supported by Camel K
lfabriko opened a new issue, #4283:
URL: https://github.com/apache/camel-k/issues/4283
In the issue https://github.com/apache/camel-k/issues/4276 was referenced unsupported CEQ (`camel-spring-redis` isn't present in used Camel K runtime 1.17 - https://repo1.maven.org/maven2/io/quarkus/platform/quarkus-camel-bom/2.16.0.Final/quarkus-camel-bom-2.16.0.Final.pom), there was raised question how should user know he is about to use unsupported component (which resulted in later error).
If I may suggest to add a statical check of integration source code and compare it with list of supported components (currently only CEQs)
--
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] rovangju commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "rovangju (via GitHub)" <gi...@apache.org>.
rovangju commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1662692169
Gotcha, my point is that it's installed with the camel-k / kamel cli and/or Helm chart, so when you list available kamelets that get installed, it looks like it's an option that will work with that specific installation.
I'm not sure how to configure a spring-boot compatible runtime for camel-k, I'll do some digging I guess.
--
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] lfabriko closed issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "lfabriko (via GitHub)" <gi...@apache.org>.
lfabriko closed issue #4283: Add statical check if integration refers to components supported by Camel K
URL: https://github.com/apache/camel-k/issues/4283
--
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] gansheer commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1523223419
It could be a good idea to validate any Integration against the CamelCatalog used if we don't already do it. Since we are moving to being able to have multiple runtimes, which means multiple CamelCatalogs.
If there is already have this kind of validation we could still have a better error (if we can) than :
```
Cannot reconcile Integration redis: error during trait customization: component not found for uri "spring-redis:{{redisHost}}:{{redisPort}}?channels=%7B%7Bchannels%7D%7D&command=%7B%7Bcommand%7D%7D&serializer=%23class%3A%7B%7Bserializer%7D%7D" in camel catalog runtime version 1.17.0
```
--
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 #4283: Add statical check if integration refers to components supported by Camel K
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1523233333
It's not that simple. Most of the capabilities are parsed by the operator, so it's there where we do the validation (that's the error you see now). We also need to consider that the evolution is to remove the CLI, so, a user would provide the Integration via kubectl (or any other tooling), so, any kind of validation should be done by the kube controller and not by the operator.
--
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] rovangju commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "rovangju (via GitHub)" <gi...@apache.org>.
rovangju commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1661577813
Yea I just got bit by this as well - it seems there's a quarkus oriented dependency in the catalog, but the OOTB-kamelet doesn't use it?
--
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] lfabriko commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "lfabriko (via GitHub)" <gi...@apache.org>.
lfabriko commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1531237233
Thank you for answer, I have no more suggestions.
--
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] claudio4j commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "claudio4j (via GitHub)" <gi...@apache.org>.
claudio4j commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1662812214
Camel K cannot run camel-spring-boot, currently it is tied to camel-quarkus.
Plain Camel (or camel jbang) can run kameletsbindings
--
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 #4283: Add statical check if integration refers to components supported by Camel K
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1523196744
When you say "static" you mean to validate the Integration against the CamelCatalog ahead of time, is it that?
--
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] lfabriko commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "lfabriko (via GitHub)" <gi...@apache.org>.
lfabriko commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1523254004
I meant CamelCatalog; it seems to me confusing that not all kamelets installed by Camel K operator can be used in integrations (only those filtered by presence of their components in the catalog).
--
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] claudio4j commented on issue #4283: Add statical check if integration refers to components supported by Camel K
Posted by "claudio4j (via GitHub)" <gi...@apache.org>.
claudio4j commented on issue #4283:
URL: https://github.com/apache/camel-k/issues/4283#issuecomment-1662166107
@rovangju which kamelet do you refer ?
--
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