Skip to content

removing duplicate ports in docker create parameters#45439

Open
Jeyanthinath wants to merge 1 commit intomoby:masterfrom
Jeyanthinath:duplicate_port_fix
Open

removing duplicate ports in docker create parameters#45439
Jeyanthinath wants to merge 1 commit intomoby:masterfrom
Jeyanthinath:duplicate_port_fix

Conversation

@Jeyanthinath
Copy link
Copy Markdown
Contributor

Fixes #45199

Created a function, that de-duplicate the port and send a warning message in the api response.

@thaJeztah
Copy link
Copy Markdown
Member

^ there's a typo in the commit message (parmaeters)

@thaJeztah thaJeztah changed the title removing duplicate ports in docker create parmaeters removing duplicate ports in docker create parameters Apr 30, 2023
@Jeyanthinath Jeyanthinath force-pushed the duplicate_port_fix branch 2 times, most recently from 44369db to 930d6b1 Compare June 18, 2023 11:04
@Jeyanthinath
Copy link
Copy Markdown
Contributor Author

@thaJeztah can you please validate this PR, only failure test case [docker-py] (might be stuck)

ERROR: failed to solve: process "/bin/sh -c sed -ri \"s/(httpredir|deb).debian.org/${APT_MIRROR:-deb.debian.org}/g\" /etc/apt/sources.list     && sed -ri \"s/(security).debian.org/${APT_MIRROR:-security.debian.org}/g\" /etc/apt/sources.list" did not complete successfully: exit code: 2
Error: The operation was canceled

Signed-off-by: Jeyanthinath Muthuram <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the duplicate_port_fix branch from 412cca6 to 7f2a2a4 Compare March 30, 2026 12:57
@github-actions github-actions Bot added the area/daemon Core Engine label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API should either reject, de-duplicate, or warn duplicate exposed-ports

2 participants