Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Allow to overwrite the ko base image path #1688

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Conversation

Laubi
Copy link
Contributor

@Laubi Laubi commented Feb 6, 2025

What this PR does / Why we need it:

This change allows us to use a different proxy instead of querying docker directly, potentially breaking the pipeline because of "You have reached your pull rate limit".

By default, it still pulls from the docker-hub

Documentation about ko: https://ko.build/configuration/

Special notes for your reviewer:

Does this PR introduce a user-facing change?

no

@Laubi Laubi marked this pull request as ready for review February 6, 2025 14:36
@Laubi Laubi requested a review from a team as a code owner February 6, 2025 14:36
Copy link

github-actions bot commented Feb 6, 2025

Unit Test Results

1 791 tests  ±0   1 790 ✅ ±0   54s ⏱️ ±0s
  130 suites ±0       1 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 874a9bc. ± Comparison against base commit c97f376.

♻️ This comment has been updated with latest results.

tomazpu
tomazpu previously approved these changes Feb 10, 2025
Copy link
Contributor

@arthurpitman arthurpitman left a comment

Choose a reason for hiding this comment

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

Looks good, but I'm curious: how does the 3.20 in alpine:3.20 get updated? Will this still happen with this change?

@Laubi
Copy link
Contributor Author

Laubi commented Feb 10, 2025

Looks good, but I'm curious: how does the 3.20 in alpine:3.20 get updated? Will this still happen with this change?

In the Makefile.

You specify KO_DEFAULTBASEIMAGE=$(KO_BASE_IMAGE_PATH)/alpine:3.20, and alpine:3.20 will be used.
The base_image_path is basically the registry from which the image should be pulled, which is docker.io by default, and we can overwrite it.

@Laubi Laubi force-pushed the ci/use-docker-proxy branch from cd4ed66 to ad52fcd Compare February 10, 2025 13:54
This allows us to use a different proxy instead of querying docker directly, potentially breaking the pipeline because of "You have reached your pull rate limit".

By default, it still pulls from docker hub
@Laubi Laubi force-pushed the ci/use-docker-proxy branch from ad52fcd to 874a9bc Compare February 12, 2025 09:40
@Laubi Laubi merged commit adeacdc into main Feb 12, 2025
12 checks passed
@Laubi Laubi deleted the ci/use-docker-proxy branch February 12, 2025 09:52
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.

3 participants