Skip to content

[SPARK-56555][K8S] Add ubuntu mirror repositories to K8s Dockerfile for redundancy#55436

Closed
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:add-ubuntu-mirror-k8s
Closed

[SPARK-56555][K8S] Add ubuntu mirror repositories to K8s Dockerfile for redundancy#55436
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:add-ubuntu-mirror-k8s

Conversation

@sarutak
Copy link
Copy Markdown
Member

@sarutak sarutak commented Apr 21, 2026

What changes were proposed in this pull request?

Similar to #55389, this PR adds ubuntu mirror repositories to K8s Dockerfiles for redundancy.
mirrors.edge.kernel.org seems sufficiently public and reliable.

Why are the changes needed?

Sometimes, ubuntu repositories including archive.ubuntu.com and security.ubuntu.com are very slow, causing docker build failure.
We saw such failure on GA.
https://github.com/sarutak/spark/actions/runs/24552461498/job/71781474186

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manually confirmed docker build works with modified Dockerfile.

Was this patch authored or co-authored using generative AI tooling?

Kiro CLI / Opus 4.6

Add mirrors.edge.kernel.org as a fallback apt mirror to improve CI stability.
The mirror is only added when the base image is Ubuntu (guarded by checking sources file existence).
@sarutak sarutak marked this pull request as ready for review April 21, 2026 05:37
@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 21, 2026

IIRC, this also affects users who use bin/docker-image-tool.sh to create spark image?

@sarutak
Copy link
Copy Markdown
Member Author

sarutak commented Apr 21, 2026

@pan3793

IIRC, this also affects users who use bin/docker-image-tool.sh to create spark image?

Yes. However, the change only adds a mirror as an additional apt source. The original repositories remain untouched, so it works as a fallback. Also, the mirror is only added when the base image is Ubuntu, so non-Ubuntu base images are not affected.

Copy link
Copy Markdown
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you, @sarutak .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants