hi everyone, we’ve got a usecase to open up the Re...
# all-things-deployment
a
hi everyone, we’ve got a usecase to open up the RestAPI to access outside the k8s cluster. I noticed the current helm chart didn’t have a template for an ingress on the datahub-gms service. I’ve opened this PR, which adds the template for the ingress. Its my first PR in the repo, so let me know if it doesn’t meet the required structures and standards. Thanks!
e
Thanks for contributing!! We will merge it in soon
a
Brilliant, thanks.
I’ve also updated the community chart for neo4j so it can deal with extra environment variables correctly. This lets us control the memory limits for neo4j within the pod. Should I open another PR to bump the chart version?
e
Yes Please! Thanks a lot!!
a
Should I wait for this to merge in before doing that? Might get a bit confusing with the version numbers on the charts!
m
Welcome to the community @adamant-pharmacist-61996 and thanks for that first PR!
😀 1