-
Notifications
You must be signed in to change notification settings - Fork 110
feat: Validator binary crate, Dockerfile, and Debian package #2053
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
base: next
Are you sure you want to change the base?
Changes from 14 commits
5190a34
935377e
3d55610
fc6be76
cbba850
b28ce85
15d93a9
1b4da98
948785c
5a2510d
6dce3fb
f841cb8
ca16628
8863563
762a3e7
55ca499
c307650
bd92bd6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This will come up a bunch. Should we create separate "base images" for building and runtime?
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As far as I understand that's not possible without using some extensions. We could add dockerfile-x though and move the builder and runner image definitions to a common Dockerfile referenced by the per-service Dockerfiles.
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have gone with a merged Dockerfile with ARGs for bin and port. Might not be flexible enough for us in the end, but it might be (same will work fine for the ntx-builder image).
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets see how it goes. I don't like that it assumes the binaries are the same and can be coupled like this. I think what I was assuming originally is that we would publish build and runtime base images. I don't understand why that isn't possible; isn't that how exist?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. IIRC that requires the base image to exist in a registry |
Uh oh!
There was an error while loading. Please reload this page.