-
Notifications
You must be signed in to change notification settings - Fork 58
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
Allow to specify the name of the Dockerfile #2187
Comments
Hey @leovct ! This is definitely something we want to support. Thanks for letting us know this would be helpful for you so we can prioritize it internaly! |
Thanks @leovct ; this would help with the applicability of ImageBuildSpec! If we can call this kurtosis/container-engine-lib/lib/backend_interface/objects/image_build_spec/image_build_spec.go Line 15 in be97c38
|
Yep happy to take a look :) |
## Description: Allow to specify the Dockerfile name in `ImageBuildSpec`. Resolves #2187 - Add a new `build_file` attribute in `ImageBuildSpec`. - Update `ImageBuildSpec` tests and add custom build file tests - Update the `ImageBuildSpec`, `TargetStageAttr` validator to `nil` since it's an optional value. - Update `ImageBuildSpec`docs. - (not related) `go mod tidy`. ## Is this change user facing? ✅ <!-- If yes, please add the "user facing" label to the PR --> <!-- If yes, don't forget to include docs changes where relevant --> ## References (if applicable): <!-- Add relevant Github Issues, Discord threads, or other helpful information. -->
Background & motivation
By default,
ImageBuildSpec
will search for a file calledDockerfile
in thebuild_context_dir
. I would like to be able to specify the name of the Dockerfile via an optional argument.Just like what
docker build
allows:Btw I'm open to work on this to get more familiar with the code base if possible :)
Desired behaviour
Here is the new specification of
ImageBuildSpec
with an optional additional argument calledfile
.How important is this to you?
Nice to have; this feature would make using Kurtosis more enjoyable.
What area of the product does this pertain to?
Other: anything not covered by the above
The text was updated successfully, but these errors were encountered: