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 info should include info about Pasta #18561

Closed
PhrozenByte opened this issue May 14, 2023 · 4 comments · Fixed by #18640
Closed

podman info should include info about Pasta #18561

PhrozenByte opened this issue May 14, 2023 · 4 comments · Fixed by #18640
Assignees
Labels
Good First Issue This issue would be a good issue for a first time contributor to undertake. kind/feature Categorizes issue or PR as related to a new feature. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. pasta pasta(1) bugs or features

Comments

@PhrozenByte
Copy link
Contributor

PhrozenByte commented May 14, 2023

Feature request description

podman info is supposed to display Podman related system information. This includes information about networking. For this reason podman info also includes a section about e.g. slirp4netns, looking like the following:

host:
  slirp4netns:
    executable: /usr/bin/slirp4netns
    package: slirp4netns-1.2.0-12.fc38.x86_64
    version: |-
      slirp4netns version 1.2.0
      commit: 656041d45cfca7a4176f6b7eed9e4fe6c11e8383
      libslirp: 4.7.0
      SLIRP_CONFIG_VERSION_MAX: 4
      libseccomp: 2.5.3

Right now there's no similar info about Pasta, Podman's new and promising rootless networking option. I suggest adding such section.

The use case is the same as for slirp4netns: To check whether Pasta is installed, whether it's usable and the exact version being used (one could run Pasta with an arbitrary version).

Suggest potential solution

podman info should also include info about Pasta, e.g. like the following (feel free to add other related info):

host:
  pasta:
    executable: /usr/bin/pasta
    package: passt-0^20230509.g96f8d55-1.fc38.x86_64
    version: |-
      passt version 2023_05_09.96f8d55-10-gf1e0a67

Have you considered any alternatives?

n/a

Additional context

n/a

@PhrozenByte PhrozenByte added the kind/feature Categorizes issue or PR as related to a new feature. label May 14, 2023
@Luap99 Luap99 added pasta pasta(1) bugs or features Good First Issue This issue would be a good issue for a first time contributor to undertake. labels May 15, 2023
@Luap99
Copy link
Member

Luap99 commented May 15, 2023

Sounds good to me, interested in fixing this yourself?

@PhrozenByte
Copy link
Contributor Author

Even though I'd love to start with a new programming language (never worked with Go before), there's too much stuff on the list right now 🙈

@HirazawaUi
Copy link
Contributor

I want work for this, if you don't mind, i will finish this issue.
/assign

@rhatdan
Copy link
Member

rhatdan commented May 18, 2023

You got it.

HirazawaUi added a commit to HirazawaUi/podman that referenced this issue May 24, 2023
[NO NEW TESTS NEEDED]

Fixes: containers#18561

Signed-off-by: binghongtao <[email protected]>
@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 Aug 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 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. kind/feature Categorizes issue or PR as related to a new feature. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. pasta pasta(1) bugs or features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants