From c7ccf1d324e057e7fe959ee1967f90e5925df0f5 Mon Sep 17 00:00:00 2001
From: Gerrit User 6159 <6159@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Date: Wed, 21 Mar 2018 09:46:57 +0000
Subject: [PATCH] Update patch set 4

Patch Set 4:

(5 comments)

Patch-set: 4
Reviewer: Gerrit User 6159 <6159@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Verified=0
---
 82796da846dd4332d83296d212637d9aa879882c | 89 ++++++++++++++++++++++++
 1 file changed, 89 insertions(+)
 create mode 100644 82796da846dd4332d83296d212637d9aa879882c

diff --git a/82796da846dd4332d83296d212637d9aa879882c b/82796da846dd4332d83296d212637d9aa879882c
new file mode 100644
index 0000000..17d79eb
--- /dev/null
+++ b/82796da846dd4332d83296d212637d9aa879882c
@@ -0,0 +1,89 @@
+{
+  "comments": [
+    {
+      "key": {
+        "uuid": "df7087c5_719ba565",
+        "filename": "defaults/main.yml",
+        "patchSetId": 4
+      },
+      "lineNbr": 9,
+      "author": {
+        "id": 6159
+      },
+      "writtenOn": "2018-03-21T09:46:57Z",
+      "side": 1,
+      "message": "why do you need this variable and the one above?\n\nWe should be able to do everything with what\u0027s in the cinder_config dict. right?",
+      "revId": "82796da846dd4332d83296d212637d9aa879882c",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "df7087c5_f1c33556",
+        "filename": "defaults/main.yml",
+        "patchSetId": 4
+      },
+      "lineNbr": 35,
+      "author": {
+        "id": 6159
+      },
+      "writtenOn": "2018-03-21T09:46:57Z",
+      "side": 1,
+      "message": "This dict seems to be duplicated",
+      "revId": "82796da846dd4332d83296d212637d9aa879882c",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "df7087c5_51e6a1ea",
+        "filename": "tasks/provision.yml",
+        "patchSetId": 4
+      },
+      "lineNbr": 1,
+      "author": {
+        "id": 6159
+      },
+      "writtenOn": "2018-03-21T09:46:57Z",
+      "side": 1,
+      "message": "should be three - AFAIK",
+      "revId": "82796da846dd4332d83296d212637d9aa879882c",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "df7087c5_11f9097f",
+        "filename": "tasks/provision.yml",
+        "patchSetId": 4
+      },
+      "lineNbr": 6,
+      "author": {
+        "id": 6159
+      },
+      "writtenOn": "2018-03-21T09:46:57Z",
+      "side": 1,
+      "message": "Why are you moving to `template` instead of using `config_template`?\n\nI think it\u0027d be better to stick with config_template",
+      "revId": "82796da846dd4332d83296d212637d9aa879882c",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    },
+    {
+      "key": {
+        "uuid": "df7087c5_b1085d91",
+        "filename": "templates/cinder.conf.j2",
+        "patchSetId": 4
+      },
+      "lineNbr": 23,
+      "author": {
+        "id": 6159
+      },
+      "writtenOn": "2018-03-21T09:46:57Z",
+      "side": 1,
+      "message": "the connection info is in the `cinder_config` dict. We should probably use that one.",
+      "revId": "82796da846dd4332d83296d212637d9aa879882c",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    }
+  ]
+}
\ No newline at end of file