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

If cidfile exists, do not proceed #531

Closed
wants to merge 1 commit into from

Conversation

baude
Copy link
Member

@baude baude commented Mar 22, 2018

Both podman run and create have an option to write the container ID to a file. The option
is called cidfile. If the cidfile exists, we should not create or run a container but rather
output a sensical error message.

Resolves: #530

Signed-off-by: baude [email protected]

Both podman run and create have an option to write the container ID to a file. The option
is called cidfile.  If the cidfile exists, we should not create or run a container but rather
output a sensical error message.

Resolves: containers#530

Signed-off-by: baude <[email protected]>
@mheon
Copy link
Member

mheon commented Mar 22, 2018

LGTM

@rhatdan
Copy link
Member

rhatdan commented Mar 23, 2018

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 900f9f0 has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit 900f9f0 with merge c55e371...

@rh-atomic-bot
Copy link
Collaborator

☀️ Test successful - status-papr
Approved by: rhatdan
Pushing c55e371 to master...

@baude baude deleted the cidfilecheck branch December 22, 2019 18:45
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 26, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants