Skip to content

Comments

WIP: Add HAProxy ingress controller support to theia-cloud Helm chart#104

Draft
jfaltermeier wants to merge 1 commit intomainfrom
jf/haproxy
Draft

WIP: Add HAProxy ingress controller support to theia-cloud Helm chart#104
jfaltermeier wants to merge 1 commit intomainfrom
jf/haproxy

Conversation

@jfaltermeier
Copy link
Contributor

  • Add default HAProxy ingress annotations and class name handling
  • Add ingress path suffix helper and pass it to the operator
  • Use controller-specific path suffix in service and landing page ingress templates

eclipse-theia/theia-cloud#464

@jfaltermeier jfaltermeier force-pushed the jf/haproxy branch 6 times, most recently from 681b3ac to 8a1dbf2 Compare January 26, 2026 09:26
@jfaltermeier
Copy link
Contributor Author

One remark, the ($|(/.*)) to (/|$)(.*) change in the nginx pattern matches the delimiter and the path as separate groups.
This was an initial attempt to reuse this regexp for HAProxy as well, but in the end it works differently there. I still think this matching makes more sense, so I kept it in.

* add default HAProxy ingress annotations for instances, landing page,
and service ingresses
* add helpers for controller-specific ingress path suffix and pathType
* pass ingressPathSuffix to the operator and update ingress paths and
nginx rewrite-target accordingly
* add project-info

eclipse-theia/theia-cloud#464
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.

1 participant