-
Notifications
You must be signed in to change notification settings - Fork 582
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
fix: Make deploy use new build scripts #406
Conversation
Codecov Report
@@ Coverage Diff @@
## main #406 +/- ##
==========================================
- Coverage 68.57% 68.18% -0.40%
==========================================
Files 155 158 +3
Lines 9048 9228 +180
Branches 73 73
==========================================
+ Hits 6205 6292 +87
- Misses 2244 2309 +65
- Partials 599 627 +28
Continue to review full report at Codecov.
|
# Once `provisionerd` is available, we'll also need that binary | ||
# COPY bin/provisionerd /provisionerd | ||
# RUN chmod +x /provisionerd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating this!
cp ./images/coder/run.sh ./bin | ||
docker build --network=host -t us-docker.pkg.dev/coder-blacktriangle-dev/ci/coder:latest -f images/coder/Dockerfile ./bin | ||
cp ./images/coder/run.sh ./dist/coder_$(GOOS)_$(GOARCH) | ||
docker build --network=host -t us-docker.pkg.dev/coder-blacktriangle-dev/ci/coder:latest -f images/coder/Dockerfile ./dist/coder_$(GOOS)_$(GOARCH) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I missed this in your other PR where the binaries changed - we don't have a test for this yet, unfortunately (deploy
isn't checked at all on PRs).
We could add a smoke test where we run the docker image and verify that we can curl
and get the login page, to help make it easier to catch these in the future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh true!
No description provided.