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/01/22 17:13:14 UTC

[GitHub] [camel-k-runtime] nicolaferraro opened a new issue #592: Catalog contains duplicate entry for same schema (cron)

nicolaferraro opened a new issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592


   See: https://github.com/apache/camel-k/issues/1939#issuecomment-765560895
   
   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

[GitHub] [camel-k-runtime] astefanutti commented on issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
astefanutti commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-767411247


   To the best of my _limited_ knowledge, it seems more logical to me that camel-k-runtime artifacts do **not** declare schemes, so I'd go for 2 as well.


----------------------------------------------------------------
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 #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-767400153


   > > I think we have two options here:
   > > 
   > > 1. since `camel-k-master` and `camel-k-cron` have transitive dependencies on the related camel-quarkus components, we can ignore the camel-quarkus ones when when the catalog is computed (so camel-k-runtime ones have higher priority)
   > > 2. do not add schemes to camel-k-runtime artefact as they are used to provide capabilities
   > > 
   > > @nicolaferraro @astefanutti what do you think ?
   > 
   > I'm not sure they have transitive dependencies, actually iirc the CI was failing when the runtime version of the artifacts was chosen.
   > 
   > Yeah, `camel-k-cron` does not seem to depend on `camel-quarkus-cron`.
   
   That is likely to be a leftover of the migration to quarkus as the runtime, it needs to be fixed in any case


----------------------------------------------------------------
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 #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-766983281


   I think we have two options here:
   
   1. since `camel-k-master` and `camel-k-cron` have transitive dependencies on the related camel-quarkus components, we can ignore the camel-quarkus ones when when the catalog is computed (so camel-k-runtime ones have higher priority) 
   2. do not add schemes to camel-k-runtime artefact as they are used to provide capabilities
   
   @nicolaferraro @astefanutti 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] lburgazzoli commented on issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-765569748


   is that true also for the master component ?


----------------------------------------------------------------
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] nicolaferraro commented on issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
nicolaferraro commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-765574553


   > is that true also for the master component ?
   
   Yes. I think we can add a guard to fail in case of duplicate schemas.
   Also for master the capability contains reference to the runtime part.


----------------------------------------------------------------
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 closed issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
lburgazzoli closed issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592


   


----------------------------------------------------------------
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] nicolaferraro commented on issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
nicolaferraro commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-767390915


   > I think we have two options here:
   > 
   > 1. since `camel-k-master` and `camel-k-cron` have transitive dependencies on the related camel-quarkus components, we can ignore the camel-quarkus ones when when the catalog is computed (so camel-k-runtime ones have higher priority)
   > 2. do not add schemes to camel-k-runtime artefact as they are used to provide capabilities
   > 
   > @nicolaferraro @astefanutti what do you think ?
   
   I'm not sure they have transitive dependencies, actually iirc the CI was failing when the runtime version of the artifacts was chosen.
   
   Yeah, `camel-k-cron` does not seem to depend on `camel-quarkus-cron`.


----------------------------------------------------------------
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 #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-766983281


   I think we have two options here:
   
   1. since `camel-k-master` and `camel-k-cron` have transitive dependencies on the related camel-quarkus components, we can ignore the camel-quarkus ones when when the catalog is computed (so camel-k-runtime ones have higher priority) 
   2. do not add schemes to camel-k-runtime artefact as they are used to provide capabilities
   
   @nicolaferraro @astefanutti 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] nicolaferraro commented on issue #592: Catalog contains duplicate entry for same schema (cron)

Posted by GitBox <gi...@apache.org>.
nicolaferraro commented on issue #592:
URL: https://github.com/apache/camel-k-runtime/issues/592#issuecomment-767391274


   I'd do option 2 so.


----------------------------------------------------------------
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