Skip to content

Commit

Permalink
Merge pull request #429 from mattcary/cloudbuild
Browse files Browse the repository at this point in the history
Tweak make rules to make cloudbuild work
  • Loading branch information
andyzhangx authored Mar 12, 2022
2 parents a65891c + 6bda62a commit fee2439
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,12 @@ update:
hack/update-dependencies.sh
hack/verify-update.sh

# There seems to be some shell quoting problem with cloudbuild.yaml, and trying
# to pass multiple make targets appears as a single quoted string. See
# cloudbuild.yaml for how this is used.
.PHONY: cloudbuild
cloudbuild: container-all push-manifest push-latest

.PHONY: verify
verify: unit-test
hack/verify-all.sh
Expand Down
2 changes: 1 addition & 1 deletion cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ steps:
# set the home to /root explicitly to if using docker buildx
- HOME=/root
args:
- container-all push-manifest push-latest
- cloudbuild

substitutions:
_STAGING_PROJECT: 'k8s-staging-cloud-provider-gcp'

0 comments on commit fee2439

Please sign in to comment.