rgw/sfs: handle result from metadata update

Signed-off-by: Joao Eduardo Luis <[email protected]>
aquarist-labs · Nov 23, 2023 · 15dba00 · 15dba00
1 parent ec5fa45
commit 15dba00
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/rgw/driver/sfs/multipart.cc b/src/rgw/driver/sfs/multipart.cc
@@ -510,7 +510,9 @@ int SFSMultipartUploadV2::complete(
        .delete_at = ceph::real_time()}
   );
   try {
-    objref->metadata_finish(store, bucketref->get_info().versioning_enabled());
+    res = objref->metadata_finish(
+        store, bucketref->get_info().versioning_enabled()
+    );
   } catch (const std::system_error& e) {
     lsfs_err(dpp) << fmt::format(
                          "failed to update db object {}: {}", objref->name,
@@ -519,6 +521,11 @@ int SFSMultipartUploadV2::complete(
                   << dendl;
     return -ERR_INTERNAL_ERROR;
   }
+  if (!res) {
+    lsfs_err(dpp) << fmt::format("failed to update db object {}", objref->name)
+                  << dendl;
+    return -ERR_INTERNAL_ERROR;
+  }
 
   // mark multipart upload done
   res = mpdb.mark_done(upload_id);