]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
Fixed quotes and removed combined_ prefix from variables that no longer need it
authorJames Saint-Rossy <james_saintrossy@cable.comcast.com>
Tue, 16 Aug 2016 21:49:30 +0000 (17:49 -0400)
committerJames Saint-Rossy <james_saintrossy@cable.comcast.com>
Tue, 16 Aug 2016 21:49:30 +0000 (17:49 -0400)
roles/ceph-osd/tasks/activate_osds.yml
roles/ceph-osd/tasks/check_devices.yml
roles/ceph-osd/tasks/check_devices_auto.yml
roles/ceph-osd/tasks/check_devices_static.yml
roles/ceph-osd/tasks/main.yml
roles/ceph-osd/tasks/scenarios/bluestore.yml
roles/ceph-osd/tasks/scenarios/dmcrypt-dedicated-journal.yml
roles/ceph-osd/tasks/scenarios/dmcrypt-journal-colocation.yml
roles/ceph-osd/tasks/scenarios/journal_collocation.yml
roles/ceph-osd/tasks/scenarios/raw_multi_journal.yml

index 4219e005ffed66c39f90d5530d610f7ac80910aa..542b9516732c7f09056b34edb2b6aaeb25c91e7e 100644 (file)
@@ -16,8 +16,8 @@
 - name: activate osd(s) when device is a disk
   command: ceph-disk activate {{ item.1 | regex_replace('^(\/dev\/cciss\/c[0-9]{1}d[0-9]{1})$', '\\1p') }}1
   with_together:
-    - combined_ispartition_results.results
-    - devices
+    - "{{ ispartition_results.results }}"
+    - "{{ devices }}"
   changed_when: false
   failed_when: false
   register: activate_osd_disk
@@ -41,8 +41,8 @@
 - name: activate osd(s) when device is a disk (dmcrypt)
   command: ceph-disk activate --dmcrypt {{ item.1 | regex_replace('^(\/dev\/cciss\/c[0-9]{1}d[0-9]{1})$', '\\1p') }}1
   with_together:
-    - combined_ispartition_results.results
-    - devices
+    - "{{ ispartition_results.results }}"
+    - "{{ devices }}"
   changed_when: false
   failed_when: false
   register: activate_osd_disk_dmcrypt
     msg: "ceph-disk failed to create an OSD"
   when:
     " 'ceph-disk: Error: ceph osd create failed' in item.get('stderr', '') "
-  with_items: "{{combined_activate_osd_disk_results.results}}"
+  with_items: "{{ combined_activate_osd_disk_results.results }}"
 
 # NOTE (leseb): this task is for partitions because we don't explicitly use a partition.
 - name: activate osd(s) when device is a partition
   command: "ceph-disk activate {{ item.1 }}"
   with_together:
-    - combined_ispartition_results.results
-    - devices
+    - "{{ ispartition_results.results }}"
+    - "{{ devices }}"
   changed_when: false
   failed_when: false
   when:
   command: systemctl enable ceph-osd@{{ item }}
   changed_when: false
   failed_when: false
-  with_items: osd_id.stdout_lines
+  with_items: "{{ osd_id.stdout_lines }}"
   when:
     - use_systemd
     - is_after_hammer
     name: ceph-osd@{{ item }}
     state: started
     enabled: yes
-  with_items: osd_id.stdout_lines
+  with_items: "{{ osd_id.stdout_lines }}"
   changed_when: false
   when:
     - use_systemd
index 72dfa90e59ce284b332b56fb90fd12178f214e1c..4a07ea49a7563052d9595161afc3d004b6f80669 100644 (file)
 
 - include: ./check_devices_static.yml
   when: not osd_auto_discovery
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./check_devices_auto.yml
   when: osd_auto_discovery
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - name: check the partition status of the journal devices
   shell: "parted --script {{ item }} print > /dev/null 2>&1"
-  with_items: '{{ raw_journal_devices }}'
+  with_items: "{{ raw_journal_devices }}"
   changed_when: false
   failed_when: false
   register: journal_partition_status
