Fix push endpoint stream

[umohnani8]

The push binding endpoint wasn't actually writing the output data to the stream when quiet=false and there was no push error.
Do not hard code quiet=true anymore, take into account the user input.

Does this PR introduce a user-facing change?

Fix a bug in the push binding endpoint where the output wasn't being added to the stream when the push was successful. 

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: umohnani8

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [umohnani8]

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

[umohnani8]

I am not sure if there is a way to add tests for this, it does look like there is a test for this here https://github.com/containers/podman/blob/main/test/apiv2/12-imagesMore.at#L36 but it doesn't work as expected? @edsantiago any idea if there is a good way to test this?

[Luap99]

I am not sure if there is a way to add tests for this, it does look like there is a test for this here https://github.com/containers/podman/blob/main/test/apiv2/12-imagesMore.at#L36 but it doesn't work as expected? @edsantiago any idea if there is a good way to test this?

Looks like the test is calling the compat endpoint and not the libpod one, so you need to add another which calls t POST libpod/images/...

[Luap99]

code changes itself LGTM, just two comments for the test

[edsantiago]

Even with slashes removed, tests fail. @umohnani8 you can run tests locally with:

$ ./test/apiv2/test-apiv2 12 

[umohnani8]

Thanks @edsantiago, tests should be happy now!

[rhatdan]

LGTM
@Luap99 @mheon @baude @lsm5 @ashley-cui PTAL

[mheon]

/lgtm