Skip to content
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

upgrading zookeeper to 3.7.1 #485

Merged
merged 2 commits into from
Jun 14, 2022

Conversation

Amit-Singh40
Copy link
Contributor

Signed-off-by: Amit-Singh40 amit.singh30@dell.com

Change log description

Upgraded the zookeeper version to 3.7.1 to fix vulnerabilities in 3.7.0

Purpose of the change

Fixes #484

What the code does

Upgrades the zookeeper to 3.7.1

How to verify it

Perform jenkins tests: Pravega_ZookeeperOperator, Pravega_Zookeeper_Upgrade and Pravega_ZookeeperOperator_E2E and github build.

Signed-off-by: Amit-Singh40 <amit.singh30@dell.com>
@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #485 (5c749ef) into master (1cc1844) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #485   +/-   ##
=======================================
  Coverage   85.04%   85.04%           
=======================================
  Files          12       12           
  Lines        1605     1605           
=======================================
  Hits         1365     1365           
  Misses        155      155           
  Partials       85       85           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1cc1844...5c749ef. Read the comment docs.

Copy link
Contributor

@jkhalack jkhalack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anishakj
Copy link
Contributor

@Amit-Singh40 Could you please change the version in docker/zu/build.gradle.kts as well?

Signed-off-by: Amit-Singh40 <amit.singh30@dell.com>
Copy link
Contributor

@anishakj anishakj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anishakj anishakj merged commit 6dea1e2 into pravega:master Jun 14, 2022
@AKamyshnikova
Copy link
Contributor

Any plans publish image with this fix?

mmoscher pushed a commit to mmoscher/zookeeper-operator that referenced this pull request Oct 12, 2022
* upgrading zookeeper to 3.7.1

Signed-off-by: Amit-Singh40 <amit.singh30@dell.com>

* updating zookeeper version in build.gradle.kts and readme file

Signed-off-by: Amit-Singh40 <amit.singh30@dell.com>
Signed-off-by: Marco Moscher <moscher@modell-aachen.de>
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.

Use zookeeper 3.7.1
4 participants