stacklok / toolhive / 20468138523

51%

push

github

K8s Client: Apply Pod Spec Annotations (#3152)

Summary
The applyPodTemplatePatch function in pkg/container/kubernetes/client.go was only copying Labels from the pod template patch but NOT Annotations.

Fixes #3153

Fix
// Copy annotations from the patched spec to our template
if patchedSpec.ObjectMetaApplyConfiguration != nil && len(patchedSpec.Annotations) > 0 {
    baseTemplate = baseTemplate.WithAnnotations(patchedSpec.Annotations)
}
Tests Created
Unit test (pkg/container/kubernetes/client_test.go):

TestApplyPodTemplatePatchAnnotations - Tests the function directly
E2E test (test/e2e/chainsaw/operator/single-tenancy/test-scenarios/pod-annotations/):

Creates MCPServer with custom annotations in podTemplateSpec
Verifies annotations appear on the StatefulSet's pod template
The chainsaw tests previously failed without the fix:

---------

Signed-off-by: Jeremy Drouillard <jeremy@stacklok.com>

3 of 3 new or added lines in 1 file covered. (100.0%)

28539 of 56434 relevant lines covered (50.57%)

58.67 hits per line