[bitnami/nginx] Changed default type to ClusterIP.

[kometen]

Description of the change

Benefits

Possible drawbacks

Applicable issues

• fixes [bitnami/nginx] Changed the default type from LoadBalancer to ClusterIP #25085

Additional information

Checklist

• Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
• Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
• Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
• All commits signed off and in agreement of Developer Certificate of Origin (DCO)

[carrodher]

Thank you for initiating this pull request. We appreciate your effort.

Could you please add a description of the necessity of this change?

Just a friendly reminder that it's important to sign your commits. Adding your signature certifies that you either authored the patch or have the necessary rights to contribute to the changes. You can find detailed information on how to do this in the “Sign your work” section of our contributing guidelines.

Apart from that, could you please bump the chart version in the Chart.yaml file?

Feel free to reach out if you have any questions or need assistance with the signing process.

[kometen]

Hi. Thank you for your kind response.

Many, or some, :-) will deploy this nginx helm chart with an already existing ingress service in place. So it makes more sense to have the service type to ClusterIP rather than LoadBalancer as default.

I had a similar issus as this closed issue at #11846.

Thank you.

[github-actions]

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

[github-actions]

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary.