Skip to content

Rename all references to kpod to podman#145

Closed
rhatdan wants to merge 2 commits intocontainers:masterfrom
rhatdan:podman
Closed

Rename all references to kpod to podman#145
rhatdan wants to merge 2 commits intocontainers:masterfrom
rhatdan:podman

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.

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 <dwalsh@redhat.com>
@umohnani8
Copy link
Member

LGTM

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

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@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 <dwalsh@redhat.com>

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