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 2022/03/17 09:18:25 UTC
[GitHub] [camel-k] aboucham opened a new issue #3127: Log a warn message once the component used has not a quarkus extension
aboucham opened a new issue #3127:
URL: https://github.com/apache/camel-k/issues/3127
Log a warn message once the component used has not a quarkus extension.
For the moment, we see only the following lines in the operator log:
```
{"level":"info","ts":1647441251.3206353,"logger":"camel-k.maven.build","msg":"[INFO] Scanning for projects..."}
{"level":"info","ts":1647441252.2915804,"logger":"camel-k.maven.build","msg":"[ERROR] [ERROR] Some problems were encountered while processing the POMs:"}
{"level":"info","ts":1647441252.2962983,"logger":"camel-k.maven.build","msg":"[ERROR] 'dependencies.dependency.version' for org.apache.camel:camel-aws-starter:jar is missing. @ line 64, column 17"}
{"level":"info","ts":1647441252.3024638,"logger":"camel-k.maven.build","msg":"[ERROR] 'dependencies.dependency.version' for org.apache.camel:camel-http4:jar is missing. @ line 72, column 17"}
{"level":"info","ts":1647441252.3025827,"logger":"camel-k.maven.build","msg":" @ "}
{"level":"info","ts":1647441252.3131278,"logger":"camel-k.maven.build","msg":"[ERROR] The build could not read 1 project -> [Help 1]"}
```
--
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] phantomjinx commented on issue #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
phantomjinx commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1082990473
@squakez
No. I don't believe it is. I think this refers to adding more logging messages whereas #3149 seeks to grab the quarkus logging if the command immediately [fails](https://github.com/apache/camel-k/blob/main/pkg/util/command.go#L41).
--
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 #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1083005684
Thanks @phantomjinx. I was trying to think a simple approach to parse the log, capture the kind of error and bubble up into the Builder error conditions. However, for the sake of simplicity, I think we can start inspecting the content of the [maven log line here](https://github.com/apache/camel-k/blob/main/pkg/util/maven/maven_log.go#L67), and if it complains of a camel dependency missing a version (such as in the issue description) we can add an additional line of log explaining that likely, the component has not a Camel Quarkus extension valid. @parul5sahoo I hope that can give you some hint on how to proceed.
--
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] parul5sahoo commented on issue #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
parul5sahoo commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1084050698
Incase the issue would need a bit of investing, I would like to collaborate on this issue with an experienced collaborator if that's possible..
--
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 #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1082780752
I think some possible way to fix it is to scan the log provided from the builder. However, I'm just noticing a more general scenario could be covered by #3149. @phantomjinx can you please confirm that with the work you're doing that you are covering also the error mentioned in this issue?
--
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] jamesnetherton commented on issue #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1083056942
Can't we verify the validity of any requested camel artifacts by attempting to look them up in the catalog, instead of parsing logs?
--
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] parul5sahoo commented on issue #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
parul5sahoo commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1080308444
Hey @aboucham, I would like to work on this issue, although I am a beginner, this seems like a fairly simpler solution to work around.
--
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 #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1083120353
> Can't we verify the validity of any requested camel artifacts by attempting to look them up in the catalog, instead of parsing logs?
Yeah, it's a very good idea. Probably we can extend the functions in https://github.com/apache/camel-k/blob/main/pkg/util/camel/camel_dependencies.go to lookup into the catalog, otherwise returning an error. If we go through this path, I am not sure if it's still suitable as a "good first issue" for contributors as it requires a bit of knowledge of the project.
--
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 #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1080379992
Just assigned to you. Please, let us know if you need any help.
--
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 #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
squakez commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1084234207
> Incase the issue would need a bit of investing, I would like to collaborate on this issue with an experienced collaborator if that's possible..
Don't worry. Please, look for some other issue that you'd like to work on. I'm unassigning for now.
--
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] parul5sahoo commented on issue #3127: Log a warn message once the component used has not a quarkus extension
Posted by GitBox <gi...@apache.org>.
parul5sahoo commented on issue #3127:
URL: https://github.com/apache/camel-k/issues/3127#issuecomment-1082073307
yes, it would be great if you can share some resources to refer, sort of code walkthroughs or any resource that shall be useful in giving a headstart.
--
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