Skip to content

[18.09 BACKPORT] Pick a random host port if the user does not specify a host port.#2369

Merged
mavenugo merged 1 commit intomoby:bump_18.09from
mavenugo:win
Apr 23, 2019
Merged

[18.09 BACKPORT] Pick a random host port if the user does not specify a host port.#2369
mavenugo merged 1 commit intomoby:bump_18.09from
mavenugo:win

Conversation

@mavenugo
Copy link
Contributor

For overlay, l2bridge, and l2tunnel, if the user does not specify a host port, windows driver will select a random port for them. This matches linux behavior.
For ics and nat networks the windows OS will choose the port.

Signed-off-by: Pradip Dhara [email protected]
Signed-off-by: Madhu Venugopal [email protected]

For overlay, l2bridge, and l2tunnel, if the user does not specify a host port, windows driver will select a random port for them.  This matches linux behavior.
For ics and nat networks the windows OS will choose the port.

Signed-off-by: Pradip Dhara <[email protected]>
Signed-off-by: Madhu Venugopal <[email protected]>
@thaJeztah
Copy link
Member

Can you change the title to use [18.09 backport] ? (makes it slightly easier to track)

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (not a maintainer)

Copy link
Contributor

@selansen selansen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mavenugo mavenugo changed the title [BACKPORT] Pick a random host port if the user does not specify a host port. [18.09 BACKPORT] Pick a random host port if the user does not specify a host port. Apr 23, 2019
@mavenugo mavenugo merged commit 872f0a8 into moby:bump_18.09 Apr 23, 2019
@thaJeztah
Copy link
Member

backport of #2368

thaJeztah added a commit to thaJeztah/docker that referenced this pull request Apr 23, 2019
full diff: moby/libnetwork@c902989...872f0a8

- moby/libnetwork#2354 [18.09 backport] Cleanup the cluster provider when the agent is closed
  - backport of moby/libnetwork#2307 Fix for problem where agent is stopped and does not restart
  - fixes docker/for-linux#495 Docker swarm overlay networking not working after --force-new-cluster
- moby/libnetwork#2369 [18.09 BACKPORT] Pick a random host port if the user does not specify a host port
  - backport of moby/libnetwork#2368 (windows) Pick a random host port if the user does not specify a host port

Signed-off-by: Sebastiaan van Stijn <[email protected]>
docker-jenkins pushed a commit to docker-archive/docker-ce that referenced this pull request Apr 23, 2019
full diff: moby/libnetwork@c902989...872f0a8

- moby/libnetwork#2354 [18.09 backport] Cleanup the cluster provider when the agent is closed
  - backport of moby/libnetwork#2307 Fix for problem where agent is stopped and does not restart
  - fixes docker/for-linux#495 Docker swarm overlay networking not working after --force-new-cluster
- moby/libnetwork#2369 [18.09 BACKPORT] Pick a random host port if the user does not specify a host port
  - backport of moby/libnetwork#2368 (windows) Pick a random host port if the user does not specify a host port

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Upstream-commit: 5354408039681020f9ad6afe4bf696fc90f9ce69
Component: engine
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request May 7, 2019
Additional post-18.09.5 fixes:

Builder:
- Fixed COPY and ADD with multiple <src> to not invalidate cache if
  DOCKER_BUILDKIT=1.moby/moby#38964

Networking:
- Cleaned up the cluster provider when the agent is closed. moby/libnetwork#2354
- Windows: Now selects a random host port if the user does not specify a
  host port.  moby/libnetwork#2369
- --service-cluster-ip-range is now configurable for UCP install.
  docker/orca#10263

Signed-off-by: Peter Korsgaard <[email protected]>
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request May 7, 2019
Additional post-18.09.5 fixes:

Builder:
- Fixed COPY and ADD with multiple <src> to not invalidate cache if
  DOCKER_BUILDKIT=1.moby/moby#38964

Networking:
- Cleaned up the cluster provider when the agent is closed. moby/libnetwork#2354
- Windows: Now selects a random host port if the user does not specify a
  host port.  moby/libnetwork#2369
- --service-cluster-ip-range is now configurable for UCP install.
  docker/orca#10263

Signed-off-by: Peter Korsgaard <[email protected]>
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request May 10, 2019
Additional post-18.09.5 fixes:

Builder:
- Fixed COPY and ADD with multiple <src> to not invalidate cache if
  DOCKER_BUILDKIT=1.moby/moby#38964

Networking:
- Cleaned up the cluster provider when the agent is closed. moby/libnetwork#2354
- Windows: Now selects a random host port if the user does not specify a
  host port.  moby/libnetwork#2369
- --service-cluster-ip-range is now configurable for UCP install.
  docker/orca#10263

Signed-off-by: Peter Korsgaard <[email protected]>
(cherry picked from commit d692ecb)
Signed-off-by: Peter Korsgaard <[email protected]>
buildroot-auto-update pushed a commit to buildroot/buildroot that referenced this pull request May 10, 2019
Additional post-18.09.5 fixes:

Builder:
- Fixed COPY and ADD with multiple <src> to not invalidate cache if
  DOCKER_BUILDKIT=1.moby/moby#38964

Networking:
- Cleaned up the cluster provider when the agent is closed. moby/libnetwork#2354
- Windows: Now selects a random host port if the user does not specify a
  host port.  moby/libnetwork#2369
- --service-cluster-ip-range is now configurable for UCP install.
  docker/orca#10263

Signed-off-by: Peter Korsgaard <[email protected]>
(cherry picked from commit c7e5f9c)
Signed-off-by: Peter Korsgaard <[email protected]>
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

4 participants