-
Notifications
You must be signed in to change notification settings - Fork 653
Description
/kind feature
What happened:
hello, maintainers
occm is currently support X-forwarded-for with service annotation. However, there is no support for X-forwarded-Port and X-forwarded-Proto, which are commonly required by HTTP applications behind L7 load balancers.
How to reproduce it:
And this is my proposed annotations.
ServiceAnnotationLoadBalancerXForwardedPort = "loadbalancer.openstack.org/x-forwarded-port"
ServiceAnnotationLoadBalancerXForwardedProto = "loadbalancer.openstack.org/x-forwarded-proto"According to the Octavia official documentation, both annotations are supported as listener insert_headers options.
See: https://docs.openstack.org/octavia/latest/user/feature-classification/index.html#operation_insert_headers_X_Forwarded_Port
Anything else we need to know?:
If maintainers agree with this approach, I would be happy to open a PR to implement this feature.
Please let me know if that would be acceptable.
Environment:
I'm using occm on kubernetes clusters with f5 driver.