Merge pull request #222 from weaveworks/refactor/remove-update

README.md

@@ -15,7 +15,7 @@ The primary use case for flintlock is to create microVMs on a bare-metal host wh
 
 Using API requests (via gRPC or HTTP):
 
-- Create, update, delete microVMs using Firecracker
+- Create and delete microVMs using Firecracker
 - Manage the lifecycle of microVMs (i.e. start, stop, pause)
 - Configure microVM metadata via cloud-init, ignition etc
 - Use OCI images for microVM volumes, kernel and initrd

api/services/microvm/v1alpha1/microvms.proto

@@ -28,11 +28,6 @@ service MicroVM {
       post: "/v1alpha1/microvm"
     };
   }
-  rpc UpdateMicroVM(UpdateMicroVMRequest) returns (UpdateMicroVMResponse) {
-    option (google.api.http) = {
-      put: "/v1alpha1/microvm/{microvm.namespace}/{microvm.id}"
-    };
-  }
   rpc DeleteMicroVM(DeleteMicroVMRequest) returns (google.protobuf.Empty) {
     option (google.api.http) = {
       delete: "/v1alpha1/microvm/{namespace}/{id}"
@@ -65,15 +60,6 @@ message DeleteMicroVMRequest {
   string namespace = 2;
 }
 
-message UpdateMicroVMRequest {
-  flintlock.types.MicroVMSpec  microvm = 1;
-  google.protobuf.FieldMask update_mask = 2;
-}
-
-message UpdateMicroVMResponse {
-  flintlock.types.MicroVMSpec  microvm = 1;
-}
-
 message GetMicroVMRequest {
   string id = 1;
   string namespace = 2;

api/services/microvm/v1alpha1/microvms.swagger.json

@@ -39,44 +39,6 @@
         ]
       }
     },
-    "/v1alpha1/microvm/{microvm.namespace}/{microvm.id}": {
-      "put": {
-        "operationId": "MicroVM_UpdateMicroVM",
-        "responses": {
-          "200": {
-            "description": "A successful response.",
-            "schema": {
-              "$ref": "#/definitions/v1alpha1UpdateMicroVMResponse"
-            }
-          },
-          "default": {
-            "description": "An unexpected error response.",
-            "schema": {
-              "$ref": "#/definitions/rpcStatus"
-            }
-          }
-        },
-        "parameters": [
-          {
-            "name": "microvm.namespace",
-            "description": "Namespace is the name of the namespace the microvm belongs to.",
-            "in": "path",
-            "required": true,
-            "type": "string"
-          },
-          {
-            "name": "microvm.id",
-            "description": "ID is the identifier of the microvm.\nIf this empty at creation time a ID will be automatically generated.",
-            "in": "path",
-            "required": true,
-            "type": "string"
-          }
-        ],
-        "tags": [
-          "MicroVM"
-        ]
-      }
-    },
     "/v1alpha1/microvm/{namespace}": {
       "get": {
         "operationId": "MicroVM_ListMicroVMs",
@@ -549,14 +511,6 @@
           }
         }
       }
-    },
-    "v1alpha1UpdateMicroVMResponse": {
-      "type": "object",
-      "properties": {
-        "microvm": {
-          "$ref": "#/definitions/typesMicroVMSpec"
-        }
-      }
     }
   }
 }