feat: Mark VM as deleted

Mark the VM in the spec as deleted and let the reconciliation loop handle it. For mocking purpose, I introduced a Clock in ports.Collection. Changed the event to Update event from Delete event. For the reconciliation loop, they are the same. Let me know if we still want to use the Delete event. fixes #74
liquidmetal-dev · Oct 8, 2021 · dca8227 · dca8227
1 parent 7b5ff28
commit dca8227
Show file tree

Hide file tree

Showing 16 changed files with 257 additions and 167 deletions.
diff --git a/api/services/microvm/v1alpha1/microvms.pb.go b/api/services/microvm/v1alpha1/microvms.pb.go
diff --git a/api/services/microvm/v1alpha1/microvms.pb.gw.go b/api/services/microvm/v1alpha1/microvms.pb.gw.go
diff --git a/api/services/microvm/v1alpha1/microvms.proto b/api/services/microvm/v1alpha1/microvms.proto
@@ -93,4 +93,4 @@ message ListMicroVMsResponse {
 
 message ListMessage {
   reignite.types.MicroVMSpec  microvm = 1;
-}
+}
diff --git a/api/services/microvm/v1alpha1/microvms.swagger.json b/api/services/microvm/v1alpha1/microvms.swagger.json
@@ -323,6 +323,11 @@
           "type": "string",
           "format": "date-time",
           "description": "UpdatedAt indicates the time the microvm was last updated."
+        },
+        "deletedAt": {
+          "type": "string",
+          "format": "date-time",
+          "description": "DeletedAt indicates the time the microvm was marked as deleted."
         }
       },
       "description": "MicroVMSpec represents the specification for a microvm."

diff --git a/api/services/microvm/v1alpha1/microvms_grpc.pb.go b/api/services/microvm/v1alpha1/microvms_grpc.pb.go
-Original file line number
+Diff line change
@@ Expand Up / @@ -93,4 +93,4 @@ message ListMicroVMsResponse { @@
     message ListMessage {
       reignite.types.MicroVMSpec  microvm = 1;
-    }
+    }