You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "nodece (via GitHub)" <gi...@apache.org> on 2025/06/03 04:12:42 UTC
[PR] [improve][test] Default-enable bk http in test setup [pulsar]
nodece opened a new pull request, #24375:
URL: https://github.com/apache/pulsar/pull/24375
### Motivation
bk http is an important feature. During bk upgrades, the http server may stop working unexpectedly.
A similar issue occurred in https://github.com/apache/pulsar/pull/20070, which didn't include tests to cover bk http functionality.
### Modifications
- Enabled bk http by default in the test configuration to ensure the http server is started and functional.
### Documentation
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `doc` <!-- Your PR contains doc changes. -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->
--
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
Re: [PR] [improve][test] Default-enable bk http in test setup [pulsar]
Posted by "nodece (via GitHub)" <gi...@apache.org>.
nodece commented on PR #24375:
URL: https://github.com/apache/pulsar/pull/24375#issuecomment-2933693993
@lhotari Closed by https://github.com/apache/pulsar/pull/20149, which includes these changes.
--
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
Re: [PR] [improve][test] Default-enable bk http in test setup [pulsar]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #24375:
URL: https://github.com/apache/pulsar/pull/24375#issuecomment-2933645966
> * Enabled bk http by default in the test configuration to ensure the http server is started and functional.
@nodece Are you planning to add a test case that also verifies that it actually works?
--
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
Re: [PR] [improve][test] Default-enable bk http in test setup [pulsar]
Posted by "nodece (via GitHub)" <gi...@apache.org>.
nodece closed pull request #24375: [improve][test] Default-enable bk http in test setup
URL: https://github.com/apache/pulsar/pull/24375
--
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