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

docker: extra_hosts argument to add hosts to a container’s /etc/hosts… #6185

Merged
merged 3 commits into from
Aug 1, 2023

Conversation

Azanul
Copy link
Contributor

@Azanul Azanul commented Jul 28, 2023

Fixes #6139

Emulates Docker CLI's --add-host argument

This adds a extra_hosts argument to docker_build which will get passed through to Docker/BuildKit for adding ExtraHosts.

… file

Docker CLI has a `--add-host` argument

This adds a `extra_hosts` argument to `docker_build` which will get
passed through to Docker/BuildKit for adding ExtraHosts.

Signed-off-by: Azanul <[email protected]>
Copy link
Member

@nicks nicks left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

the tests are failing because you haven't run the code generator, but lemme see how hard that is to add on top

Signed-off-by: Azanul <[email protected]>
Signed-off-by: Azanul <[email protected]>
@Azanul
Copy link
Contributor Author

Azanul commented Aug 1, 2023

LGTM, thanks!

the tests are failing because you haven't run the code generator, but lemme see how hard that is to add on top

I've generated the code and updated the tests, now the CI is passing

@nicks nicks merged commit 70a5364 into tilt-dev:master Aug 1, 2023
1 check passed
nicks added a commit to nicks/tilt that referenced this pull request Aug 29, 2023
nicks added a commit that referenced this pull request Aug 30, 2023
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.

Docker Build Utility Doesn't Have --add-host Option
2 participants