diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2017-09-21 10:21:34 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-09-21 10:21:34 -0700 |
| commit | 505764651b3f8279ffe8881e9b26851d1dd14258 (patch) | |
| tree | 910d1f193dfdb9646bdec6c4331efebb4646a5fb /roles/docker | |
| parent | f18cf4fd09448f2546d1c903f68acadb0c1367d9 (diff) | |
| parent | 0fee9f4e194101cfbd094d140b59d1b262b9b6a6 (diff) | |
| download | openshift-505764651b3f8279ffe8881e9b26851d1dd14258.tar.gz openshift-505764651b3f8279ffe8881e9b26851d1dd14258.tar.bz2 openshift-505764651b3f8279ffe8881e9b26851d1dd14258.tar.xz openshift-505764651b3f8279ffe8881e9b26851d1dd14258.zip | |
Merge pull request #5477 from mgugino-upstream-stage/hot-fix-oreg_url
Automatic merge from submit-queue
Remove default value for oreg_url
Due to some plays importing variables from roles
directly, oreg_url was being set to a default
value when it otherwise shouldn't be.
This commit removes the default values for oreg_url
to ensure existing logic works as desired.
Fixes: https://github.com/openshift/openshift-ansible/issues/5455
Diffstat (limited to 'roles/docker')
| -rw-r--r-- | roles/docker/defaults/main.yml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/roles/docker/defaults/main.yml b/roles/docker/defaults/main.yml index 7e206ded1..81f3ee9e4 100644 --- a/roles/docker/defaults/main.yml +++ b/roles/docker/defaults/main.yml @@ -1,6 +1,6 @@ --- docker_cli_auth_config_path: '/root/.docker' -oreg_url: '' -oreg_host: "{{ oreg_url.split('/')[0] if '.' in oreg_url.split('/')[0] else '' }}" +# oreg_url is defined by user input. +oreg_host: "{{ oreg_url.split('/')[0] if (oreg_url is defined and '.' in oreg_url.split('/')[0]) else '' }}" oreg_auth_credentials_replace: False |