Update crictl version to v1.31.1

[luolanzone]

No description provided.

[antoninbas]

I have noticed this warning, however this is not a great approach to tackling it:

• IMO, we shouldn't add a tag that doesn't exist (such as latest for antrea/base-ubuntu)
• having a specific version as the tag (e.g., ARG GO_VERSION=1.23) defeats the purpose of having one source of truth for the version, and means a larger diff when bumping up the dependency
• when building and forgetting the tag (e.g., from a script), it's better to have an error (missing tag) rather than build with the wrong "default" tag. This type of issue can create surprising behaviors and be hard to troubleshoot.

#6486 should reduce the amount of warnings. We could do something similar for the golang image and use a build context. Otherwise, I would suggest silencing that specific warning for now.

[luolanzone]

Hi @antoninbas I have reverted the change and made some cleanup only, PTAL, thanks.

[luolanzone]

Ah, I just noticed that you already removed unused args in the PR #6486, thanks for that. I think we still need to update crictl's version to 1.31.1 but it's not critical, do you want to include the change in your PR or keep in this PR?

[antoninbas]

Ah, I just noticed that you already removed unused args in the PR #6486, thanks for that. I think we still need to update crictl's version to 1.31.1 but it's not critical, do you want to include the change in your PR or keep in this PR?

I think we can have a dedicated PR to update the crictl version