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: output of ps with a conatiner publishing 1000 ports is sub-optimal #1358

Closed
jfilak opened this issue Aug 28, 2018 · 13 comments
Closed
Assignees
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@jfilak
Copy link

jfilak commented Aug 28, 2018

Is this a BUG REPORT or FEATURE REQUEST?:

/kind feature

Description
If I publish many ports, the output of docker ps gets unreadable.

Steps to reproduce the issue:

  1. podman run -it -p 1000-2000:1000-2000 --rm --name manyports centos /bin/bash

  2. podman ps

Describe the results you received:

The output contains information about every port for both protocols.

Describe the results you expected:
I guess the same information which is used to start the container would be enough - 1000-2000:1000-2000.

Additional information you deem important (e.g. issue happens only occasionally):

issue happens always

Output of podman version:

Version:       0.8.3
Go Version:    go1.10.3
OS/Arch:       linux/amd64

Output of podman info:

host:
  Conmon:
    package: podman-0.8.3-1.git9d09a4d.fc28.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 1.12.0-dev, commit: c72f69ce3f3fd7e46dced78dd99b67ab5441441e-dirty'
  MemFree: 12090945536
  MemTotal: 16547180544
  OCIRuntime:
    package: runc-1.0.0-50.dev.git20aff4f.fc28.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.0'
  SwapFree: 8346660864
  SwapTotal: 8346660864
  arch: amd64
  cpus: 4
  hostname: brql33992173a
  kernel: 4.17.18-200.fc28.x86_64
  os: linux
  uptime: 34m 5.58s
insecure registries:
  registries: []
registries:
  registries:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ContainerStore:
    number: 1
  GraphDriverName: overlay
  GraphOptions:
  - overlay.mountopt=nodev
  - overlay.override_kernel_check=true
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
  ImageStore:
    number: 1
  RunRoot: /var/run/containers/storage

Additional environment details (AWS, VirtualBox, physical, etc.):

physical Fedora 28

@rhatdan
Copy link
Member

rhatdan commented Aug 28, 2018

@baude PTAL

@mheon mheon added the bug label Aug 29, 2018
@vrothberg
Copy link
Member

vrothberg commented Nov 19, 2018

This still seems to be an issue. I am going fifo through our issue, sorry to be noisy today. @baude, do you find time to work on this or shall we re-assign?

@mheon
Copy link
Member

mheon commented Nov 19, 2018 via email

@rhatdan
Copy link
Member

rhatdan commented Dec 22, 2018

@mheon Are you still working on this?

@mheon
Copy link
Member

mheon commented Dec 22, 2018

@rhatdan Yes, hope to get to it sometime during January

@rhatdan
Copy link
Member

rhatdan commented Mar 8, 2019

@mheon Any updates?

@mheon
Copy link
Member

mheon commented Mar 8, 2019

This is a solid week of work to get right due to the need for DB changes. I'd like to get it resolved, but it's lower priority than... most other things?

@kunalkushwaha
Copy link
Collaborator

kunalkushwaha commented Mar 15, 2019

This need parsing ports and formatting like

<snip>   centos    "/bin/bash"  <snip>    0.0.0.0:1000-2000->1000-2000/tcp   manyports

basically group ports in function portsToString , when list of continuous ports are mapped.

@mheon, I can work on this, If you are busy.

kunalkushwaha added a commit to kunalkushwaha/libpod that referenced this issue Mar 15, 2019
continuous published ports are grouped in ps output.
bugfix: containers#1358

Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
@rhatdan
Copy link
Member

rhatdan commented Mar 15, 2019

Awesome @kunalkushwaha Thanks

@kunalkushwaha
Copy link
Collaborator

@rhatdan Thank you for the invitation :)

@rhatdan
Copy link
Member

rhatdan commented Mar 15, 2019

Were glad to add you to the contributors.

@rhatdan
Copy link
Member

rhatdan commented Mar 15, 2019

And now I can assign you to issues. :^)

@kunalkushwaha
Copy link
Collaborator

This issue is fixed with #2664 & #2685.

Hence closing the issue.

muayyad-alsadi pushed a commit to muayyad-alsadi/libpod that referenced this issue Apr 21, 2019
continuous published ports are grouped in ps output.
bugfix: containers#1358

Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
@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 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 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

No branches or pull requests

6 participants