Skip to content

Conversation

adityaagg09
Copy link
Collaborator

@adityaagg09 adityaagg09 commented Oct 14, 2025

…on into a single commit for future upgrade integration

Description

(Write a concise description including what, why, how)
This PR consolidates all Backup-Change related changes into a single commit to simplify future upgrades and version rollouts. The goal is to make it easier for subsequent developers to collect, review, and reapply Backup-Change configurations in upcoming releases.

All the commit the backup-config tag are present inside this pr :- https://docs.google.com/spreadsheets/d/1B8G3-VmnqolrxBeNAI2XptkVvy9-Y8D7B0TnYWp5zXg/edit?gid=85047633#gid=85047633

Tests

The following tests are written for this issue:
(List the names of added unit/integration tests)

The following validation was performed:

  1. Ran the command mvn clean package -DskipTests
    [INFO] Reactor Summary for Apache ZooKeeper 3.8.4: [INFO] [INFO] Apache ZooKeeper ................................... SUCCESS [ 1.442 s] [INFO] Apache ZooKeeper - Documentation ................... SUCCESS [ 0.393 s] [INFO] Apache ZooKeeper - Jute ............................ SUCCESS [ 4.017 s] [INFO] Apache ZooKeeper - Server .......................... SUCCESS [ 8.396 s] [INFO] Apache ZooKeeper - Metrics Providers ............... SUCCESS [ 0.117 s] [INFO] Apache ZooKeeper - Prometheus.io Metrics Provider .. SUCCESS [ 1.072 s] [INFO] Apache ZooKeeper - Client .......................... SUCCESS [ 0.102 s] [INFO] Apache ZooKeeper - Recipes ......................... SUCCESS [ 0.243 s] [INFO] Apache ZooKeeper - Recipes - Election .............. SUCCESS [ 0.325 s] [INFO] Apache ZooKeeper - Recipes - Lock .................. SUCCESS [ 0.316 s] [INFO] Apache ZooKeeper - Recipes - Queue ................. SUCCESS [ 0.297 s] [INFO] Apache ZooKeeper - Assembly ........................ SUCCESS [ 1.366 s] [INFO] Apache ZooKeeper - Compatibility Tests ............. SUCCESS [ 0.099 s] [INFO] Apache ZooKeeper - Compatibility Tests - Curator ... SUCCESS [ 0.305 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 18.567 s [INFO] Finished at: 2025-10-14T14:20:43+05:30 [INFO] ------------------------------------------------------------------------

  2. Tried to setup the local zookeeper server it was able to establish the quorum

Screenshot 2025-10-14 at 2 18 43 PM

The following is the result of the "mvn test" command on the appropriate module:
(If CI test fails due to known issue, please specify the issue and test PR locally. Then copy & paste the result of "mvn test" to here.)

Changes that Break Backward Compatibility (Optional)

My PR contains changes that break backward compatibility or previous assumptions for certain methods or API. They include:
(Consider including all behavior changes for public methods or API. Also include these changes in merge description so that other developers are aware of these changes. This allows them to make relevant code changes in feature branches accounting for the new method/API behavior.)

Documentation (Optional)

In case of new functionality, my PR adds documentation in the following wiki page:
(Link the GitHub wiki you added)

…on into a single commit for future upgrade integration
@adityaagg09 adityaagg09 marked this pull request as ready for review October 14, 2025 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant