Skip to content

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

Draft
amcgee wants to merge 6 commits intomasterfrom
feat-cluster-ha
Draft

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

Conversation

@amcgee
Copy link
Copy Markdown
Contributor

@amcgee amcgee commented Mar 1, 2023

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
Copy link
Copy Markdown
Contributor Author

amcgee commented Mar 1, 2023

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

0c2695b

@amcgee
Copy link
Copy Markdown
Contributor 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