Skip to content

Fix Docker Build/Run Issues#6

Merged
aaj3f merged 3 commits intomainfrom
task/prep-docker-usage
Oct 19, 2023
Merged

Fix Docker Build/Run Issues#6
aaj3f merged 3 commits intomainfrom
task/prep-docker-usage

Conversation

@aaj3f
Copy link
Collaborator

@aaj3f aaj3f commented Oct 19, 2023

A few changes here

  • Dockerfile used wrong artifact names so that the ENTRYPOINT failed on docker run
  • config.edn needed a :docker profile value for consensus-this-server
  • (it looks like my Makefile suggestions for docker-build, docker-run, and docker-push made their way in here somehow!) docker-build needed a --load flag to make it into a local machine's image repository

Copy link
Contributor

@zonotope zonotope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aaj3f aaj3f merged commit 1d24a3f into main Oct 19, 2023
@aaj3f aaj3f deleted the task/prep-docker-usage branch October 19, 2023 21:03
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.

2 participants