Skip to content

[improve][pip]PIP-296: support storing brokerClient certificates in metadata store #21044

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ericsyh
Copy link
Contributor

@ericsyh ericsyh commented Aug 22, 2023

Fixes #xyz

Main Issue: #xyz

PIP: #xyz

Motivation

PIP for improving the createCluster API under configuring the geo-replication to a remote cluster with TLS enabled.

Modifications

Add the design doc for PIP 295. Pls note this PIP design doc is incomplete since I can't cover the detailed design and will engage some developing resources to help fulfill.

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

@ericsyh
Copy link
Contributor Author

ericsyh commented Aug 22, 2023

@mattisonchao @poorbarcode PTAL.

@github-actions github-actions bot added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Aug 22, 2023
@mattisonchao mattisonchao changed the title [improve][pip]PIP-295: support storing brokerClient certificates in zookeeper [improve][pip]PIP-295: support storing brokerClient certificates in metadata store Sep 5, 2023
@ericsyh ericsyh changed the title [improve][pip]PIP-295: support storing brokerClient certificates in metadata store [improve][pip]PIP-296: support storing brokerClient certificates in metadata store Sep 6, 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

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

@github-actions github-actions bot added the Stale label Oct 9, 2023
@Technoboy- Technoboy- added this to the 3.3.0 milestone Dec 22, 2023
@coderzc coderzc modified the milestones: 3.3.0, 3.4.0 May 8, 2024
@lhotari lhotari modified the milestones: 4.0.0, 4.1.0 Oct 11, 2024
@lhotari lhotari removed this from the 4.1.0 milestone Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. Stale type/PIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants