Refactor to enable local storage

Refactor code from assisted_deployment.sh to enable_local_storage.sh
so that the local-storage operator can be enabled for deployed clusters
independent of the other assisted dependencies.

Note this currently only works for ga builds (not nightly/ci)

Author: Steven Hardy

assisted_deployment.sh

@@ -22,76 +22,9 @@ ASSISTED_OPERATOR_INDEX="${ASSISTED_OPERATOR_INDEX:-quay.io/ocpmetal/assisted-se
 ASSETS_DIR="${OCP_DIR}/saved-assets/assisted-installer-manifests"
 
 
-function generate_local_storage() {
-  mkdir -p "${ASSETS_DIR}"
-
-  cat >"${ASSETS_DIR}/01-local-storage-operator.yaml" <<EOF
-apiVersion: operators.coreos.com/v1alpha2
-kind: OperatorGroup
-metadata:
-  name: local-operator-group
-  namespace: openshift-local-storage
-spec:
-  targetNamespaces:
-    - openshift-local-storage
----
-apiVersion: operators.coreos.com/v1alpha1
-kind: Subscription
-metadata:
-  name: local-storage-operator
-  namespace: openshift-local-storage
-spec:
-  installPlanApproval: Automatic
-  name: local-storage-operator
-  source: redhat-operators
-  sourceNamespace: openshift-marketplace
-EOF
-
-  cat >"${ASSETS_DIR}/02-local-volume.yaml" <<EOCR
-apiVersion: local.storage.openshift.io/v1
-kind: LocalVolume
-metadata:
-  name: assisted-service
-  namespace: openshift-local-storage
-spec:
-  logLevel: Normal
-  managementState: Managed
-  storageClassDevices:
-$(fill_local_storage)
-      storageClassName: assisted-service
-      volumeMode: Filesystem
-EOCR
-}
-
-
-function fill_local_storage() {
-  if [ ! -z "${VM_EXTRADISKS_LIST}" ]; then
-cat <<EOF
-    - devicePaths:
-EOF
-  fi
-
-  for disk in ${VM_EXTRADISKS_LIST}; do
-cat <<EOF
-        - /dev/$disk
-EOF
-  done
-}
-
-
 function deploy_local_storage() {
-  oc adm new-project openshift-local-storage || true
-
-  oc annotate project openshift-local-storage openshift.io/node-selector=''
-
-  generate_local_storage
-
-  echo "Creating local storage operator group and subscription..."
-  oc apply -f "${ASSETS_DIR}/01-local-storage-operator.yaml"
-  wait_for_crd "localvolumes.local.storage.openshift.io"
-
-  echo "Creating local volume and storage class..."
-  oc apply -f "${ASSETS_DIR}/02-local-volume.yaml"
+  export STORAGE_CLASS_NAME="assisted-service"
+  ${SCRIPTDIR}/enable_local_storage.sh
 }
 
 

enable_local_storage.sh

@@ -0,0 +1,90 @@
+#!/usr/bin/env bash
+set -ex
+
+source logging.sh
+source common.sh
+source utils.sh
+
+ASSETS_DIR=${ASSETS_DIR:-"${OCP_DIR}/enable-local-storage"}
+STORAGE_CLASS_NAME=${STORAGE_CLASS_NAME:-local-storage}
+
+
+function generate_local_storage() {
+  mkdir -p "${ASSETS_DIR}"
+
+  cat >"${ASSETS_DIR}/01-local-storage-operator.yaml" <<EOF
+apiVersion: operators.coreos.com/v1alpha2
+kind: OperatorGroup
+metadata:
+  name: local-operator-group
+  namespace: openshift-local-storage
+spec:
+  targetNamespaces:
+    - openshift-local-storage
+---
+apiVersion: operators.coreos.com/v1alpha1
+kind: Subscription
+metadata:
+  name: local-storage-operator
+  namespace: openshift-local-storage
+spec:
+  installPlanApproval: Automatic
+  name: local-storage-operator
+  source: redhat-operators
+  sourceNamespace: openshift-marketplace
+EOF
+
+  cat >"${ASSETS_DIR}/02-local-volume.yaml" <<EOCR
+apiVersion: local.storage.openshift.io/v1
+kind: LocalVolume
+metadata:
+  name: ${STORAGE_CLASS_NAME}
+  namespace: openshift-local-storage
+spec:
+  logLevel: Normal
+  managementState: Managed
+  storageClassDevices:
+$(fill_local_storage)
+      storageClassName: ${STORAGE_CLASS_NAME}
+      volumeMode: Filesystem
+EOCR
+}
+
+
+function fill_local_storage() {
+  if [ ! -z "${VM_EXTRADISKS_LIST}" ]; then
+cat <<EOF
+    - devicePaths:
+EOF
+  fi
+
+  for disk in ${VM_EXTRADISKS_LIST}; do
+cat <<EOF
+        - /dev/$disk
+EOF
+  done
+}
+
+
+function deploy_local_storage() {
+  oc adm new-project openshift-local-storage || true
+
+  oc annotate --overwrite project openshift-local-storage openshift.io/node-selector=''
+
+  generate_local_storage
+
+  echo "Creating local storage operator group and subscription..."
+  oc apply -f "${ASSETS_DIR}/01-local-storage-operator.yaml"
+  wait_for_crd "localvolumes.local.storage.openshift.io"
+
+  echo "Creating local volume and storage class..."
+  oc apply -f "${ASSETS_DIR}/02-local-volume.yaml"
+}
+
+
+if [ "${VM_EXTRADISKS}" != "false" ]; then
+  deploy_local_storage
+else
+  echo "Cannot deploy local storage unless VM_EXTRADISKS is enabled"
+  exit 1
+fi