search

saikatharryc / helm

Helm chart for judge0 and Muescheli the antivirus
4 stars 2 forks source link

changed port name from http to http-port #9

Open NiranjanSwabhav opened 2 years ago

NiranjanSwabhav commented 2 years ago

Solves the service-pod (svc name: judge0) connection issue

off77th commented 10 months ago

Nice catch. The current helm-chart version has the wrong Service port binding. In the Deployment configuration, we have application port, called "http-port": https://github.com/saikatharryc/helm/blob/master/charts/judge0/templates/deployment.yaml#L52-L54 But in the Service.yaml "ports" section targets "http" instead of "http-port": https://github.com/saikatharryc/helm/blob/master/charts/judge0/templates/service.yaml#L25-L31

This PR solves the issue.