Skip to content

added docker network_mode options in documentation#21986

Merged
potiuk merged 1 commit into
apache:mainfrom
Bowrna:docker-network-docs
Mar 6, 2022
Merged

added docker network_mode options in documentation#21986
potiuk merged 1 commit into
apache:mainfrom
Bowrna:docker-network-docs

Conversation

@Bowrna

@Bowrna Bowrna commented Mar 4, 2022

Copy link
Copy Markdown
Contributor

closes: #8418
related: #8418
Added documentation for Docker operator network_mode option.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

In host network stack, port_binding option is incompatible as container and it's host shares same networking namespace and the container has the same IP address as that of the host. However in create_host_config function in DockerOperator we don't support port_binding as param. So I am not sure if we have to add this detail Incompatible with port_bindings in the documentation.

@Bowrna Bowrna force-pushed the docker-network-docs branch 3 times, most recently from bd775b0 to dc08e17 Compare March 6, 2022 07:59
@Bowrna Bowrna force-pushed the docker-network-docs branch from dc08e17 to 11022cc Compare March 6, 2022 09:24
@potiuk potiuk merged commit 7acc190 into apache:main Mar 6, 2022
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update docker operator network documentation

3 participants