-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Implements extra_hosts for docker_image module #59540
Conversation
The test
|
The test
|
fa90a9f
to
5856c2a
Compare
Would it be possible to add integration tests for this feature? |
5856c2a
to
4ddfae3
Compare
ff86877
to
7277eec
Compare
Please don't rebase and make changes in the same force push. That makes reviewing a lot harder than it needs to be! |
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.
One thing you definitely need is a changelog fragment.
The test
The test
|
Allows custom hosts on docker_image module. The of this option made impossible to use docker_image module to build images that required a custom hostname in /etc/hosts. For running containers this option was already present. While the python-docker API uses extra_hosts term, our existing module already uses etc_hosts argument, so it sounds better to have some consistency between docker_container and docker_image. Fixes: ansible#59233
7277eec
to
0bc076e
Compare
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.
Looks good! I marked a couple of nits, besides that it is ready for merging IMO!
Co-Authored-By: Felix Fontein <felix@fontein.de>
Co-Authored-By: Felix Fontein <felix@fontein.de>
Co-Authored-By: Felix Fontein <felix@fontein.de>
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.
shipit
@ssbarnea thanks for implementing this! It will appear in Ansible 2.9. And also thanks for taking ownership of |
SUMMARY
Allows custom hosts on docker_image module via etc_hosts parameter. Same parameter is already supported by docker_container module.
Fixes: #59233
ISSUE TYPE
COMPONENT NAME
docker_image
ADDITIONAL INFORMATION
This unblocks ansible/molecule#2182