Skip to content

Conversation

@zeroflag
Copy link
Contributor

@zeroflag zeroflag commented Feb 7, 2023

What changes were proposed in this pull request?

Upgrading curator and zookeeper.

How was this patch tested?

$ mvn test -Dtest=ZookeeperTokenStateServiceTest

@zeroflag zeroflag self-assigned this Feb 8, 2023
@smolnar82
Copy link
Contributor

@zeroflag - the changes themselves look good to me. There is only one thing I'd like to clarify: what's the driver of this change? The old versions are affected by some CVEs?

Copy link
Contributor

@pzampino pzampino left a comment

Choose a reason for hiding this comment

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

LGTM

@zeroflag zeroflag merged commit 383e045 into apache:master Feb 9, 2023
zeroflag pushed a commit to zeroflag/knox that referenced this pull request Mar 1, 2023
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.

3 participants