Skip to content

CURATOR-596 Upgrade ZooKeeper to 3.7.0 #385

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.

Sign up for GitHub

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

Jump to bottom
Closed
eolivelli wants to merge 4 commits into from
Closed

CURATOR-596 Upgrade ZooKeeper to 3.7.0 #385

eolivelli wants to merge 4 commits into from

Conversation

@eolivelli
Copy link
Contributor

@eolivelli eolivelli commented May 17, 2021

No description provided.

@eolivelli eolivelli requested review from Randgalt and cammckenzie May 17, 2021 13:17
@cammckenzie
Copy link
Contributor

cammckenzie commented May 18, 2021

Changes look fine to me, but it appears the build unit tests are failing on the CI build?

@eolivelli
Copy link
Contributor Author

eolivelli commented May 18, 2021

Yes.
Probably it is better to just update to latest 3.6.3 and move to 3.7.0 when I am done on #379

@eolivelli
Copy link
Contributor Author

eolivelli commented May 18, 2021

On the other hand people may try to use Curator with 3.7.0 and we must be sure that they do not have problems.
I will investigate the failures

@Hronom
Copy link

Hronom commented Dec 13, 2021

What the status?

@eolivelli
Copy link
Contributor Author

eolivelli commented Dec 14, 2021

@Hronom
thanks for the reminder !

I have resolved the conflicts, if CI passes I will commit the patch.

Maybe we can follow up with a release of Curator

@elkozmon elkozmon mentioned this pull request Dec 17, 2021
Open
@tommy04062019
Copy link

tommy04062019 commented Feb 18, 2022

Anything new here?I've waiting for this a long time

@tisonkun
Copy link
Member

tisonkun commented May 10, 2022

@eolivelli it seems there is still some test failure :(

@mariouso
Copy link

mariouso commented Jun 1, 2022

Hi @eolivelli is there any update on the curator with 3.7.1 zk version and the new testing server?

@eolivelli
Copy link
Contributor Author

eolivelli commented Jul 1, 2022

@tisonkun feel free to pick up this patch

@eolivelli
Copy link
Contributor Author

eolivelli commented Jul 1, 2022

we should move to 3.7.1

@tisonkun
Copy link
Member

tisonkun commented Jul 14, 2022

Resolved by #426.

@tisonkun tisonkun closed this Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cammckenzie cammckenzie Awaiting requested review from cammckenzie

@Randgalt Randgalt Awaiting requested review from Randgalt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@eolivelli @cammckenzie @Hronom @tommy04062019 @tisonkun @mariouso