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

feat: add proof-of-concept high availability cluster config #28

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

amcgee
Copy link
Member

@amcgee amcgee commented Mar 1, 2023

No description provided.

@bobjolliffe
Copy link

Nice work. A few small improvements on server,xml

  1. -- replace with shutdown="-1". You are opening a control port unnecessarily, also with a well known cleartext password. Just simplify and get rid of it :-)
  2. Get rid of the ajp connector. I don't think it is being used
  3. In production we wouldn't usually have unpackWars=true. In fact better to just provide tomcat with readonly access to the unpacked webapp. It might be reasonable enough to do it this way for the POC, but if you want to get closer to production ready (people will copy and run this!) it might be worth to implement it this way.

@amcgee
Copy link
Member Author

amcgee commented Mar 1, 2023

@bobjolliffe thanks! Is this more or less what you meant?

0c2695b

@amcgee
Copy link
Member Author

amcgee commented Jun 7, 2023

@Philip-Larsen-Donnelly pointed out that this probably needs to be extended with a shared logical volume for the file store

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.

2 participants