fix(config): Change the default service type to ClusterIP

[MateusCastello]

*fix(config): Change the default service type to ClusterIP

There is no need for the front-end and datahub-gms to deafault this services as LoadBalancers, as this configuration opens an external public IP on Cloud Environments, leading to potential security breaches on developing and quick-start scenarios.

[github-actions]

This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io.

[pjaak]

Hi,

We ran into a similar issue recently where we specified an ingress resource that was internal, so we thought it was deployed internally, however, the default sub-chart values had also created a public-facing AWS load balancer. The fact that these values weren't exposed in the parent chart values means it is easy to miss. Would be good to get this reviewed by someone. I was going to create a PR with the same changes until I saw this one. LGTM

Thanks

[LiamStorkey]

This LGTM - I recently had a similar security alert go off for this default sub-chart public load balancer.

[RyanHolstien]

Hiya @MateusCastello ! Would you be willing to update the chart versions on this PR? Unfortunately you made this PR directly from your master branch instead of creating a branch so it's not letting me update the chart versions.

[github-actions]

This PR is stale. We will close it in 30 days if there is no comment or activity. If you want feedback but not able to get it on github please head to #contribute channel in slack at https://slack.datahubproject.io.

[github-actions]

Closing stale PR.