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

Dockerfile #549

Closed
wants to merge 6 commits into from
Closed

Dockerfile #549

wants to merge 6 commits into from

Conversation

tibor19
Copy link

@tibor19 tibor19 commented Aug 3, 2023

This is the dockerfile branch that I created for keipy

@m00sey m00sey self-assigned this Sep 7, 2023
@m00sey
Copy link
Member

m00sey commented Sep 7, 2023

@tibor19 we have this image under images/keripy.dockerfile

However, there is a new PR #563 which might get the build time decrease you were looking for in the poc repos?

@tibor19
Copy link
Author

tibor19 commented Sep 14, 2023

@m00sey we can go with the multi-stage approach, however there are 2 things things in this pr missing from #563, the entry point doesn't start up the witness(which might not be needed, as this can be passed as parameter for the compose), and prameter filed was hard-coded, instead of being passed. I am happy with whatever choice you make.

@pfeairheller
Copy link
Member

I'm going to close this and submit a PR with the fix to configuring.py for the filed parameter.

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.

3 participants