sijie edited a comment on issue #6683: [broker]Handle BadVersionException 
thrown by updateSchemaLocator()
URL: https://github.com/apache/pulsar/pull/6683#issuecomment-611984314
 
 
   > The fact that the health check never works when running with Docker images 
built from the 2.5.1 candidate is a regression. 
   
   I don't understand "never" here. The current buggy behavior is - it works 
when the topic exists, and it will fail the first time when the topic was 
deleted due to inactivity. 
   
   I have just verified that branch-2.5 + this fix has fixed the buggy behavior 
described above.
   
   ```
   $ curl localhost:8080/admin/v2/brokers/health
   ok[sijie@sijies-mbp pulsar ]$ curl localhost:8080/admin/v2/brokers/health
   ok[sijie@sijies-mbp pulsar ]$ curl localhost:8080/admin/v2/brokers/health
   ok
   ```
   
   ---
   
   If you are seeing health check failure using Docker Images, I don't think it 
is a problem of the code. It sounds more like a problem of your docker image. 
Because the broker code shouldn't have any difference running locally than 
running in a docker.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to