zalando-incubator / kube-ingress-aws-controller / 21943516252

72%

push

github

feat: Track SSL policy source to prevent LB recreation on CLI changes (#774)

- Add HasSSLPolicyAnnotation field to distinguish annotation-based SSL
policy from CLI defaults
- Only enforce SSL policy matching for shared LBs when ingress has
explicit annotation
- Add SSL policy to inSync() check to trigger updates instead of new LB
creation
- Update load balancer SSL policy when adding ingress to allow CLI
changes to propagate
- Add comprehensive unit tests for SSL policy annotation detection and
matching logic

Fixes issue where changing --ssl-policy command-line flag would create
new load balancers instead of updating existing ones. Now only ingresses
with explicit SSL policy annotations will create separate load
balancers, while CLI changes will update existing LBs.

Fixes #772"

Signed-off-by: Lambert Pandian <lambert_pandian@trimble.com>

25 of 25 new or added lines in 2 files covered. (100.0%)

2866 of 3997 relevant lines covered (71.7%)

11.26 hits per line