Skip to content

Replace nginx with haproxy#31

Merged
bryanbellsmith merged 14 commits into
mainfrom
replace-nginx-with-haproxy
Feb 6, 2026
Merged

Replace nginx with haproxy#31
bryanbellsmith merged 14 commits into
mainfrom
replace-nginx-with-haproxy

Conversation

@bryanbellsmith

@bryanbellsmith bryanbellsmith commented Feb 5, 2026

Copy link
Copy Markdown
Collaborator

Purpose

The ingress controller ingress-nginx will soon be deprecated. The intent of this change is to replace this chart with haproxy-ingress. Additionally, the external secrets operator instructions for Azure have been added.

What's changed

  • Removed chart dependencies - instructions added to manually install these instead (which simplifies some chart interactions)
  • Switched README.md instructions for ingress-nginx with haproxy-ingress
  • Updated README.md with External Secrets Operator instructions for AKS
  • Renamed images in README.md to correspond with AWS ESO instructions
  • Added images for Azure ESO instructions
  • Updated NOTES.txt to include pod instructions regardless of ingress setup
  • Reworked cert-manager templates for self-signed certificates
  • Minor bugfixes for External Secrets Operator templates
  • Switched examples in values.yaml to be YAML formatted, rather than mixing JSON
  • Added a lot more comments in values.yaml to describe the options
  • Removed cert-manager and ingress-nginx sections from values.yaml, they are now configured outside of the Private AI chart
  • Minor updates to missing parameters in values.yaml

Testing

  • Tested locally with helm lint
  • Tested by Github CI
  • Tested in an AKS cluster
  • Tested in an AWS cluster

Checklist

  • Update chart version if planning a release
  • Update README.md

@a-guiducci a-guiducci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 🚀

@bryanbellsmith
bryanbellsmith merged commit 5b91fa3 into main Feb 6, 2026
1 check passed
@bryanbellsmith
bryanbellsmith deleted the replace-nginx-with-haproxy branch February 6, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants