diff options
| author | Michael Gugino <mgugino@redhat.com> | 2017-08-23 15:23:03 -0400 |
|---|---|---|
| committer | Michael Gugino <mgugino@redhat.com> | 2017-08-25 08:55:49 -0400 |
| commit | 53028f806e86b2985e79e9103ffbc78ce83b6d57 (patch) | |
| tree | f2fd5ea40d6da1222e20c72a24fb71dd5d9dceda | |
| parent | b8e45471aa70881ff608962f0fec236c31c2398d (diff) | |
| download | openshift-53028f806e86b2985e79e9103ffbc78ce83b6d57.tar.gz openshift-53028f806e86b2985e79e9103ffbc78ce83b6d57.tar.bz2 openshift-53028f806e86b2985e79e9103ffbc78ce83b6d57.tar.xz openshift-53028f806e86b2985e79e9103ffbc78ce83b6d57.zip | |
Remove near-meta role openshift_cli_facts
openshift_cli_facts only runs a single task, and otherwise
the only existence is to serve as a link in the dependency
chain towards openshift_facts.
This commit flattens openshift_cli into one role.
| -rw-r--r-- | playbooks/common/openshift-cluster/initialize_facts.yml | 1 | ||||
| -rw-r--r-- | roles/openshift_cli/meta/main.yml | 2 | ||||
| -rw-r--r-- | roles/openshift_cli_facts/meta/main.yml | 15 | ||||
| -rw-r--r-- | roles/openshift_cli_facts/tasks/main.yml | 6 |
4 files changed, 2 insertions, 22 deletions
diff --git a/playbooks/common/openshift-cluster/initialize_facts.yml b/playbooks/common/openshift-cluster/initialize_facts.yml index 4bf5d33b1..e90a49390 100644 --- a/playbooks/common/openshift-cluster/initialize_facts.yml +++ b/playbooks/common/openshift-cluster/initialize_facts.yml @@ -115,6 +115,7 @@ debug_level: "{{ openshift_debug_level | default(2) }}" deployment_type: "{{ openshift_deployment_type }}" deployment_subtype: "{{ openshift_deployment_subtype | default(None) }}" + cli_image: "{{ osm_image | default(None) }}" cluster_id: "{{ openshift_cluster_id | default('default') }}" hostname: "{{ openshift_hostname | default(None) }}" ip: "{{ openshift_ip | default(None) }}" diff --git a/roles/openshift_cli/meta/main.yml b/roles/openshift_cli/meta/main.yml index c1de367d9..04a1ce873 100644 --- a/roles/openshift_cli/meta/main.yml +++ b/roles/openshift_cli/meta/main.yml @@ -15,4 +15,4 @@ dependencies: - role: openshift_docker when: not skip_docker_role | default(False) | bool - role: openshift_common -- role: openshift_cli_facts +- role: openshift_facts diff --git a/roles/openshift_cli_facts/meta/main.yml b/roles/openshift_cli_facts/meta/main.yml deleted file mode 100644 index 59acde215..000000000 --- a/roles/openshift_cli_facts/meta/main.yml +++ /dev/null @@ -1,15 +0,0 @@ ---- -galaxy_info: - author: Jason DeTiberus - description: OpenShift CLI Facts - company: Red Hat, Inc. - license: Apache License, Version 2.0 - min_ansible_version: 1.9 - platforms: - - name: EL - versions: - - 7 - categories: - - cloud -dependencies: -- role: openshift_facts diff --git a/roles/openshift_cli_facts/tasks/main.yml b/roles/openshift_cli_facts/tasks/main.yml deleted file mode 100644 index dd1ed8965..000000000 --- a/roles/openshift_cli_facts/tasks/main.yml +++ /dev/null @@ -1,6 +0,0 @@ ---- -# TODO: move this to a new 'cli' role -- openshift_facts: - role: common - local_facts: - cli_image: "{{ osm_image | default(None) }}" |