api_server: renamed OkStatus::Updated to NoContent

This status code is going to be used also on create, when we
don't need to return a body.

Signed-off-by: Andreea Florescu <[email protected]>

Author: andreeaflorescu

api_server/src/request/mod.rs

@@ -88,7 +88,7 @@ impl fmt::Debug for SyncRequest {
 pub enum OkStatus {
     Created,
     Ok,
-    Updated,
+    NoContent,
 }
 
 impl GenerateResponse for OkStatus {
@@ -97,7 +97,7 @@ impl GenerateResponse for OkStatus {
         match *self {
             Created => empty_response(StatusCode::Created),
             Ok => empty_response(StatusCode::Ok),
-            Updated => empty_response(StatusCode::NoContent),
+            NoContent => empty_response(StatusCode::NoContent),
         }
     }
 }
@@ -279,7 +279,7 @@ mod tests {
         let mut ret = OkStatus::Created.generate_response();
         assert_eq!(ret.status(), StatusCode::Created);
 
-        ret = OkStatus::Updated.generate_response();
+        ret = OkStatus::NoContent.generate_response();
         assert_eq!(ret.status(), StatusCode::NoContent);
     }
 

vmm/src/device_config/net.rs

@@ -109,7 +109,7 @@ impl NetworkInterfaceConfigs {
             self.validate_unique_mac(&body.guest_mac)?;
             self.if_list[index].update_from_body(body);
         }
-        Ok(SyncOkStatus::Updated)
+        Ok(SyncOkStatus::NoContent)
     }
 
     fn validate_unique_mac(&self, mac: &Option<MacAddr>) -> result::Result<(), SyncError> {

vmm/src/lib.rs

@@ -725,7 +725,7 @@ impl Vmm {
                             }
                             return device_manager
                                 .update_drive(address, new_size)
-                                .map(|_| SyncOkStatus::Updated)
+                                .map(|_| SyncOkStatus::NoContent)
                                 .map_err(|_| {
                                     SyncError::DriveOperationFailed(
                                         DriveError::BlockDeviceUpdateFailed,
@@ -1742,7 +1742,7 @@ mod tests {
                 allow_mmds_requests: false,
             };
             match vmm.put_net_device(network_interface) {
-                Ok(outcome) => assert!(outcome == SyncOkStatus::Updated),
+                Ok(outcome) => assert!(outcome == SyncOkStatus::NoContent),
                 Err(_) => assert!(false),
             }
         }