Skip to content

Return image ID in PullImageResponse.image_ref to stay compatible with future k8s versions#549

Merged
awmirantis merged 2 commits into
Mirantis:masterfrom
stlaz:cri-image-ref
May 19, 2026
Merged

Return image ID in PullImageResponse.image_ref to stay compatible with future k8s versions#549
awmirantis merged 2 commits into
Mirantis:masterfrom
stlaz:cri-image-ref

Conversation

@stlaz
Copy link
Copy Markdown
Contributor

@stlaz stlaz commented Mar 12, 2026

Related to #546 and kubernetes/kubernetes#137024

Proposed Changes

  • never return the digest in PullImageResponse.image_ref
  • fix logging in core/stats_windows.go (unrelated, can be removed if requested)
  • bump cri-tools to current master instead of roughly 3 years old ref in integration GH action

@stlaz
Copy link
Copy Markdown
Contributor Author

stlaz commented Mar 31, 2026

@athongsamai you seem to have been the latest person active in the repository as far as I can tell. If you could please review the PR or find somebody to do it, that'd be great.

I posted more context into the #cri-dockerd channel on k8s Slack if needed.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Comment thread .github/workflows/integration.yml Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@stlaz stlaz force-pushed the cri-image-ref branch 2 times, most recently from 4278f1d to 472122c Compare May 18, 2026 07:33
@stlaz
Copy link
Copy Markdown
Contributor Author

stlaz commented May 18, 2026

@awmirantis please run the CI.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread core/stats_windows.go Outdated
Comment thread core/image.go
Comment thread core/image_test.go
Comment thread libdocker/fake_client.go Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Comment thread libdocker/fake_client.go
Comment thread libdocker/fake_client.go
Comment thread libdocker/fake_client.go Outdated
stlaz added 2 commits May 18, 2026 10:34
This is required to stay compatible with the latest CRI API
requirements.

Signed-off-by: Stanislav Láznička <slznika@microsoft.com>
Signed-off-by: Stanislav Láznička <slznika@microsoft.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread libdocker/fake_client.go
Copy link
Copy Markdown
Contributor

@awmirantis awmirantis left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks

@stlaz
Copy link
Copy Markdown
Contributor Author

stlaz commented May 19, 2026

@awmirantis thank you 🙇 I see I need a review from at least one other person with write access, would you be able to fetch anyone?

If we manage to merge this, when could we expect a release with the fix? I would include it in kubernetes/website#54879

Copy link
Copy Markdown
Contributor

@athongsamai athongsamai left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@awmirantis awmirantis merged commit 83d711f into Mirantis:master May 19, 2026
16 of 17 checks passed
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