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

Delete clustering information that references zookeeper and helix #2806

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FireBurn
Copy link

@FireBurn FireBurn commented Sep 1, 2020

It is totally inaccurate now, zookeeper and helix support was removed from Appformer 3 years ago

It should probably be replaced with something regarding NFS but I'm still figuring that out

It is totally inaccurate now, zookeeper and helix support was removed from Appformer 3 years ago

It should probably be replaced with something regarding NFS but I'm still figuring that out
@kie-ci
Copy link

kie-ci commented Sep 1, 2020

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

2 similar comments
@kie-ci
Copy link

kie-ci commented Sep 1, 2020

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@kie-ci
Copy link

kie-ci commented Sep 1, 2020

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@sterobin sterobin added the under review Label for PRs pending SME or peer review label Sep 1, 2020
@sterobin
Copy link
Contributor

sterobin commented Sep 1, 2020

Thanks for this @FireBurn ! If you're proposing to remove that section, then a few additional things are required:

But, relevant replacement content would be optimal anyway.

@csherrar @kaldesai Which one of you would this apply to? Can you add the relevant SME(s) to verify as well? See also #2805

Thanks guys.

@FireBurn
Copy link
Author

FireBurn commented Sep 1, 2020

I wasted quite a lot of time figuring out zookeeper and helix and and even longer time trying to figure out why clustering wasn't working with it, I'll hopefully have clustering figured out in the next week or so, and will try and write something up

Is someone already working on this? It seems like quite a big bit of functionality to be wrong for so long

@sterobin
Copy link
Contributor

sterobin commented Sep 1, 2020

@FireBurn Right, understood. I've pinged the relevant writers to see if they have any info on this or if they know who the relevant devs would be to ping here from community. Thanks.

@vidya-iyengar @heathjoy FYI as well.

@csherrar
Copy link
Contributor

csherrar commented Sep 2, 2020

@sterobin Ack, pinging SMEs.

@sutaakar @domhanak @dgutierr Do you know who the relevant devs would be to ping here from community?

Example doc link

@sutaakar
Copy link
Contributor

sutaakar commented Sep 2, 2020

I have no idea

@domhanak
Copy link
Contributor

domhanak commented Sep 3, 2020

Imho this can go away, zookeeper was used with older versions, in current time it is obsolete.
However, a doc that says how to do it now, should be reinstated in its place.

@csherrar
Copy link
Contributor

csherrar commented Sep 3, 2020

@domhanak Can you look over the Clustering Doc please and let me know if there is anything that aligns with this section's purpose? @FireBurn mentioned NFS.

Installing and configuring RHPAM/RHDM in a EAP clustered environment - Configuring an NFS server

@domhanak
Copy link
Contributor

domhanak commented Sep 3, 2020

@tomasdavidorg could you please look at above comments? Is there a possibility to have community doc for this?

@tomasdavidorg
Copy link
Contributor

Running RHPAM on clustered environment (NFS file system) is tech preview right now. I think this can be replacement for zookeeper/Helix in community.

@sterobin
Copy link
Contributor

@FireBurn See above comments.

@sterobin
Copy link
Contributor

@csherrar or @kaldesai , can you coordinate with @FireBurn and the relevant SMEs to see what needs to happen here and by whom? Same with #2805 . Both PRs seem to be at a standstill.

@vidya-iyengar @heathjoy FYI as well.

@domhanak
Copy link
Contributor

domhanak commented Jun 2, 2021

stale PR

@kie-ci
Copy link

kie-ci commented Jun 2, 2021

Can one of the admins verify this patch?

1 similar comment
@kie-ci
Copy link

kie-ci commented Jun 2, 2021

Can one of the admins verify this patch?

@FireBurn
Copy link
Author

The docs still reference zookeeper and helix in 7.56.0 (and no doubt 7.57.0 when it's tagged) when support was removed back in 7.5.0 (kiegroup/appformer@f8e4602) back in 2017 via https://issues.redhat.com/browse/AF-894

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
under review Label for PRs pending SME or peer review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants