Skip to content

[release-1.21] fix: workaround podman bug wrt error reporting#3478

Merged
knative-prow[bot] merged 1 commit intoknative:release-1.21from
knative-prow-robot:cherry-pick-3477-to-release-1.21
Mar 9, 2026
Merged

[release-1.21] fix: workaround podman bug wrt error reporting#3478
knative-prow[bot] merged 1 commit intoknative:release-1.21from
knative-prow-robot:cherry-pick-3477-to-release-1.21

Conversation

@knative-prow-robot
Copy link
Copy Markdown

This is an automated cherry-pick of #3477

fix: cannot push to cluster internal registry with podman (error API returned a 500 (Internal Server Error) but provided no error-message)

In case that the image registry hostname is not resolvable podman now
return error in bad JSON shape so it's not decodable by client.

For now we assume that '500.*but provided no error-message' means
hostname resolution error.

Signed-off-by: Matej Vašek <matejvasek@gmail.com>
@knative-prow knative-prow Bot requested review from dsimansk and jrangelramos March 9, 2026 14:19
@knative-prow knative-prow Bot added the size/XS 🤖 PR changes 0-9 lines, ignoring generated files. label Mar 9, 2026
@matejvasek
Copy link
Copy Markdown
Contributor

/approve
/lgtm

@knative-prow knative-prow Bot added the lgtm 🤖 PR is ready to be merged. label Mar 9, 2026
@knative-prow
Copy link
Copy Markdown

knative-prow Bot commented Mar 9, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knative-prow-robot, matejvasek

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

The pull request process is described 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

@knative-prow knative-prow Bot added the approved 🤖 PR has been approved by an approver from all required OWNERS files. label Mar 9, 2026
@knative-prow knative-prow Bot merged commit 3f9df74 into knative:release-1.21 Mar 9, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 🤖 PR has been approved by an approver from all required OWNERS files. lgtm 🤖 PR is ready to be merged. size/XS 🤖 PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants