search

mesos / storm

Storm on Mesos!
Apache License 2.0
139 stars 66 forks source link

use $TRAVIS_REPO_SLUG instead of hardcoded repo slug of mesos/storm #150

Closed erikdw closed 8 years ago

erikdw commented 8 years ago

Remove hardcoded GitHub & DockerHub repo slugs for deploy, in favor of the $TRAVIS_REPO_SLUG environment variable.

The advantage of is that we can do testing of build-related changes in child forks of mesos/storm without having to carefully update deploy.on.repo & DOCKER_REPO to our own personal repos. Similarly we won't have to maintain such changes in our child forks (e.g., by doing git stash save and then git stash pop when we are working in that child repo).

Tested this behavior already in my own fork of mesos/storm (erikdw/storm-mesos):

Files got uploaded to the expected locations:

erikdw commented 8 years ago

@JessicaLHartog already reviewed this proposal in #149. So just merging this now.