Closed dimitar-kostadinov closed 3 months ago
/assign
LGTM label has been added.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ialidzhikov
The full list of commands accepted by this bot can be found here.
The pull request process is described here
How to categorize this PR?
/area testing /kind test
What this PR does / why we need it: Rework the image pull check in E2E tests. The image pull check now use root pod executor and runs the following commands on the Node where the registry Pod is running:
jq -j '.mounts[] | select(.destination=="/var/lib/registry") | .source' /run/containerd/io.containerd.runtime.v2.task/k8s.io/<registry-container-id>/config.json
to get the location of '/var/lib/registry' mount on the Node.cat <mount-location>/docker/registry/v2/repositories/<image-path>/_manifests/tags/<image-tag>/current/link
to get the image index blob location.jq -j '.manifests | length' %s/docker/registry/<image-index-blob-path>/blobs/%s/data
to verify it is valid image index.This will allow
distroless
to be used as the base registry image and will solve E2E flakes caused by the missingscheduler-state.json
file.Which issue(s) this PR fixes: Part of #3
Special notes for your reviewer: N/A
Release note: