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

Simplify installation of Memgraph HA chart using IngressNginx #110

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

as51340
Copy link
Collaborator

@as51340 as51340 commented Jan 27, 2025

Users can now install HA chart that uses ingress nginx with one command.
Configuration for probes is removed since I don't see a too big value at the moment in restarting the database when something goes wrong.

@as51340 as51340 marked this pull request as ready for review January 27, 2025 09:50
@as51340 as51340 self-assigned this Jan 27, 2025
@as51340 as51340 added the memgraph-ha Memgraph HA chart issue label Jan 27, 2025
@as51340 as51340 added this to the Memgraph 2.23 milestone Jan 27, 2025
@as51340 as51340 force-pushed the simplify-helm-installation branch from 37e7d30 to b97d4e5 Compare January 27, 2025 12:39
Copy link
Collaborator

@antejavor antejavor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, discussed on the call.

@as51340 as51340 force-pushed the simplify-helm-installation branch from b97d4e5 to 8f9cca4 Compare January 27, 2025 12:53
@as51340 as51340 merged commit fe0a869 into main Jan 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
memgraph-ha Memgraph HA chart issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants