fix podman build -o -

[Napuu]

Ensure quiet flag is true when building directly to stdout.

Fixes #23063

Does this PR introduce a user-facing change?

Fixed podman build -o - to prevent status messages from appearing in stdout.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Napuu
Once this PR has been reviewed and has the lgtm label, please assign lsm5 for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[packit-as-a-service]

Ephemeral COPR build failed. @containers/packit-build please check.

[rhatdan]

This looks like it should be handled in the build options CLI?

https:/containers/buildah/blob/main/pkg/cli/build.go#L267-L275

[edsantiago]

Thank you for your PR! LGTM on first (quick) pass, but one blocker.

[lsm5]

The failed packit jobs for epel-9 and fedora-39 will go away after a rebase on main.

[edsantiago]

Test LGTM. Did you look into Dan's question? I think we should try to understand why the pkg/cli code is not having the desired effect, and fix it there. Would you be able to investigate that?

Also, for when you repush, these might be two good additions to your test:

		Expect(output).ShouldNot(ContainSubstring("STEP "))
		Expect(output).ShouldNot(ContainSubstring("COPY "))

[Napuu]

The mentioned code in Buildah isn't having the desired effect because that function is not used by Podman. A somewhat similar functionality is implemented in the function where I made my changes. Seems like it's already acknowledged that it shouldn't really be on Podman's side: https:/containers/podman/blob/main/cmd/podman/common/build.go#L270

I think that would need a bigger refactoring to get rid of that duplicated part altogether.

[rhatdan]

Can you change your PR to use said code in Buildah?

[Napuu]

Did some initial digging, and it doesn't seem as trivial as my previous change, so not for now at least.

I'll use buildah build -o - to achieve this for the time being. Might have time to investigate more in the future though.

[openshift-merge-robot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.