podman-py icon indicating copy to clipboard operation
podman-py copied to clipboard

demux doesn't work in exec_run => no way to separate stdout and stderr

Open claraphyll opened this issue 2 years ago • 1 comments

Hello,

It seems like demux in Container.exec_run doesn't work, and that there is currently no way to separate stdout and stderr when using exec through this library. Quickly looking at the code, it seems to me like the demux parameter is only referenced in the doc comment, but not used in the code: https://github.com/containers/podman-py/blob/004ebda439975fdf81d87e84ae1387cf1aa0d3fe/podman/domain/containers.py#L142C1-L142C14

Thanks for considering this issue

claraphyll avatar Sep 05 '23 13:09 claraphyll

ack. not just that, stream parameter is also unused there. I am taking a look

inknos avatar Jun 03 '24 16:06 inknos

@inknos Do you have any further update on fixing this issue ?

omkarkhatavkar avatar Jul 25 '24 13:07 omkarkhatavkar

@maxi0604 If you want to take the implementation I did for demuxing, you can see it here: https://github.com/SatelliteQE/broker/blob/master/broker/binds/containers.py#L8-L48

JacobCallahan avatar Jul 25 '24 18:07 JacobCallahan

Do you have any further update on fixing this issue ?

I am sorry I got distracted by other duties and forgot after PTO. will move it on the higher priority now.

inknos avatar Jul 26 '24 14:07 inknos