Skip to content

Add Kubernetes resource limit/request/overwrite integration tests

Arran Walker requested to merge ajwalker/k8s-resource-integration-tests into master

!2279 (merged) needs to be merged before this MR. Once !2279 (merged) is merged, this MR's target should be updated to master

What does this MR do?

Adds integration tests for k8s resource limit/request/overwrites.

Why was this MR needed?

Manual QA of these tests has been known to take a long time.

Notes for reviewer

The Is() implementation for overwriteTooHighError is updated to compare the resource and limit values. This is to ensure our tests actually return the error we expect and not just any overwriteTooHighError error.

What's the best way to test this MR?

go test -count 1 -run TestRunTestsWithFeatureFlag/testResourceLimits  ./executors/kubernetes

What are the relevant issue numbers?

Closes #26958

Edited by Georgi N. Georgiev

Merge request reports