helm-openldap
helm-openldap copied to clipboard
jp-gouin
fix: Allow IPv6 on ltb-passwd
What this PR does / why we need it:
- Listen on both IPv4 and IPv6 in the ltb-passwd to fix health probes on dual-stack clusters.
- Include upstream chart env in main values.yaml.
https://github.com/tiredofit/docker-self-service-password/issues/66
Pre-submission checklist:
- [x] Did you explain what problem does this PR solve? Or what new features have been added?
- [ ] Have you updated the readme?
- [x] Is this PR backward compatible? If it is not backward compatible, please discuss open a ticket first
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
![stale[bot] avatar](https://avatars.githubusercontent.com/in/1724?v=4 "Published by a pub.dev verified publisher"){kind=small}
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
Hi @SuperQ ,
I'm not sure your issue need a fix in the values as the chart already allows you specifying ltb-passwd.env
Maybe would make more sense to add the info in the readme ?
I know it's been a while , so feel free to close the PR and I'll include the variable in the readme in the next update
I don't really mind how it is changed, but I think the defaults for this chart should work correctly with dual stack by default.
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.