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
docs: add missing IPAM options to API docs #26857
docs: add missing IPAM options to API docs #26857
Conversation
@@ -3045,6 +3045,7 @@ Content-Type: application/json | |||
"Id": "7d86d31b1478e7cca9ebed7e73aa0fdeec46c5ca29497431d3007d2d9e15ed99", | |||
"Scope": "local", | |||
"Driver": "bridge", | |||
"EnableIPv6": false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a spurious change, and this flag was not ther ein 1.22 API version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! removed it :)
786254a
to
bd87eab
Compare
- **Driver** - Name of the network driver plugin to use. Defaults to `bridge` driver | ||
- **IPAM** - Optional custom IP scheme for the network | ||
- **Driver** - Name of the IPAM driver to use. Defaults to `default` driver | ||
- **Config** - List of IPAM configuration options, specified as a map: | ||
`{"Subnet": <CIDR>, "IPRange": <CIDR>, "Gateway": <IP address>}` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this has to be an exhaustive list, we should also add "AuxAddress": <device_name:IP address>
ping @thaJeztah 👼 |
Commit dd28ded brought in new networking features and API, but did not document the available IPAM options. This adds a description of those options. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
bd87eab
to
d69fce7
Compare
Yay, finally updated 😄 ping @aboch @vdemeester PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🐸
author LGTM |
fixes #26853
Commit dd28ded (#16645)
brought in new networking features and API, but
did not document the available IPAM options.
This adds a description of those options.
Also change the value of
CheckDuplicate
used in the examples,because "false" is the default, so not the best value to illustrate its usage
ping @mavenugo @aboch PTAL if descriptions are correct