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/08/25 06:28:54 UTC

[GitHub] [camel-quarkus] jamesnetherton opened a new pull request, #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

jamesnetherton opened a new pull request, #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045

   Otherwise it seems to make all of the following Debezium tests fail. It's safe to do this as there's a check in `DebeziumMongodbTest` to see if there's a MongoDB connection available, and if not, fails the tests.


-- 
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-quarkus] jamesnetherton merged pull request #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

Posted by GitBox <gi...@apache.org>.
jamesnetherton merged PR #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045


-- 
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-quarkus] jamesnetherton commented on pull request #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on PR #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045#issuecomment-1227003840

   > Hmmm - looks likes there's a different failure.
   
   It was because there was no tag specified for the MongoDB container image.
   
   


-- 
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-quarkus] jamesnetherton commented on pull request #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on PR #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045#issuecomment-1226889503

   > And showing a warning instead of throwing a RuntimeException would make it possible to run some of latter tests ?
   
   By 'latter tests` I mean the ones for the other Debezium components. MySQL, Postgres, SQL Server etc. Having `RuntimeException` thrown seems to prevent that. We only want the Mongo tests to fail.


-- 
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-quarkus] aldettinger commented on pull request #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

Posted by GitBox <gi...@apache.org>.
aldettinger commented on PR #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045#issuecomment-1226884499

   So the `DebeziumMongodbTestResource` is initialized only once in `DebeziumMongodbTest` ? 
   
   Tests are executed in below order ?
    + testReceiveInit
    + testInsert
    + testUpdate
    + testDelete
   
   And showing a warning instead of throwing a RuntimeException would make it possible to run some of latter tests ?


-- 
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-quarkus] jamesnetherton commented on pull request #4045: Log warning instead of throw exception for Debezium MongoDB container script execution failure

Posted by GitBox <gi...@apache.org>.
jamesnetherton commented on PR #4045:
URL: https://github.com/apache/camel-quarkus/pull/4045#issuecomment-1226986318

   Hmmm - looks likes there's a different failure. I wonder if previously we have been masking issues with Mongo. I'll dig into it a bit...


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