-
Notifications
You must be signed in to change notification settings - Fork 9
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
Passing Custom Parameters to the Image Building Job Container #68
Comments
Hello, we already possess the pertinent installation parameter https://docs.yatai.io/en/latest/installation/yatai_image_builder.html#prepare-container-registry |
I was following the instructions from the document you posted, but I specified a docker.base_image in the bentomlfile, which is an image from my private.repo. When the image-builder tries to pull this image, it encounters an error, similar to "requested HTTPS but received an HTTP response". After looking at the command-line help for kaniko, it mentions the following parameter:
According to its description, the I noticed that the current controller's code has hardcoded kaniko parameters. What I would like is the ability to pass the |
Sorry for my oversight, there actually is an implementation for this "extra parameter". It might not have been mentioned in the documentation. It is possible to achieve this through the builder_args in the configMap yatai-image-builder/controllers/resources/bentorequest_controller.go Lines 2452 to 2460 in d4ab40b
|
My base image is in an internal Docker registry without TLS. When the image builder (Kaniko) operates, it defaults to pulling images via HTTPS, which leads to a failure in building the image. I noticed that the parameters for Kaniko are hardcoded in the Bentorequest's controller, and it seems there's no option to add the
--insecure-pull
or--insecure-registry
parameters.Currently, I have modified the Kaniko image, using something like
/kaniko/origin-executor --insecure-registry=MYREGISTRY.REPO $@
to work around this issue.Is this by design?
If not, is it possible to add this parameter through an environment variable, similar to
DOCKER_INSECURE_REGISTRY
?The text was updated successfully, but these errors were encountered: