From 79c1b729f60fc0f78f67751e16f85a66bccf1fda Mon Sep 17 00:00:00 2001
From: Gerrit User 782 <782@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Date: Thu, 25 May 2023 08:44:14 +0000
Subject: [PATCH] Update patch set 1

Patch Set 1:

(5 comments)

Patch-set: 1
---
 c82bd80382b4db4da01a41b2a6a618fa1f691ada | 89 ++++++++++++++++++++++++
 1 file changed, 89 insertions(+)
 create mode 100644 c82bd80382b4db4da01a41b2a6a618fa1f691ada

diff --git a/c82bd80382b4db4da01a41b2a6a618fa1f691ada b/c82bd80382b4db4da01a41b2a6a618fa1f691ada
new file mode 100644
index 0000000000..58d28ddb77
--- /dev/null
+++ b/c82bd80382b4db4da01a41b2a6a618fa1f691ada
@@ -0,0 +1,89 @@
+{
+  "comments": [
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "14df7531_063cd3ae",
+        "filename": "magnum/drivers/cluster_api/driver.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 86,
+      "author": {
+        "id": 782
+      },
+      "writtenOn": "2023-05-25T08:44:14Z",
+      "side": 1,
+      "message": "let’s move this into the base patch?",
+      "revId": "c82bd80382b4db4da01a41b2a6a618fa1f691ada",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "65e6ce70_2309a681",
+        "filename": "magnum/drivers/cluster_api/driver.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 416,
+      "author": {
+        "id": 782
+      },
+      "writtenOn": "2023-05-25T08:44:14Z",
+      "side": 1,
+      "message": "I would rather we make template a required arg to keep it simple an explicit?",
+      "revId": "c82bd80382b4db4da01a41b2a6a618fa1f691ada",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "6c3f41fc_16900086",
+        "filename": "magnum/drivers/cluster_api/driver.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 430,
+      "author": {
+        "id": 782
+      },
+      "writtenOn": "2023-05-25T08:44:14Z",
+      "side": 1,
+      "message": "as above, make it required I think",
+      "revId": "c82bd80382b4db4da01a41b2a6a618fa1f691ada",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "d0b9158e_ad956135",
+        "filename": "magnum/drivers/cluster_api/driver.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 738,
+      "author": {
+        "id": 782
+      },
+      "writtenOn": "2023-05-25T08:44:14Z",
+      "side": 1,
+      "message": "this seems very important! clearly should happen before we update the template id",
+      "revId": "c82bd80382b4db4da01a41b2a6a618fa1f691ada",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "4cfdb553_a2bb2f0e",
+        "filename": "magnum/drivers/cluster_api/driver.py",
+        "patchSetId": 1
+      },
+      "lineNbr": 748,
+      "author": {
+        "id": 782
+      },
+      "writtenOn": "2023-05-25T08:44:14Z",
+      "side": 1,
+      "message": "why not do a save here, then a refresh() we should be able to just use cluster_template everywhere then?\n\n… actually let’s update the state I guess, I know it duplicates the conductor, but that seems worth it?",
+      "revId": "c82bd80382b4db4da01a41b2a6a618fa1f691ada",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    }
+  ]
+}
\ No newline at end of file