Merge pull request #48 from ashish-amarnath/be-non-gcr-friendly

Update publish-manager to be more non gcp friendly

README.md

@@ -36,7 +36,7 @@ Run: `./scripts/publish-manager.sh`
 
 #### Using Docker
 
-Alternatively, run: `docker build -t <MY_REPOSITORY>/capd-manager:latest .`
+Alternatively, run: `REGISTRY=<MY_REGISTRY> ./scripts/publish-manager.sh`
 
 ## Trying CAPD
 

scripts/publish-manager.sh

@@ -13,13 +13,19 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+readonly GCP_PROJECT=$(gcloud config get-value project)
+
 set -o errexit
 set -o xtrace
 
-REGISTRY=$(gcloud config get-value project)
-TAG=${TAG:-dev}
+readonly IMAGE_NAME="capd-manager"
+
+readonly GCR_REGISTRY="gcr.io/${GCP_PROJECT}"
+readonly TAG=${TAG:-dev}
+
+readonly REGISTRY=${REGISTRY:-$GCR_REGISTRY}
 
-IMAGE="gcr.io/${REGISTRY}/capd-manager:${TAG}"
+readonly IMAGE=${REGISTRY}/${IMAGE_NAME}:${TAG}
 
 docker build --file Dockerfile -t "${IMAGE}" .
-gcloud docker -- push "${IMAGE}"
+docker push "${IMAGE}"