Update patch set 5

Patch Set 5: Code-Review-1 (2 comments) Patch-set: 5 Reviewer: Gerrit User 6816 <6816@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Label: Code-Review=-1
2016-08-23 15:22:19 +00:00 · 2016-08-23 15:22:19 +00:00 · bd6f62acd8
parent a0ac7ee333
commit bd6f62acd8
2 changed files with 48 additions and 0 deletions
--- a/21
+++ b/21
@ -0,0 +1,21 @@
+{
+  "comments": [
+    {
+      "key": {
+        "uuid": "1ac06dbe_84e0a4ce",
+        "filename": "vars/debian.yml",
+        "patchSetId": 5
+      },
+      "lineNbr": 0,
+      "author": {
+        "id": 6816
+      },
+      "writtenOn": "2016-08-23T15:22:19Z",
+      "side": 1,
+      "message": "Does Watcher have absolutely no binary deps? It would seem to me that this change should be in a subsequent patch.",
+      "revId": "7367945b3a7b1312aff198602671ac1dc1eacc2d",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    }
+  ]
+}
--- a/27
+++ b/27
@ -0,0 +1,27 @@
+{
+  "comments": [
+    {
+      "key": {
+        "uuid": "1ac06dbe_0424b45b",
+        "filename": "tasks/watcher_install.yml",
+        "patchSetId": 5
+      },
+      "lineNbr": 18,
+      "author": {
+        "id": 6816
+      },
+      "writtenOn": "2016-08-23T15:22:19Z",
+      "side": 0,
+      "message": "Does this belong here? I\u0027m not sure what this has to do with making the CI tests work?",
+      "range": {
+        "startLine": 16,
+        "startChar": 0,
+        "endLine": 18,
+        "endChar": 30
+      },
+      "revId": "76a013f8145c5f18766b561f426e22315f22836d",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
+      "unresolved": false
+    }
+  ]
+}