Skip to content

Commit

Permalink
Use standard "images" target vs "dapper-image"
Browse files Browse the repository at this point in the history
As a part of simplifying our building/testing experience, all projects
are moving to using a standard `make images` command to build whatever
container images they are responsible for.

This also reduces the need for Shipyard-specific docs.

Signed-off-by: Daniel Farrell <[email protected]>
  • Loading branch information
dfarrell07 authored and skitt committed Oct 6, 2020
1 parent 8a8f116 commit f174e97
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/e2e-consuming.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
free -h
- name: Build the latest Shipyard image
run: make dapper-image
run: make images

- name: Checkout the ${{ matrix.project }} repository
uses: actions/checkout@master
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
- name: Build new images
env:
IMAGES_ARGS: --nocache ${{ env.UPX_FLAG }}
run: make dapper-image nettest
run: make images nettest

- name: Release the images
env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/scripts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:

- name: Test the compile.sh script
run: |
make dapper-image .dapper
make images .dapper
./.dapper -m bind test/scripts/compile/test.sh
deploy:
timeout-minutes: 30
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/validate-consuming.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
uses: actions/checkout@master

- name: Build the latest Shipyard image
run: make dapper-image
run: make images

- name: Checkout the ${{ matrix.project }} repository
uses: actions/checkout@master
Expand Down
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,11 @@ include Makefile.images
include Makefile.versions

# Shipyard-specific starts
clusters deploy e2e gitlint golangci-lint markdownlint nettest post-mortem unit-test validate: dapper-image
clusters deploy e2e gitlint golangci-lint markdownlint nettest post-mortem unit-test validate: images

dapper-image: export SCRIPTS_DIR=./scripts/shared
images: export SCRIPTS_DIR=./scripts/shared

dapper-image: package/.image.shipyard-dapper-base
images: package/.image.shipyard-dapper-base

.DEFAULT_GOAL := validate
# Shipyard-specific ends
Expand Down
2 changes: 1 addition & 1 deletion Makefile.dapper
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

# Only run command line goals in dapper (except things that have to run outside of dapper).
# Otherwise, make applies this rule to various files and tries to build them in dapper (which doesn't work, obviously).
$(filter-out .dapper dapper-image shell,$(MAKECMDGOALS)): .dapper
$(filter-out .dapper images shell,$(MAKECMDGOALS)): .dapper
+./.dapper -m bind make $@ $(MAKEFLAGS)

shell: .dapper
Expand Down

0 comments on commit f174e97

Please sign in to comment.