Remove atomic check and cockpit.socket

playbooks/byo/openshift-cluster/config.yml

@@ -26,4 +26,4 @@
     openshift_cluster_id: "{{ cluster_id | default('default') }}"
     openshift_debug_level: "{{ debug_level | default(2) }}"
     openshift_deployment_type: "{{ deployment_type }}"
-    openshift_deployment_subtype: "{{ deployment_subtype }}"
+    openshift_deployment_subtype: "{{ deployment_subtype | default(none) }}"

playbooks/common/openshift-cluster/additional_config.yml

@@ -15,6 +15,6 @@
     when: openshift.common.use_manageiq | bool
   - role: cockpit
     when: not openshift.common.is_atomic and ( deployment_type in ['atomic-enterprise','openshift-enterprise'] ) and
-      (osm_use_cockpit | bool or osm_use_cockpit is undefined ) and ( deployment_subtype != 'registry' )
+      (osm_use_cockpit | bool or osm_use_cockpit is undefined ) and ( openshift.common.deployment_subtype != 'registry' )
   - role: flannel_register
     when: openshift.common.use_flannel | bool

playbooks/common/openshift-cluster/openshift_hosted.yml

@@ -45,4 +45,4 @@
   - role: openshift_metrics
     when: openshift.hosted.metrics.deploy | bool
   - role: cockpit-ui
-    when: not openshift.common.is_atomic and ( deployment_subtype == 'registry' )
+    when: ( openshift.common.deployment_subtype == 'registry' )

roles/cockpit-ui/tasks/main.yml

@@ -42,10 +42,3 @@
   register: deploy_registry_console
   changed_when: "'already exists' not in deploy_registry_console.stderr"
   failed_when: "'already exists' not in deploy_registry_console.stderr and deploy_registry_console.rc != 0"
-
-- name: Enable cockpit-ui
-  service:
-    name: cockpit.socket
-    enabled: true
-    state: started
-  when: not openshift.common.is_containerized | bool

roles/openshift_facts/library/openshift_facts.py

@@ -797,7 +797,7 @@ def set_deployment_facts_if_unset(facts):
                 curr_disabled_features = set(facts['master']['disabled_features'])
                 facts['master']['disabled_features'] = list(curr_disabled_features.union(openshift_features))
         else:
-            if deployment_type == 'atomic-enterprise':
+            if facts['common']['deployment_subtype'] == 'registry':
                 facts['master']['disabled_features'] = openshift_features
 
     if 'node' in facts:

roles/openshift_facts/tasks/main.yml

@@ -24,7 +24,7 @@
     local_facts:
       # TODO: Deprecate deployment_type in favor of openshift_deployment_type
       deployment_type: "{{ openshift_deployment_type | default(deployment_type) }}"
-      deployment_subtype: "{{ openshift_deployment_subtype | default(deployment_subtype) }}"
+      deployment_subtype: "{{ openshift_deployment_subtype | default(None) }}"
       cluster_id: "{{ openshift_cluster_id | default('default') }}"
       hostname: "{{ openshift_hostname | default(None) }}"
       ip: "{{ openshift_ip | default(None) }}"

utils/src/ooinstall/openshift_ansible.py

@@ -25,9 +25,6 @@ VARIABLES_MAP = {
     'variant_subtype': 'deployment_subtype',
     'master_routingconfig_subdomain': 'openshift_master_default_subdomain',
     'proxy_http': 'openshift_http_proxy',
-    'variant_subtype': 'deployment_subtype',
-    'master_routingconfig_subdomain':'openshift_master_default_subdomain',
-    'proxy_http':'openshift_http_proxy',
     'proxy_https': 'openshift_https_proxy',
     'proxy_exclude_hosts': 'openshift_no_proxy',
 }

utils/src/ooinstall/variants.py

@@ -45,9 +45,9 @@ OSE = Variant('openshift-enterprise', 'OpenShift Container Platform',
 )
 
 REG = Variant('openshift-enterprise', 'Registry',
-    [
-        Version('3.2', 'openshift-enterprise', 'registry'),
-    ]
+              [
+                  Version('3.2', 'openshift-enterprise', 'registry'),
+              ]
 )
 
 origin = Variant('origin', 'OpenShift Origin',