Skip to content

Commit 4a7823d

Browse files
helgimboersma
authored andcommitted
fix(make): use DEIS_REGISTRY as an env var for consisteny
1 parent cc987e6 commit 4a7823d

5 files changed

Lines changed: 11 additions & 11 deletions

File tree

Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
2-
REGISTRY ?= $(DEV_REGISTRY)
1+
# If DEIS_REGISTRY is not set, try to populate it from legacy DEV_REGISTRY
2+
DEIS_REGISTRY ?= $(DEV_REGISTRY)
33
IMAGE_PREFIX ?= deis
44
COMPONENT ?= workflow
55
BUILD_TAG ?= git-$(shell git rev-parse --short HEAD)
6-
IMAGE = $(REGISTRY)/$(IMAGE_PREFIX)/$(COMPONENT):$(BUILD_TAG)
6+
IMAGE = $(DEIS_REGISTRY)/$(IMAGE_PREFIX)/$(COMPONENT):$(BUILD_TAG)
77
SHELL_SCRIPTS = $(wildcard rootfs/bin/*) $(shell find "rootfs" -name '*.sh') $(shell find "_scripts/ci" -name '*.sh')
88

99
check-docker:

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ $ kubectl exec alpine -- curl -sS 10.247.187.217:4100/version
2626
{"etcdserver":"2.2.1","etcdcluster":"2.2.0"}
2727
```
2828

29-
Next build the deis/workflow image and push it to a Docker registry. The `$DEV_REGISTRY` environment variable must point to a registry accessible to your Kubernetes cluster. You may need to configure the Docker engines on your Kubernetes nodes to allow `--insecure-registry 192.168.0.0/16` (or the appropriate address range).
29+
Next build the deis/workflow image and push it to a Docker registry. The `$DEIS_REGISTRY` environment variable must point to a registry accessible to your Kubernetes cluster. You may need to configure the Docker engines on your Kubernetes nodes to allow `--insecure-registry 192.168.0.0/16` (or the appropriate address range).
3030

3131
```console
3232
$ make docker-build docker-push

_scripts/ci/deploy.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@
66
cd "$(dirname "$0")" || exit 1
77

88
docker login -e="$DOCKER_EMAIL" -u="$DOCKER_USERNAME" -p="$DOCKER_PASSWORD" docker.io
9-
REGISTRY=docker.io IMAGE_PREFIX=deisci BUILD_TAG=v2-alpha make -C ../.. docker-build docker-push
9+
DEIS_REGISTRY=docker.io IMAGE_PREFIX=deisci BUILD_TAG=v2-alpha make -C ../.. docker-build docker-push
1010
docker login -e="$QUAY_EMAIL" -u="$QUAY_USERNAME" -p="$QUAY_PASSWORD" quay.io
11-
REGISTRY=quay.io IMAGE_PREFIX=deisci BUILD_TAG=v2-alpha make -C ../.. docker-build docker-push
11+
DEIS_REGISTRY=quay.io IMAGE_PREFIX=deisci BUILD_TAG=v2-alpha make -C ../.. docker-build docker-push

docs/contributing/hacking.rst

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -299,20 +299,20 @@ With a dev cluster now running, we are ready to set up a local Docker registry.
299299
Configure a Docker Registry
300300
---------------------------
301301

302-
The development workflow requires Docker Registry set at the ``DEV_REGISTRY``
302+
The development workflow requires Docker Registry set at the ``DEIS_REGISTRY``
303303
environment variable. If you're developing locally you can use the ``dev-registry``
304304
target to spin up a quick, disposable registry inside a Docker container.
305305
The target ``dev-registry`` prints the registry's address and port when using ``docker-machine``;
306-
otherwise, use your host's IP address as returned by ``ifconfig`` with port 5000 for ``DEV_REGISTRY``.
306+
otherwise, use your host's IP address as returned by ``ifconfig`` with port 5000 for ``DEIS_REGISTRY``.
307307

308308
.. code-block:: console
309309
310310
$ make dev-registry
311311
312312
To configure the registry for local Deis development:
313-
export DEV_REGISTRY=192.168.59.103:5000
313+
export DEIS_REGISTRY=192.168.59.103:5000
314314
315-
It is important that you export the ``DEV_REGISTRY`` variable as instructed.
315+
It is important that you export the ``DEIS_REGISTRY`` variable as instructed.
316316

317317
If you are developing elsewhere, you must set up a registry yourself.
318318
Make sure it meets the following requirements:

docs/contributing/testing.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ continuous deployment, start one locally.
5454
registry
5555
5656
To use local registry for Deis development:
57-
export DEV_REGISTRY=192.168.59.103:5000
57+
export DEIS_REGISTRY=192.168.59.103:5000
5858
5959
.. important::
6060

0 commit comments

Comments
 (0)