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/11/05 11:07:54 UTC
[GitHub] [camel-quarkus] jamesnetherton opened a new issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
jamesnetherton opened a new issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239
Needs investigating properly, but I don't think the custom configs to apply the necessary security bits to the Kafka broker are working. It may be related to the testcontainers version bump from 1.16.0 to 1.16.2.
I'm disabling the tests 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-quarkus] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-956357014
@jamesnetherton I think you are right. This is related to testcontainers 1.6.2 changes with
https://github.com/testcontainers/testcontainers-java/blob/4df9b900e79e4a3033ef0975abacf9064a0e7353/modules/kafka/src/main/java/org/testcontainers/containers/KafkaContainer.java#L76-L86
it set ```KAFKA_ADVERTISED_LISTENERS``` with ```BROKER:localhost:9092``` explicitly. So Kafka SSL is not enabled when lauching the instance.
I think this is a bug in testcontainers.
--
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] ppalaga commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961783964
I think we generally prefer marking issues as fixed only after they are merged in `main`. `quarkus-main` is an instable (is rebased regularly) branch and so the sha1 of the fix will change several times till it reaches `main`. I mean each fixed issue should contain a stable reference to the fix. This is important for both to be able to audit the fix at any time in the future and also to be able to port the fix to older branches.
--
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] zhfeng closed issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng closed issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239
--
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 closed issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
jamesnetherton closed issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239
--
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] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-956358663
It should be a simple workaround to override ```protected void configure()``` to set ```KAFKA_ADVERTISED_LISTENERS``` with ```SSL://localhost:9093,BROKER://localhost:9092```. I've checked that it works on my local machine.
--
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] zhfeng closed issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng closed issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239
--
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] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961603746
--
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] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961603746
This should be fixed in ```quarkus-main``` branch.
--
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] ppalaga commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961783964
I think we generally prefer marking issues as fixed only after they are merged in `main`. `quarkus-main` is an instable (is rebased regularly) branch and so the sha1 of the fix will change several times till it reaches `main`. I mean each fixed issue should contain a stable reference to the fix. This is important for both to be able to audit the fix at any time in the future and also to be able to port the fix to older branches.
--
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] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961805532
OK, I get it. Let's reopen 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-quarkus] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-956357014
--
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] ppalaga commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961783964
I think we generally prefer marking issues as fixed only after they are merged in `main`. `quarkus-main` is an instable (is rebased regularly) branch and so the sha1 of the fix will change several times till it reaches `main`. I mean each fixed issue should contain a stable reference to the fix. This is important for both to be able to audit the fix at any time in the future and also to be able to port the fix to older branches.
--
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] zhfeng commented on issue #3239: [Quarkus 2.5.0] Kafka SASL / SSL tests are failing
Posted by GitBox <gi...@apache.org>.
zhfeng commented on issue #3239:
URL: https://github.com/apache/camel-quarkus/issues/3239#issuecomment-961603746
--
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