NYCPlanning / deprecated-labs-zap-api

Deprecated version of the ZAP API, see https://github.com/NYCPlanning/labs-zap-api
Other
5 stars 3 forks source link

spatial download circleci docker changes broke deploy #128

Closed ghost closed 5 years ago

ghost commented 5 years ago

Master deployment of zap-api including the updated circleci configuration to include custom docker images for gdal dependency failed:

#!/bin/bash -eo pipefail
if [ "${CIRCLE_BRANCH}" == "master" ]; then
  git remote add dokku dokku@zap-api.planninglabs.nyc:zap-api && git push dokku master
elif [ "${CIRCLE_BRANCH}" == "develop" ]; then
  git remote add dokku dokku@zap-api.planninglabs.nyc:zap-api-staging && git push dokku develop:master
fi
error: src refspec master does not match any.
error: failed to push some refs to 'dokku@zap-api.planninglabs.nyc:zap-api'
Exited with code 1

Believe this was due to silent failure fallback in checkout step:

Either git or ssh (required by git to clone through SSH) is not installed in the image. Falling back to CircleCI's native git client but the behavior may be different from official git. If this is an issue, please use an image that has official git and ssh installed.
Enumerating objects: 100, done.
Counting objects: 100% (100/100), done.
Compressing objects: 100% (76/76), done.
Total 1791 (delta 39), reused 58 (delta 23), pack-reused 1691

Presumably, master was not checked out & tracked properly, and thus was not available to be pushed up to dokku remote.