phijohns-tibco opened a new issue, #17874:
URL: https://github.com/apache/pulsar/issues/17874

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) 
and found nothing similar.
   
   
   ### Motivation
   
   The currently shipped version of Zookeeper does not allow users to disable 
HTTP Track/Trace functionality. The 3.8.0 version allows this. There is no 
functional difference between 3.6.x and 3.8.x. That being said, this is where 
the fun begins. When testing this fix I found that a substantial number of 
tests failed. While I have not yet done an extensive review of the failures, I 
am seeing a pattern forming. Zookeeper 3.8.0 removed log4j from it's package. 
All of the errors I have reviewed so far have something to do with logging. I 
believe we inadvertently relying on zookeeper to supply log4j for the test 
harness instead of implicitly adding  it.
   
   ### Solution
   
   The solution for this is to ensure that log4j is available for the test 
harness. I was hoping it would be sufficient to update the scope in the 
top-level pom.xml file to <scope>compile</scope>. It appears as if that is not 
enough seeing as some of the CICD workflow explicitly state a different pom.xml 
file for testing. 
   
   ### Alternatives
   
   _No response_
   
   ### Anything else?
   
   I believe this will take be a couple of weeks to get through and 
exhaustively verify results.
   
   ### Are you willing to submit a PR?
   
   - [X] 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to