@@ -27,8 +31,8 @@
 - name: fix partitions gpt header or labels of the journal devices
   shell: "sgdisk --zap-all --clear --mbrtogpt -- {{ item.1 }} || sgdisk --zap-all --clear --mbrtogpt -- {{ item.1 }}"
   with_together:
-    - '{{ journal_partition_status.results }}'
-    - '{{ raw_journal_devices }}'
+    - "{{ journal_partition_status.results }}"
+    - "{{ raw_journal_devices }}"
   changed_when: false
   when:
     - raw_multi_journal
index d44ae5e60cb1a142417acd2f34aa75d25882efa7..7d17a57d54732eb5d19eb1f27f021456f192223d 100644 (file)
@@ -1,10 +1,10 @@
 ---
 - name: check if the device is a partition (autodiscover disks)
   shell: "echo '/dev/{{ item.key }}' | egrep '/dev/([hsv]d[a-z]{1,2}|cciss/c[0-9]d[0-9]p|nvme[0-9]n[0-9]p)[0-9]{1,2}$'"
-  with_dict: '{{ ansible_devices }}'
+  with_dict: "{{ ansible_devices }}"
   changed_when: false
   failed_when: false
-  register: '{{ combined_ispartition_results }}'
+  register: "{{ ispartition_results }}"
   when:
     - ansible_devices is defined
     - item.value.removable == "0"
@@ -14,7 +14,7 @@
   with_dict: '{{ ansible_devices }}'
   changed_when: false
   failed_when: false
-  register: combined_osd_partition_status_results
+  register: osd_partition_status_results
   when:
     - journal_collocation or raw_multi_journal
     - ansible_devices is defined
@@ -24,8 +24,8 @@
 - name: fix partitions gpt header or labels of the osd disks (autodiscover disks)
   shell: "sgdisk --zap-all --clear --mbrtogpt -- '/dev/{{ item.0.item.key }}' || sgdisk --zap-all --clear --mbrtogpt -- '/dev/{{ item.0.item.key }}'"
   with_together:
-    - '{{ combined_osd_partition_status_results.results }}'
-    - '{{ ansible_devices }}'
+    - "{{ osd_partition_status_results.results }}"
+    - "{{ ansible_devices }}"
   changed_when: false
   when:
     - journal_collocation
 
 - name: check if a partition named 'ceph' exists (autodiscover disks)
   shell: "parted --script /dev/{{ item.key }} print | egrep -sq '^ 1.*ceph'"
-  with_dict: '{{ ansible_devices }}'
+  with_dict: "{{ ansible_devices }}"
   changed_when: false
   failed_when: false
-  register: combined_parted_results
+  register: parted_results
   when:
     - ansible_devices is defined
     - item.value.removable == "0"
index 1de6067d9aed2741661f0b914a73b54f5285bb27..23d2f3b34da4b7ea7b7bcad401ba0c3bbc8efa65 100644 (file)
@@ -1,17 +1,17 @@
 ---
 - name: check if the device is a partition
   shell: "echo '{{ item }}' | egrep '/dev/([hsv]d[a-z]{1,2}|cciss/c[0-9]d[0-9]p|nvme[0-9]n[0-9]p)[0-9]{1,2}$'"
-  with_items: '{{ devices }}'
+  with_items: "{{ devices }}"
   changed_when: false
   failed_when: false
-  register: combined_ispartition_results
+  register: ispartition_results
 
 - name: check the partition status of the osd disks
   shell: "parted --script {{ item }} print > /dev/null 2>&1"
-  with_items: '{{ devices }}'
+  with_items: "{{ devices }}"
   changed_when: false
   failed_when: false
-  register: combined_osd_partition_status_results
+  register: osd_partition_status_results
   when:
     - journal_collocation or raw_multi_journal
 
@@ -23,8 +23,8 @@
 - name: fix partitions gpt header or labels of the osd disks
   shell: "sgdisk --zap-all --clear --mbrtogpt -- {{ item.1 }} || sgdisk --zap-all --clear --mbrtogpt -- {{ item.1 }}"
   with_together:
-    - '{{ combined_osd_partition_status_results.results }}'
-    - '{{ devices }}'
+    - "{{ osd_partition_status_results.results }}"
+    - "{{ devices }}"
   changed_when: false
   when:
     - journal_collocation or raw_multi_journal
@@ -32,7 +32,7 @@
 
 - name: check if a partition named 'ceph' exists
   shell: "parted --script {{ item }} print | egrep -sq '^ 1.*ceph'"
-  with_items: '{{ devices }}'
+  with_items: "{{ devices }}"
   changed_when: false
   failed_when: false
-  register: combined_parted_results
+  register: parted_results
index d965a558c86cf7e86a01bc4621a85bbb9a67f095..c5769f1bb4b4682875b81a1658e0971806879644 100644 (file)
@@ -1,36 +1,52 @@
 ---
 - include: pre_requisite.yml
   when: not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/journal_collocation.yml
   when:
     - journal_collocation
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/raw_multi_journal.yml
   when:
     - raw_multi_journal
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/osd_directory.yml
   when:
     - osd_directory
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/bluestore.yml
   when:
     - osd_objectstore == 'bluestore'
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/dmcrypt-journal-colocation.yml
   when:
     - dmcrypt_journal_colocation
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./scenarios/dmcrypt-dedicated-journal.yml
   when:
     - dmcrypt_dedicated_journal
     - not osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
 
 - include: ./docker/main.yml
   when: osd_containerized_deployment
+  # Hard code this so we will skip the entire file instead of individual tasks (Default isn't Consistent)
+  static: False
index 21cdf565cf5d6af11085a2c9edf46a0ccae7af68..68ab01a615d936881272d00f788acb508e447a52 100644 (file)
@@ -23,8 +23,8 @@
 - name: manually prepare osd disk(s)
   command: ceph-disk prepare --bluestore --cluster "{{ cluster }}" "{{ item.2 }}"
   with_together:
-    - combined_parted_results.results
-    - combined_ispartition_results.results
+    - parted_results.results
+    - ispartition_results.results
     - devices
   when:
     - not item.0.get("skipped")
index effc39d98f663b1cf924239f2ff45c5678f7c61f..60acfd5b11afb46207096a8cee3a2418c0947339 100644 (file)
@@ -12,8 +12,8 @@
 - name: prepare osd disk(s)
   command: "ceph-disk prepare --dmcrypt --cluster {{ cluster }} {{ item.2 }} {{ item.3 }}"
   with_together:
-    - combined_parted_results.results
-    - combined_ispartition_results.results
+    - parted_results.results
+    - ispartition_results.results
     - devices
     - raw_journal_devices
   changed_when: false
index 534fd3c57ef5d133e6df3b25fd8c08dad13eb87f..8de1a252f46791e40d2765b5706758567251b133 100644 (file)
@@ -22,8 +22,8 @@
 - name: manually prepare osd disk(s) (dmcrypt)
   command: ceph-disk prepare --dmcrypt --cluster "{{ cluster }}" "{{ item.2 }}"
   with_together:
-    - combined_parted_results.results
-    - combined_ispartition_results.results
+    - parted_results.results
+    - ispartition_results.results
     - devices
   when:
     - not item.0.get("skipped")
index e6fa42c44b887e95f185e2d57d027d71f4142397..ba0fbb41f9c48d62bc4cc8950ae9b8cb66537903 100644 (file)
@@ -23,8 +23,8 @@
 - name: manually prepare osd disk(s)
   command: "ceph-disk prepare --cluster {{ cluster }} {{ item.2 }}"
   with_together:
-    - combined_parted_results.results
-    - combined_ispartition_results.results
+    - parted_results.results
+    - ispartition_results.results
     - devices
   when:
     - not item.0.get("skipped")
index 36b64cc2017474ae11a5f5e91037956d994ec8c8..e1c4fce8860b34de093814af524c6bb0407839a6 100644 (file)
 - name: prepare osd disk(s)
   command: "ceph-disk prepare --cluster {{ cluster }} {{ item.2 }} {{ item.3 }}"
   with_together:
-    - combined_parted_results.results
-    - combined_ispartition_results.results
-    - devices
-    - raw_journal_devices
+    - "{{ parted_results.results }}"
+    - "{{ ispartition_results.results }}"
+    - "{{ devices }}"
+    - "{{ raw_journal_devices }}"
   changed_when: false
   when:
     - not item.0.get("skipped")