Skip to content

Revert "storage: improve error message on image mount from r/o store"#965

Merged
Luap99 merged 1 commit into
podman-container-tools:mainfrom
Luap99:revert
Jul 7, 2026
Merged

Revert "storage: improve error message on image mount from r/o store"#965
Luap99 merged 1 commit into
podman-container-tools:mainfrom
Luap99:revert

Conversation

@Luap99

@Luap99 Luap99 commented Jul 7, 2026

Copy link
Copy Markdown
Member

This reverts commit 006839b.

This is causes test failures in both podman and buildah tests[1]. The return error was changed so buildah can no longer match it.

For the next podman release we need to vendor some pasta changes so we like a working container-libs. This can be reapplied once podman/buildah are known to work with this.

#945 (comment)

This reverts commit 006839b.

This is causes test failures in both podman and buildah tests[1]. The
return error was changed so buildah can no longer match it.

For the next podman release we need to vendor some pasta changes so we
like a working container-libs. This can be reapplied once podman/buildah
are known to work with this.

podman-container-tools#945 (comment)

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@github-actions github-actions Bot added the storage Related to "storage" package label Jul 7, 2026
@Luap99

Luap99 commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

cc @mtrmac @comick @Honny1 @baude

@Honny1 Honny1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Luap99

Luap99 commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

merging to unblock the podman side

@Luap99 Luap99 merged commit d126613 into podman-container-tools:main Jul 7, 2026
32 of 35 checks passed
@Luap99 Luap99 deleted the revert branch July 7, 2026 15:22

@mtrmac mtrmac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, for the record. Thanks!

@comick

comick commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

LGTM indeed. Did some analysis at #945 (comment)

@Honny1

Honny1 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

We successfully released podman 6.0.1. I think we can revert this and create PRs for Buildah with vendoring of the change and fix. How do we proceed with this?

@mtrmac

mtrmac commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@Honny1 I think a draft PR here + test PRs vendoring into Buildah + Podman would be best; after the 3 PRs pass review and tests, quickly merge all three.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

storage Related to "storage" package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants