Skip to content

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

Draft
amcgee wants to merge 6 commits into
masterfrom
feat-cluster-ha
Draft

feat: add proof-of-concept high availability cluster config#28
amcgee wants to merge 6 commits into
masterfrom
feat-cluster-ha

Conversation

@amcgee

@amcgee amcgee commented Mar 1, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@bobjolliffe

Copy link
Copy Markdown

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

amcgee commented Mar 1, 2023

Copy link
Copy Markdown
Contributor Author

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

0c2695b

@amcgee

amcgee commented Jun 7, 2023

Copy link
Copy Markdown
Contributor Author

@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