Fix for dynamic pvs when using storageclasses.

roles/openshift_logging_elasticsearch/tasks/main.yaml

@@ -194,7 +194,9 @@
     - port: 9200
       targetPort: "restapi"
 
-- name: Creating ES storage template
+# storageclasses are used by default but if static then disable
+# storageclasses with the storageClassName set to "" in pvc.j2
+- name: Creating ES storage template - static
   template:
     src: pvc.j2
     dest: "{{ tempdir }}/templates/logging-es-pvc.yml"
@@ -203,11 +205,13 @@
     size: "{{ openshift_logging_elasticsearch_pvc_size }}"
     access_modes: "{{ openshift_logging_elasticsearch_pvc_access_modes | list }}"
     pv_selector: "{{ openshift_logging_elasticsearch_pvc_pv_selector }}"
+    storage_class_name: "{{ openshift_logging_elasticsearch_pvc_storage_class_name | default("") }}"
   when:
   - openshift_logging_elasticsearch_storage_type == "pvc"
   - not openshift_logging_elasticsearch_pvc_dynamic
 
-- name: Creating ES storage template
+# Storageclasses are used by default if configured
+- name: Creating ES storage template - dynamic
   template:
     src: pvc.j2
     dest: "{{ tempdir }}/templates/logging-es-pvc.yml"
@@ -216,8 +220,6 @@
     size: "{{ openshift_logging_elasticsearch_pvc_size }}"
     access_modes: "{{ openshift_logging_elasticsearch_pvc_access_modes | list }}"
     pv_selector: "{{ openshift_logging_elasticsearch_pvc_pv_selector }}"
-    annotations:
-      volume.beta.kubernetes.io/storage-class: "dynamic"
   when:
   - openshift_logging_elasticsearch_storage_type == "pvc"
   - openshift_logging_elasticsearch_pvc_dynamic

roles/openshift_logging_elasticsearch/templates/pvc.j2

@@ -25,3 +25,6 @@ spec:
   resources:
     requests:
       storage: {{size}}
+{% if storage_class_name is defined %}
+  storageClassName: {{ storage_class_name }}
+{% endif %}

roles/openshift_metrics/tasks/install_cassandra.yaml

@@ -36,6 +36,7 @@
     access_modes: "{{ openshift_metrics_cassandra_pvc_access | list }}"
     size: "{{ openshift_metrics_cassandra_pvc_size }}"
     pv_selector: "{{ openshift_metrics_cassandra_pv_selector }}"
+    storage_class_name: "{{ openshift_metrics_cassanda_pvc_storage_class_name | default("", true) }}"
   with_sequence: count={{ openshift_metrics_cassandra_replicas }}
   when:
   - openshift_metrics_cassandra_storage_type != 'emptydir'
@@ -50,8 +51,6 @@
     obj_name: "{{ openshift_metrics_cassandra_pvc_prefix }}-{{ item }}"
     labels:
       metrics-infra: hawkular-cassandra
-    annotations:
-      volume.beta.kubernetes.io/storage-class: dynamic
     access_modes: "{{ openshift_metrics_cassandra_pvc_access | list }}"
     size: "{{ openshift_metrics_cassandra_pvc_size }}"
     pv_selector: "{{ openshift_metrics_cassandra_pv_selector }}"

roles/openshift_metrics/templates/pvc.j2

@@ -32,3 +32,6 @@ spec:
   resources:
     requests:
       storage: {{size}}
+{% if storage_class_name is defined %}
+  storageClassName: {{ storage_class_name }}
+{% endif %}