helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

allow defining ipFamilyPolicy for external service

Open nadiamoe opened this issue 1 year ago • 11 comments

For dual stack (IPv4 and IPv6) clusters, services need to be explicitly opted-in into having two addresses. This is done through the ipFamilyPolicy field, as documented in https://kubernetes.io/docs/concepts/services-networking/dual-stack/#services.

I've wired the templates in a way where if this field is not present, nothing is rendered. This preserves the usual behavior of not specifying this value altogether for services.

nadiamoe avatar Aug 05 '24 11:08 nadiamoe

CI/CD fails for reasons unknown to me, but I think the PR itself should be okay.

nadiamoe avatar Aug 17 '24 20:08 nadiamoe

@fastlorenzo this looks good to me, can we get it merged?

DrPsychick avatar Oct 07 '24 21:10 DrPsychick

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Nov 07 '24 02:11 github-actions[bot]

Unstale

nadiamoe avatar Nov 10 '24 19:11 nadiamoe

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Dec 11 '24 02:12 github-actions[bot]

Unstale

WebSpider avatar Dec 11 '24 07:12 WebSpider

Hey folks, this has been lingering for a long while now. Is there anything I can do to help get this merged? Due to the nature of helm charts, this not being upstream makes me maintain and deploy a fork of the chart with this commit on it, which is not great. Thanks!

nadiamoe avatar Dec 11 '24 11:12 nadiamoe

Hi @roobre I'm in contact with the maintainer to see how we can spped things up here.

DrPsychick avatar Dec 15 '24 21:12 DrPsychick

@fastlorenzo Sorry for the direct ping, but you seem to be the most recent contributor here. Is there any chance you can stamp this? Should be a simple change 🙏🏻

nadiamoe avatar Mar 04 '25 11:03 nadiamoe

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Oct 02 '25 02:10 github-actions[bot]

@nadiamoe Can you rebase?

promasu avatar Oct 17 '25 14:10 promasu

I know it's been quite a while, but we're now slowly getting up to speed with timely reviews again. @nadiamoe would you mind to rebase the PR? It's already approved and I'll merge it right away.

DrPsychick avatar Dec 05 '25 20:12 DrPsychick