load-balancer-and-ingress-services-for-kubernetes icon indicating copy to clipboard operation
load-balancer-and-ingress-services-for-kubernetes copied to clipboard

execution of gateway and httproute in any order

Open arihantg opened this issue 5 months ago • 8 comments

This PR add below changes:

  1. Re-validation and ingestion of related HTTPRoute on gateway updated
  2. Bug fix to avoid port append on Vhmatch on child vs on gateway update of port number

Manual Tests Done:

Invalid gateway to partially valid gateway with 2 routes -> routes got revalidated and 1 child vs got created Invalid gateway to vallid gateway with 2 routes -> routes got revalidated and all child vs got created partally valid gatway to valid gateway -> routes got revalidated and remaining child vs got re-created

Gateway api UT are passing on jenkins build

14:01:14 sudo docker run \
14:01:15 -e ENDPOINTSLICES_ENABLED="false" \
14:01:15 -w=/go/src/github.com/vmware/load-balancer-and-ingress-services-for-kubernetes \
14:01:15 -v /home/aviuser/workspace/ako_OS-pr-builder:/go/src/github.com/vmware/load-balancer-and-ingress-services-for-kubernetes golang:bullseye \
14:01:15 go test -mod=vendor github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests/... -failfast -timeout 0 -coverprofile cover-20.out -coverpkg=./...
14:01:15 	github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests		coverage: 0.0% of statements
14:02:08 ok  	github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests/graphlayer	413.219s	coverage: 19.4% of statements in ./...
14:09:33 ok  	github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests/ingestion	270.680s	coverage: 3.2% of statements in ./...
14:09:33 ok  	github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests/npltests	60.408s	coverage: 16.6% of statements in ./...
14:09:33 ok  	github.com/vmware/load-balancer-and-ingress-services-for-kubernetes/tests/gatewayapitests/status	334.853s	coverage: 16.6% of statements in ./...
14:09:33 make[1]: Leaving directory '/home/aviuser/workspace/ako_OS-pr-builder'

arihantg avatar Sep 04 '24 13:09 arihantg