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

Add support for "docker swarm init --data-path-port" #296

Closed
jensens opened this issue Feb 7, 2022 · 2 comments · Fixed by #466
Closed

Add support for "docker swarm init --data-path-port" #296

jensens opened this issue Feb 7, 2022 · 2 comments · Fixed by #466

Comments

@jensens
Copy link

jensens commented Feb 7, 2022

SUMMARY

The docker swarm init parameter --data-path-port is not supported, at least as I searched I did not find it.

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

community.docker_swarm

ADDITIONAL INFORMATION

--data-path-port can be passed to docker swarm init to use a different port for the bridge network.

Background/use-case:
In our customers setup the port is was already used by another network in VMWare, so we were forced to set the bridge/VXLAN UDP port to a different one using mentioned parameter.

see https://docs.docker.com/engine/reference/commandline/swarm_init/#options

I would expect a YAML like this

- name: Init a new swarm with different data-path-port parameter
  docker_swarm:
    state: present
    data_path_port: 4790
@felixfontein
Copy link
Collaborator

A quick search of https://docker-py.readthedocs.io/en/stable/api.html and the checked out git repo does not indicate this is supported by Docker SDK for Python. (On the API side, the property is called DataPathPort.)

@felixfontein felixfontein added docker-plain plain Docker (no swarm, no compose, no stack) waiting on Docker SDK for Python support labels Feb 8, 2022
@felixfontein felixfontein added docker-swarm Docker Swarm and removed docker-plain plain Docker (no swarm, no compose, no stack) labels Apr 24, 2022
@thehedgefrog
Copy link
Contributor

This is now supported in docker-py via this PR: docker/docker-py#2987

The collection would need to be updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants