You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "dlg99 (via GitHub)" <gi...@apache.org> on 2023/04/13 23:37:24 UTC

[GitHub] [pulsar] dlg99 opened a new issue, #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

dlg99 opened a new issue, #20091:
URL: https://github.com/apache/pulsar/issues/20091

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Version
   
   3.0 RC 1
   
   ### Minimal reproduce step
   
   similar to https://github.com/apache/pulsar/issues/20071
   
   See https://lists.apache.org/thread/f2fhvyx202xzxbho909j430h63yvwjlo
   
   ### What did you expect to see?
   
   Regression caught on CI
   
   ### What did you see instead?
   
   regression caught outside of CI
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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@pulsar.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] MMirelli commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "MMirelli (via GitHub)" <gi...@apache.org>.
MMirelli commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1515827528

   I can try to add the test. Should it be an integration test like this one: https://github.com/MMirelli/pulsar/pull/2?


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] michaeljmarshall commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "michaeljmarshall (via GitHub)" <gi...@apache.org>.
michaeljmarshall commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1514919230

   Multiple contributors verified the fix, which is sufficient to unblock the release, in my opinion. We should follow up soon and add a test that enables the broker interceptors.


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] github-actions[bot] commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1556062786

   The issue had no activity for 30 days, mark with Stale label.


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1556068320

   > I can try to add the test. Should it be an integration test like this one: https://github.com/MMirelli/pulsar/pull/2?
   
   This looks a good example.


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] cbornet commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "cbornet (via GitHub)" <gi...@apache.org>.
cbornet commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1513875374

   Is this fixed by https://github.com/apache/pulsar/pull/20127 ?


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] dlg99 commented on issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "dlg99 (via GitHub)" <gi...@apache.org>.
dlg99 commented on issue #20091:
URL: https://github.com/apache/pulsar/issues/20091#issuecomment-1514901398

   @cbornet the fix you linked (BK upgrade) fixes the rootcause of the last issue we found, the issue is that we don't run integration-level test of pulsar + interceptors. 
   The issue is not fixed but I don't think it is a release blocker


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar] tisonkun closed issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled

Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun closed issue #20091: [Bug] CI: pulsar does not run tests with interceptors enabled
URL: https://github.com/apache/pulsar/issues/20091


-- 
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@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org