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

ignore proxy imagees with digests when checking version #12059

Merged
merged 1 commit into from Feb 15, 2024
Merged

Conversation

adleong
Copy link
Member

@adleong adleong commented Feb 10, 2024

Fixes #12058

When proxy images are specified by digest rather than by tag, linkerd check will erroneously assume that the digest is a tag and attempt to compare it to the current Linkerd version.

Instead, we ignore images with digests since there isn't an easy way to determine what version a digest corresponds to.

Signed-off-by: Alex Leong <alex@buoyant.io>
@adleong adleong requested a review from a team as a code owner February 10, 2024 00:11
Copy link
Member

@mateiidavid mateiidavid left a comment

Choose a reason for hiding this comment

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

LGTM. Seems like a good way to do it due to the reasons you've mentioned in the description.

Copy link
Member

@alpeb alpeb left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@adleong adleong merged commit d8be3e0 into main Feb 15, 2024
33 checks passed
@adleong adleong deleted the alex/indigestion branch February 15, 2024 19:02
@adleong adleong mentioned this pull request Feb 15, 2024
adleong added a commit that referenced this pull request Feb 16, 2024
* Allowed the `MutatingWebhookConfig` timeout value to be configured ([#12028])
  (thanks @mikebell90)
* Added a counter for items dropped from destination controller workqueue
  ([#12079])
* Fixed a spurious `linkerd check` error when using container images with
  digests ([#12059])
* Fixed an issue where inbound policy could be incorrect after certain policy
  resources are deleted ([#12088])

[#12028]: #12028
[#12079]: #12079
[#12059]: #12059
[#12088]: #12088

Signed-off-by: Alex Leong <alex@buoyant.io>
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.

Linkerd check displays incorrect image versions.
3 participants