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

podman pod inspect POD is inconsistent with podman inspect IMAGE|CONTAINER #6757

Closed
markstos opened this issue Jun 24, 2020 · 14 comments · Fixed by #8102
Closed

podman pod inspect POD is inconsistent with podman inspect IMAGE|CONTAINER #6757

markstos opened this issue Jun 24, 2020 · 14 comments · Fixed by #8102
Assignees
Labels
Good First Issue This issue would be a good issue for a first time contributor to undertake. In Progress This issue is actively being worked by the assignee, please do not work on this at this time. kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@markstos
Copy link
Contributor

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)

/kind bug

Description

Looking to inspect something? Try podman inspect:

podman inspect IMAGE
podman inspect CONTAINER

Oh, you want to inspect a pod? You might guess that's not supported yet. It's supported, but the command name is inconsistent:

podman pod inspect POD

Steps to reproduce the issue:

  1. Look at podman inspect --help

  2. Try to figure out how to inspect a pod.

Describe the results you received:

No support for inspecting pods was found

Describe the results you expected:

Either podman inspect should support inspecting pods, or if there's a good reason for having the pod inspection feature under podman pod, then at least podman inspect --help should mention the existence of the feature to redirect users who reasonably look for it under podman inspect.

Output of podman version:

podman 2.0

For example, see #6734 where I use was told to give pod inspection output but failed to find it under podman inspect, causing more back-and-forth just to locate the command.

@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Jun 24, 2020
@markstos markstos changed the title podman pod inspect POD is inconsistent with podman inspect POD|CONTAINER podman pod inspect POD is inconsistent with podman inspect IMAGE|CONTAINER Jun 24, 2020
@mheon
Copy link
Member

mheon commented Jun 24, 2020

It looks like we also need to make podman inspect inspect volumes and networks for Docker compatibility.

@rhatdan
Copy link
Member

rhatdan commented Jun 25, 2020

podman container inspect
podman image inspect
podman volume inspect
podman network inspect

Docker was moving away from not using three level of commands, I guess they have moved back

@rhatdan
Copy link
Member

rhatdan commented Jun 25, 2020

The problem with using just podman inspect, is you have a naming conflicts.

@markstos
Copy link
Contributor Author

markstos commented Jun 25, 2020 via email

@rhatdan
Copy link
Member

rhatdan commented Jun 25, 2020

Interested in opening a PR to add this information?

@markstos
Copy link
Contributor Author

@rhatdan #6773

@rhatdan
Copy link
Member

rhatdan commented Jun 29, 2020

Thanks @markstos

@github-actions
Copy link

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

@rhatdan
Copy link
Member

rhatdan commented Jul 30, 2020

@marcosps Since we merged the documentation change should I close this PR? Or should we actually make podman inspect also handle network and volumes to match Docker?

@marcosps
Copy link
Contributor

@rhatdan I believe you wanted to ping @markstos :)

@markstos
Copy link
Contributor Author

markstos commented Jul 30, 2020

@rhatdan Because of the tagline alias docker=podman, if that's still the current vision you should pursue the compatibility.

Personally, I never loved Docker and my company is not invested in it. I'm investigating podman as "post-Docker" tool, so close compatibility is not a personal concern. I do like consistent interfaces and clear docs, though.

@mheon
Copy link
Member

mheon commented Sep 8, 2020

@ashley-cui PTAL

@rhatdan
Copy link
Member

rhatdan commented Oct 5, 2020

The information is there now.

$ podman inspect --help
Display the configuration of object denoted by ID

Description:
  Displays the low-level information on an object identified by name or ID.
  For more inspection options, see:

      podman container inspect
      podman image inspect
      podman network inspect
      podman pod inspect
      podman volume inspect

@rhatdan
Copy link
Member

rhatdan commented Oct 5, 2020

@ashley-cui Are you still working on this?

@rhatdan rhatdan added the Good First Issue This issue would be a good issue for a first time contributor to undertake. label Oct 5, 2020
@ashley-cui ashley-cui added In Progress This issue is actively being worked by the assignee, please do not work on this at this time. and removed stale-issue labels Oct 13, 2020
@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 22, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Good First Issue This issue would be a good issue for a first time contributor to undertake. In Progress This issue is actively being worked by the assignee, please do not work on this at this time. kind/bug Categorizes issue or PR as related to a bug. 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 a pull request may close this issue.

6 participants