kubevirt / containerized-data-importer / #6007

50%

Build Type

push

travis-ci

Committed by web-flow

Commit Message

Add open-default-ports annotation to upload server pod (#4124)

* Extract upload server port into a common constant

Replace hardcoded 8443 references in the upload controller with
common.UploadServerPort to prevent silent breakage if the port
ever changes.

Assisted-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Ram Lavi <ralavi@redhat.com>

* Add open-default-ports annotation to upload server pod

Since the upload service became headless (#4052), DNS resolves directly
to the upload server pod IP. In OVN-Kubernetes clusters where the target
namespace uses network segmentation, the pod's default network interface
is locked down. The k8s.ovn.org/open-default-ports annotation opens port
8443 on the default interface, allowing the upload proxy to connect.

The annotation is harmless on non-OVN clusters where it is simply
ignored.

Assisted-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Ram Lavi <ralavi@redhat.com>

---------

Signed-off-by: Ram Lavi <ralavi@redhat.com>

Coverage Stats

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

13 existing lines in 2 files now uncovered.

14972 of 30205 relevant lines covered (49.57%)

0.56 hits per line