Closed warrenvw closed 6 years ago
Please remove the # TODO: In a future PR, deploy artifact to S3.
from the CircleCI config, - we don't need it in this repo.
Hm, curious why it's running the deploy-ova
(and failing there) after removing the branch
config from CircleCI in favor of tag
-only.
Looks like it was the CircleCI native git that messed things up in my earlier tests.. once I used an image with git installed, the deploy started working properly. Woot!
In summary, the deploy-ova
job is only executed for a tag push. A branch push does not result in this job being executed.
Great work :+1: Just left one minor comment.
Okay. I'll address in another PR.
Closes #9.
Remaining packer-st2 CI tasks
GITHUB_TOKEN
to packer-st2 CircleCI environment.circleci/config.yml
such that a non-tagged build will result in thedeploy-ova
job NOT being executed.ghr
tool indeploy-ova
job, providingCIRCLE_PROJECT_USERNAME
,CIRCLE_PROJECT_REPONAME
andCIRCLE_TAG
..circleci/config.yml
such that a tagged build will result in thedeploy-ova
job being executed.Remaining test tasks