-
Notifications
You must be signed in to change notification settings - Fork 16
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
Dockerfile templates for one-off images #279
Comments
Template would look similar but with a custom build cmd and the
|
only my 2 cents on this: For func we also thought about having the template in the func/kn-plugin-func repot, but decided to keep this template in the hack repo, as that way, we have one single place to maintain the production Dockerfiles (/templates) and don't have to do this in multiple repos. |
I don't see a problem adding another embedded template in the hack repo. So far we have only the following ones:
The number of custom templates will NOT grow indefinitely. We have just a few. |
@creydr @pierDipi @mgencur
Per my PR to onboard client repo to CI generators. I'd added very handy Dockerfile generators. There's similar issue to
func-until
image though. There'scli-artifact
image produced to carry archives platform dependent binaries. It's utilized in OpenShift Console download view.I still need to figure out how to template this. I don't want to every grow embeded templates. I was thinking about
Dockerfile.template
being stored in client midstream. And generator would discover it automagically or in a more controlled way through--from-template-file
(whatever name makes sense) here.But inherently every major change in templates like included metadata and so on would require change of the local file.
Would it be just simpler to have another template type stored as embedded template here?
For far I have only hardcoded everything version: https://github.com/openshift-knative/client/blob/release-v1.15/openshift/ci-operator/knative-images/cli-artifacts/Dockerfile
The text was updated successfully, but these errors were encountered: