Skip to content

Commit

Permalink
DOC: document incompatibilty between route-acl annotation and multipl…
Browse files Browse the repository at this point in the history
…e matches path in Ingress
  • Loading branch information
hdurand0710 authored and oktalz committed Nov 8, 2023
1 parent 302a139 commit 6ddc870
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
5 changes: 5 additions & 0 deletions documentation/annotations.md
Original file line number Diff line number Diff line change
Expand Up @@ -1451,6 +1451,11 @@ haproxy.org/response-set-header: |

:information_source: In order for the service to be handled by the Ingress Controller, it is still mandatory to put it in an ingress rule. Using only `route-acl` won't be enough.

:information_source: Note that this annotation is not compatible with an Ingress having multiple paths that will match a request.
Without this annotation, the precedence is given first to the longest matching path.
But with the annotation, the first use_backend rule in the config that matches the request will be used.


Possible values:

- A string describing an in-line [HAProxy ACL](https://www.haproxy.com/blog/introduction-to-haproxy-acls/).
Expand Down
4 changes: 4 additions & 0 deletions documentation/doc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1383,6 +1383,10 @@ annotations:
- Insert a custom route (use_backend rule) to route ingress traffic to the annotated service based on the provided ACL.
tip:
- In order for the service to be handled by the Ingress Controller, it is still mandatory to put it in an ingress rule. Using only `route-acl` won't be enough.
- |
Note that this annotation is not compatible with an Ingress having multiple paths that will match a request.
Without this annotation, the precedence is given first to the longest matching path.
But with the annotation, the first use_backend rule in the config that matches the request will be used.
values:
- A string describing an in-line [HAProxy ACL](https://www.haproxy.com/blog/introduction-to-haproxy-acls/).
applies_to:
Expand Down

0 comments on commit 6ddc870

Please sign in to comment.