Skip to content

Commit

Permalink
Deal with missing make in docker image, fix typo
Browse files Browse the repository at this point in the history
  • Loading branch information
errordeveloper committed Jun 5, 2018
1 parent 11c792f commit 56ed5bf
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 10 deletions.
12 changes: 3 additions & 9 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
@@ -1,22 +1,16 @@
version: 2
jobs:
test:
docker:
# https://circleci.com/docs/2.0/building-docker-images/#docker-version
- image: docker:17.11.0-ce
# use machine build, as otherwise we cannot use make and have to duplicate makefile
machine: true
steps:
- checkout
- setup_remote_docker: {docker_layer_caching: true}
# just a simple build for now, but will use skaffold if it gets more complex
# TODO(p3): run unit tests
- run: make eksctl_image
build:
docker:
# https://circleci.com/docs/2.0/building-docker-images/#docker-version
- image: docker:17.11.0-ce
machine: true
steps:
- checkout
- setup_remote_docker: {docker_layer_caching: true}
- run:
name: Build and publish
command: |
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ eksctl_image: eksctl_build_image

release: eksctl_build_image
docker run \
--env=GIHUB_TOKEN \
--env=GITHUB_TOKEN \
--volume=$(CURDIR):/go/src/github.com/weaveworks/eksctl \
--workdir=/go/src/github.com/weaveworks/eksctl \
eksctl_build \
Expand Down

0 comments on commit 56ed5bf

Please sign in to comment.