EventStore / EventStore.Charts

EventStore official Helm Charts
Other
29 stars 47 forks source link

remove exposing of ports since the service is headless #52

Open yashbhutwala opened 4 years ago

yashbhutwala commented 4 years ago

Signed-off-by: Yash Bhutwala ymb002@bucknell.edu

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

What this PR does / why we need it: Since a headless service is specified and therefore DNS handles the request, rather the kube-proxy, there is no need to specify exposing the ports. If DNS is being used, all container ports are open for communication 😄

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer: