memgraph / helm-charts

Helm charts for deploying Memgraph, an open-source in-memory graph database.
https://memgraph.github.io/helm-charts/
Apache License 2.0
13 stars 9 forks source link

Update test-connection.yaml #24

Closed pwretmo closed 2 months ago

pwretmo commented 3 months ago

"{{ .Release.Name }}-memgraph-test" does not generate a unique name, which is required when a helm chart includes multiple instances of the memgraph chart.

For example when a chart has two dependencies on memgraph:

dependencies:
- name: memgraph
  version: ~0.1.0
  repository: https://memgraph.github.io/helm-charts
- name: memgraph
  alias: memgraph2
  version: ~0.1.0
  repository: https://memgraph.github.io/helm-charts
katarinasupe commented 3 months ago

Hi @pwretmo, thank you for the contribution. The plan is to wrap up https://github.com/memgraph/helm-charts/pull/22 tomorrow, with Memgraph 2.17. @antejavor, the maintainer, will be back from vacation next week and then he'll review this PR and include it in the next release. I hope that works for you 🙏

pwretmo commented 3 months ago

Hi @katarinasupe, thanks for the update! :)