[pbs-devel] [RFC v2 proxmox-backup 31/42] datastore: get and set owner for S3 store backend

Christian Ebner c.ebner at proxmox.com
Thu May 29 16:31:56 CEST 2025


Read or write the ownership information from/to the corresponding
object in the S3 object store. Keep that information available if
the bucket is reused as datastore.

Signed-off-by: Christian Ebner <c.ebner at proxmox.com>
---
 pbs-datastore/src/datastore.rs | 39 ++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/pbs-datastore/src/datastore.rs b/pbs-datastore/src/datastore.rs
index d016e2139..52ec8218e 100644
--- a/pbs-datastore/src/datastore.rs
+++ b/pbs-datastore/src/datastore.rs
@@ -816,6 +816,25 @@ impl DataStore {
         backup_group: &pbs_api_types::BackupGroup,
     ) -> Result<Authid, Error> {
         let full_path = self.owner_path(ns, backup_group);
+
+        if let DatastoreBackend::S3(s3_client) = self.backend()? {
+            let object_key = format!(
+                "{}/{backup_group}/owner",
+                ns.path()
+                    .to_str()
+                    .ok_or_else(|| format_err!("unexpected owner path"))?,
+            );
+            let response =
+                proxmox_async::runtime::block_on(s3_client.get_object(object_key.as_str().into()))?
+                    .ok_or_else(|| format_err!("fetching owner failed"))?;
+            let content =
+                proxmox_async::runtime::block_on(hyper::body::HttpBody::collect(response.content))?;
+            let owner = String::from_utf8(content.to_bytes().trim_ascii_end().to_vec())?;
+            return owner
+                .parse()
+                .map_err(|err| format_err!("parsing owner for {backup_group} failed: {err}"));
+        }
+
         let owner = proxmox_sys::fs::file_read_firstline(full_path)?;
         owner
             .trim_end() // remove trailing newline
@@ -844,6 +863,26 @@ impl DataStore {
     ) -> Result<(), Error> {
         let path = self.owner_path(ns, backup_group);
 
+        if let DatastoreBackend::S3(s3_client) = self.backend()? {
+            let object_key = format!(
+                "{}/{backup_group}/owner",
+                ns.path()
+                    .to_str()
+                    .ok_or_else(|| format_err!("unexpected owner path"))?,
+            );
+            let data = hyper::body::Body::from(format!("{auth_id}\n"));
+            let response = proxmox_async::runtime::block_on(
+                s3_client.put_object(object_key.as_str().into(), data),
+            )?;
+            match response {
+                PutObjectResponse::NeedsRetry => bail!("failed to set owner, needs retry"),
+                PutObjectResponse::PreconditionFailed => {
+                    bail!("failed to set owner, precondition failed")
+                }
+                PutObjectResponse::Success(_) => (),
+            }
+        }
+
         let mut open_options = std::fs::OpenOptions::new();
         open_options.write(true);
 
-- 
2.39.5





More information about the pbs-devel mailing list