Skip to content

start using images from registry.k8s.io#4728

Open
upodroid wants to merge 1 commit intokubernetes-sigs:mainfrom
upodroid:fix-image-registries
Open

start using images from registry.k8s.io#4728
upodroid wants to merge 1 commit intokubernetes-sigs:mainfrom
upodroid:fix-image-registries

Conversation

@upodroid
Copy link
Copy Markdown
Member

@upodroid upodroid commented Apr 3, 2026

What type of PR is this?

/kind bug

What this PR does / why we need it:

All end-user facing images need to be served from registry.k8s.io, you can't serve any images hosted at gcr.io/k8-staging-* or us-central1-docker.pkg.dev/k8s-staging-images/* to end users. The project has a policy for serving docker images to end users here.

Unfortunately, we got hit with a large bill, and we need to shut down gcr.io/k8s-staging-gateway-api as soon as possible. To do that:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/bug Categorizes issue or PR as related to a bug. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 3, 2026
@k8s-ci-robot k8s-ci-robot requested review from candita and robscott April 3, 2026 13:10
@k8s-ci-robot k8s-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Apr 3, 2026
requests:
cpu: 10m
- name: tls-backend
image: registry.k8s.io/gateway-api/echo-basic:v1.5.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

$ docker pull registry.k8s.io/gateway-api/echo-basic:v1.5.1
Error response from daemon: manifest for registry.k8s.io/gateway-api/echo-basic:v1.5.1 not found: manifest unknown: Failed to fetch "v1.5.1"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I opened a PR promoting these missing images

- --crt=/cert.crt
- --key=/cert.key
- name: echo
image: registry.k8s.io/gateway-api/echo-advanced:v1.5.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

docker pull registry.k8s.io/gateway-api/echo-advanced:v1.5.1
Error response from daemon: manifest for registry.k8s.io/gateway-api/echo-advanced:v1.5.1 not found: manifest unknown: Failed to fetch "v1.5.1"

@rikatz
Copy link
Copy Markdown
Member

rikatz commented Apr 6, 2026

@upodroid
Copy link
Copy Markdown
Member Author

upodroid commented Apr 6, 2026

Before I backfill the images, I asked this question on Slack

whats the difference between echo-basic and echo-advanced? Also, echo-advanced is a copy of the istio app image which hasn't been updated in a long time

https://kubernetes.slack.com/archives/CR0H13KGA/p1775222564804039?thread_ts=1775079862.624539&cid=CR0H13KGA

@youngnick
Copy link
Copy Markdown
Contributor

echo-basic is for the north-south tests, while echo-advanced is for east-west mesh cases. You're correct that's it's a copy of the istio app image, the reason to do that was to move the the test artifact under Gateway API's control, rather than being under Istio's control. That way, the Gateway API conformance tests are not dependent on another organization's binary.

allowedRoutes:
namespaces:
from: Same
- name: http
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm wondering why the whitespace changes weren't caught by the linter. Could you fix them anyway?

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: upodroid
Once this PR has been reviewed and has the lgtm label, please assign danwinship for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. release-note-none Denotes a PR that doesn't merit a release note. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants