From e070db531fdc65118e1ce3af6d7043e1a3f94c69 Mon Sep 17 00:00:00 2001
From: Giulio Fidente <gfidente@redhat.com>
Date: Wed, 6 Feb 2019 17:20:10 +0100
Subject: [PATCH] Update NodeDataLookup examples to use JSON instead of strings

The changes introduced to resolve bug 1814070 assume NodeDataLookup
to be a real JSON object instead of a string. This updates our
examples in the docs.

Change-Id: I116fd249b4330ac9992428a1c77409a7376a1d73
---
 .../advanced_deployment/ceph_config.rst       | 46 ++++++++-----------
 .../node_specific_hieradata.rst               |  6 +--
 2 files changed, 21 insertions(+), 31 deletions(-)

diff --git a/doc/source/install/advanced_deployment/ceph_config.rst b/doc/source/install/advanced_deployment/ceph_config.rst
index d798d584..3351d882 100644
--- a/doc/source/install/advanced_deployment/ceph_config.rst
+++ b/doc/source/install/advanced_deployment/ceph_config.rst
@@ -322,33 +322,25 @@ An example of what the `~/my-node-settings.yaml` file, as described in
 will keep using filestore is the following::
 
   parameter_defaults:
-    NodeDataLookup: |
-    {
-      "00000000-0000-0000-0000-0CC47A6EFDCC": {
-        "devices": [
-          "/dev/sdb",
-          "/dev/sdc"
-        ],
-        "dedicated_devices": [
-          "/dev/nvme0n1",
-          "/dev/nvme0n1"
-        ],
-        "osd_scenario": "non-collocated",
-        "osd_objectstore": "filestore"
-      },
-      "00000000-0000-0000-0000-0CC47A6F13FF": {
-        "devices": [
-          "/dev/sdb",
-          "/dev/sdc"
-        ],
-        "dedicated_devices": [
-          "/dev/nvme0n1",
-          "/dev/nvme0n1"
-        ],
-        "osd_scenario": "non-collocated",
-        "osd_objectstore": "filestore"
-      }
-    }
+    NodeDataLookup:
+      00000000-0000-0000-0000-0CC47A6EFDCC:
+        devices:
+          - /dev/sdb
+          - /dev/sdc
+        dedicated_devices:
+          - /dev/nvme0n1
+          - /dev/nvme0n1
+        osd_scenario: non-collocated
+        osd_objectstore: filestore
+      00000000-0000-0000-0000-0CC47A6F13FF:
+        devices:
+          - /dev/sdb
+          - /dev/sdc
+        dedicated_devices:
+          - /dev/nvme0n1
+          - /dev/nvme0n1
+        osd_scenario: non-collocated
+        osd_objectstore: filestore
 
 Be sure to set every existing Ceph filestore server to the filestore
 parameters by its machine unique UUID. If the above is not done and
diff --git a/doc/source/install/advanced_deployment/node_specific_hieradata.rst b/doc/source/install/advanced_deployment/node_specific_hieradata.rst
index 39795c34..e890315d 100644
--- a/doc/source/install/advanced_deployment/node_specific_hieradata.rst
+++ b/doc/source/install/advanced_deployment/node_specific_hieradata.rst
@@ -44,8 +44,7 @@ or puppet-ceph (Ocata and older).
 For ceph-ansible use::
 
   parameter_defaults:
-    NodeDataLookup: |
-      {"32e87b4c-c4a7-41be-865b-191684a6883b": {"devices": ["/dev/sdc"]}}
+    NodeDataLookup: {"32e87b4c-c4a7-41be-865b-191684a6883b": {"devices": ["/dev/sdc"]}}
 
 For puppet-ceph use::
 
@@ -53,8 +52,7 @@ For puppet-ceph use::
     OS::TripleO::CephStorageExtraConfigPre: /path/to/tripleo-heat-templates/puppet/extraconfig/pre_deploy/per_node.yaml
 
   parameter_defaults:
-    NodeDataLookup: |
-      {"32e87b4c-c4a7-41be-865b-191684a6883b": {"ceph::profile::params::osds": {"/dev/sdc": {}}}}
+    NodeDataLookup: {"32e87b4c-c4a7-41be-865b-191684a6883b": {"ceph::profile::params::osds": {"/dev/sdc": {}}}}
 
 In the above example we're customizing only a single key for a single node, but
 the structure is that of a UUID-mapped hash so it is possible to customize