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

feat: support port ranges when starting a container (#3204) #3654

Merged
merged 3 commits into from Aug 25, 2023

Conversation

lstocchi
Copy link
Contributor

@lstocchi lstocchi commented Aug 23, 2023

What does this PR do?

it adds the logic to handle port ranges in the form of -

Screenshot/screencast of this PR

range_port

What issues does this PR fix or reference?

it resolves #3204

How to test this PR?

  1. start a container by mapping a range of ports and check that it actually worked by inspecting it

@lstocchi lstocchi marked this pull request as ready for review August 24, 2023 09:25
@lstocchi lstocchi requested review from benoitf and a team as code owners August 24, 2023 09:25
@lstocchi lstocchi requested review from dgolovin and cdrage and removed request for a team August 24, 2023 09:25
Signed-off-by: lstocchi <lstocchi@redhat.com>
Signed-off-by: lstocchi <lstocchi@redhat.com>
Signed-off-by: lstocchi <lstocchi@redhat.com>
@lstocchi lstocchi merged commit 7dbde47 into containers:main Aug 25, 2023
8 checks passed
@lstocchi lstocchi deleted the i3204 branch August 25, 2023 07:51
@podman-desktop-bot podman-desktop-bot added this to the 1.4.0 milestone Aug 25, 2023
mairin pushed a commit to mairin/podman-desktop that referenced this pull request Aug 29, 2023
…containers#3654)

* feat: support port ranges when starting a container (containers#3204)

Signed-off-by: lstocchi <lstocchi@redhat.com>

* fix: add tests

Signed-off-by: lstocchi <lstocchi@redhat.com>

* fix: update test

Signed-off-by: lstocchi <lstocchi@redhat.com>

---------

Signed-off-by: lstocchi <lstocchi@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding support for range port mappings
3 participants