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

Docker files for alarm server #3312

Closed
wants to merge 8 commits into from
Closed

Docker files for alarm server #3312

wants to merge 8 commits into from

Conversation

georgweiss
Copy link
Collaborator

After som trials and errors.... Docker for the alarm service comes with some limitations. I have created three files to cover the use cases:

  • Run only Zookeeper and Kafka. Saves user from downloading and configuring Kafka.
  • Run the entire stack to import a configuration, only for default topic "Accelerator"
  • Run the entire stack in "normal mode", only for default topic "Accelerator"

Peer testing and feed-back appreciated.

@georgweiss georgweiss requested a review from shroffk March 5, 2025 15:37
@georgweiss georgweiss closed this Mar 6, 2025
@shroffk
Copy link
Member

shroffk commented Mar 6, 2025

Is there a reason you closed this PR?

@georgweiss
Copy link
Collaborator Author

Yes, I realized there might be a better strategy:

  1. Docker compose bundling Kafka/Zookeeper, Elastic and MongoDB. This would then be common for all middleware services.
  2. Docker compose for the individual middleware services.

Will create a new PR when I have verified this scheme.

@shroffk
Copy link
Member

shroffk commented Mar 6, 2025

yup that makes perfect sense...
docker compose for middlelayer backends... this would make networking much easier

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