michaeljmarshall opened a new pull request, #20336:
URL: https://github.com/apache/pulsar/pull/20336

   Builds on: https://github.com/apache/pulsar/pull/20289
   
   ### Motivation
   
   There are many certificates in our test code base. It would be much simpler 
to have one place were we create and manage certificates so that when we need 
to make changes, they are consolidated.
   
   There is likely one or two more PRs to finish consolidating certs.
   
   ### Modifications
   
   * Remove certs that are no longer used
   * Replace references to old certs with references to the 
`certificate-authority` certs
   * Create new server certs with valid hostnames on them so that tests will 
pass. Document the process used to create these certs.
   * Fix an issue in the `PulsarTestContext` where the configuration was not 
correctly updated.
   * Remove configurations that allow for insecure connections in tests that 
are doing some kind of TLS verification. The only places where we leave 
insecure validation in place is tests that are specifically verifying the 
functionallity.
   
   ### Verifying this change
   
   When tests pass, this change will be correctly verified.
   
   ### Documentation
   
   - [x] `doc`
   This PR includes doc changes
   
   ### Matching PR in forked repository
   
   PR in forked repository: https://github.com/michaeljmarshall/pulsar/pull/48


-- 
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