RPM is currently the only way to install CRIO

playbooks/init/basic_facts.yml

@@ -42,12 +42,6 @@
       openshift_is_atomic: "{{ ostree_booted.stat.exists }}"
       openshift_is_containerized: "{{ ostree_booted.stat.exists or (containerized | default(false) | bool) }}"
 
-  - name: Set use_crio to True if cri-o rpm is requested
-    set_fact:
-      openshift_use_crio: True
-    when:
-    - openshift_crio_use_rpm | default(False) | bool
-
   # TODO: Should this be moved into health checks??
   # Seems as though any check that happens with a corresponding fail should move into health_checks
   # Fail as early as possible if Atomic and old version of Docker

roles/openshift_cli/defaults/main.yml

@@ -14,7 +14,4 @@ openshift_image_default: "{{ l_os_registry_url | regex_replace('${component}' |
 openshift_cli_image: "{{ (system_images_registry == 'docker') | ternary(openshift_image_default, (openshift_image_default.split('/')|length==2) | ternary(system_images_registry + '/' + openshift_image_default, openshift_image_default)) }}"
 system_openshift_cli_image: "{{ (system_images_registry == 'docker') | ternary('docker:' + openshift_cli_image, openshift_cli_image) }}"
 
-openshift_crio_use_rpm: False
-
-l_use_cli_atomic_image: "{{ (openshift_use_crio_only | bool and not openshift_crio_use_rpm | bool) }}"
-l_openshift_cli_image_backend: "{{ (l_use_cli_atomic_image | bool) | ternary('atomic', 'docker') }}"
+l_openshift_cli_image_backend: "{{ (openshift_use_crio_only | bool) | ternary('atomic', 'docker') }}"

roles/openshift_cli/tasks/main.yml

@@ -9,14 +9,14 @@
   - name: Pull CLI Image (docker)
     docker_image:
       name: "{{ openshift_cli_image }}"
-    when: not l_use_cli_atomic_image | bool
+    when: not openshift_use_crio_only | bool
 
   - name: Pull CLI Image (atomic)
     command: >
       atomic pull --storage ostree {{ system_openshift_cli_image }}
     register: pull_result
     changed_when: "'Pulling layer' in pull_result.stdout"
-    when: l_use_cli_atomic_image | bool
+    when: openshift_use_crio_only | bool
 
   # openshift_container_binary_sync is a custom module in lib_utils
   - name: Copy client binaries/symlinks out of CLI image for use on the host