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

Rename all references to kpod to podman #145

Closed
wants to merge 2 commits into from

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented Dec 15, 2017

No description provided.

@rhatdan
Copy link
Member Author

rhatdan commented Dec 15, 2017

@TomSweeneyRedHat
Copy link
Member

LGTM assuming happy tests. Hard to tell if any were missed. I say press the merge button while looking between your fingers. I'd also suggest holding the merge to Monday morn. Murphy lets the chances go up exponentially for problems from PR's like this on a Friday night.

| [kpod-unpause(1)](/docs/kpod-unpause.1.md) | Unpause one or more running containers |[![...](/docs/play.png)](https://asciinema.org/a/141292)|
| [kpod-version(1)](/docs/kpod-version.1.md) | Display the version information |[![...](/docs/play.png)](https://asciinema.org/a/mfrn61pjZT9Fc8L4NbfdSqfgu)|
| [kpod-wait(1)](/docs/kpod-wait.1.md) | Wait on one or more containers to stop and print their exit codes||
| [podman(1)](/docs/podman.1.md) | Simple management tool for pods and images ||
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to make changes in vendor/
We should get a PR into CRI-O to fix readme and transfer there and revendor to remove the last few mentions

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops.
Fixed

@mheon
Copy link
Member

mheon commented Dec 16, 2017

One extra file in vendor got added otherwise LGTM

@rhatdan rhatdan force-pushed the podman branch 4 times, most recently from e72819f to 7951f88 Compare December 18, 2017 14:28
Remove or comment out unused functions.

Signed-off-by: Daniel J Walsh <[email protected]>
@umohnani8
Copy link
Member

LGTM

The decision is in, kpod is going to be named podman.

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan
Copy link
Member Author

rhatdan commented Dec 18, 2017

@rh-atomic-bot r=umohnani8

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 1e1e998 has been approved by umohnani8

@rh-atomic-bot
Copy link
Collaborator

⚡ Test exempted: pull fully rebased and already tested.

rh-atomic-bot pushed a commit that referenced this pull request Dec 18, 2017
The decision is in, kpod is going to be named podman.

Signed-off-by: Daniel J Walsh <[email protected]>

Closes: #145
Approved by: umohnani8
afbjorklund added a commit to afbjorklund/minikube that referenced this pull request Jul 29, 2018
Simple management tool for containers and images

"podman shares state with crio and so has the
capability to debug pods/images created by crio"

containers/podman#145
afbjorklund added a commit to afbjorklund/minikube that referenced this pull request Jul 30, 2018
Simple management tool for containers and images

"podman shares state with crio and so has the
capability to debug pods/images created by crio"

containers/podman#145
dlorenc pushed a commit to kubernetes/minikube that referenced this pull request Jul 30, 2018
Simple management tool for containers and images

"podman shares state with crio and so has the
capability to debug pods/images created by crio"

containers/podman#145
baude pushed a commit to baude/podman that referenced this pull request Aug 31, 2019
Pass interface name and MAC back to runtime
@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 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 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.

5 participants