please add --log-driver for podman exec

[allisonkarlitskaya]

As per #6555 (comment)

For now, I'm going to force log-driver off for podman exec; we can add the flag at a later date

The issue got closed with the hard-wiring of "none" but there was no follow-up to add the flag, as planned.

[Luap99]

While adding the log options is easy, I am not sure how you want to read the logs then?

@mheon PTAL

[mheon]

We can add logging without that much difficulty - just a bit of plumbing required to get the appropriate flags to Conmon. Question is really how do you read those logs, as podman logs isn't going to do it.

[allisonkarlitskaya]

I actually just want to set the driver to passthrough.

[Luap99]

What would be the advantage of the passthrough driver compared to the normal podman exec?

[allisonkarlitskaya]

I want my file descriptors going through to the inside, without podman acting as an intermediary.

[rhatdan]

Does this work for you?

man podman exec
...
   --preserve-fds=N
       Pass down to the process N additional file descriptors (in addition to 0, 1, 2).  The total FDs will be 3+N.

[allisonkarlitskaya]

Does this work for you?

man podman exec
...
   --preserve-fds=N
       Pass down to the process N additional file descriptors (in addition to 0, 1, 2).  The total FDs will be 3+N.

Yes. That's what I'm using now. 😅

It actually works fine. It just seemed odd to me that it was necessary when there's the passthrough option on run but not exec. And then I found the issue saying that there was an eventual intent to add the option for exec as well, so I opened the issue.

[Luap99]

I think adding passthrough support is fine, journald could also be added since users could read the logs via journalctl.

But keep in mind that passthrough is only supported when the fds are not connected to a terminal.

[allisonkarlitskaya]

I'd be using it with an AF_UNIX socketpair.

[rhatdan]

Interested in opening a PR to make this happen?

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

@flouthoc Interested in taking a look at this?

[flouthoc]

Looks a small change ,can add this flag but I am not sure if this is even easily testable in CI or not @mheon WDYT ?

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

@flouthoc Reminding