Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

--cidfile support for podman exec #21256

Open
findesgh opened this issue Jan 14, 2024 · 4 comments
Open

--cidfile support for podman exec #21256

findesgh opened this issue Jan 14, 2024 · 4 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature. stale-issue

Comments

@findesgh
Copy link

Feature request description

I've had some situations, where being able to use the --cidfile option with podman exec would've been convenient. As far as I can tell, however, that is not possible, which strikes me as somewhat odd, since, e.g. stop, kill, rm and pause/unpause all support it (I just checked and attach apparently also does not support it).

Suggest potential solution

No response

Have you considered any alternatives?

Workarounds for this exist and are quite straight-forward. One can name a container or use external tools to extract an ID from a cid-file, i.e. this is certainly not a high priority issue.

Additional context

From a UX point of view one might argue that all options to identify a container to a command should be supported by all commands that take a container as argument so one might extend this issue to include attach (and possibly others I've overlooked?).

@findesgh findesgh added the kind/feature Categorizes issue or PR as related to a new feature. label Jan 14, 2024
@vrothberg
Copy link
Member

Thanks for reaching out, @findesgh ! This looks like a nice improvement. Are you interested in opening a PR?

@findesgh
Copy link
Author

@vrothberg: I know virtually zero Go 😅. A cursory look at the code suggests that this shouldn't be too hard, however, so might be a good opportunity to learn something.
I'll make a stab at it, we'll see how it goes.

@vrothberg
Copy link
Member

Awesome, thanks! Feel free to ask questions here :) You can take a look at the how the other commands implement --cidfile. The commit would also need a test but we can collaborate on that.

Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. stale-issue
Projects
None yet
Development

No branches or pull requests

2 participants