all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Christian Ebner <c.ebner@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox-backup v2 5/6] bin: expose list buckets as proxmox-backup-manager s3 subcommand
Date: Wed, 30 Jul 2025 16:40:07 +0200	[thread overview]
Message-ID: <20250730144008.642429-10-c.ebner@proxmox.com> (raw)
In-Reply-To: <20250730144008.642429-1-c.ebner@proxmox.com>

Allows to list the buckets from the cli, mostly to be feature
compatible with the rest of the api endpoints for s3.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>
---
changes since version 1:
- no changes

 src/bin/proxmox_backup_manager/s3.rs | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/src/bin/proxmox_backup_manager/s3.rs b/src/bin/proxmox_backup_manager/s3.rs
index f660baf4a..a94371e09 100644
--- a/src/bin/proxmox_backup_manager/s3.rs
+++ b/src/bin/proxmox_backup_manager/s3.rs
@@ -1,5 +1,5 @@
 use proxmox_router::{cli::*, ApiHandler, RpcEnvironment};
-use proxmox_s3_client::{S3_BUCKET_NAME_SCHEMA, S3_CLIENT_ID_SCHEMA};
+use proxmox_s3_client::{S3BucketListItem, S3_BUCKET_NAME_SCHEMA, S3_CLIENT_ID_SCHEMA};
 use proxmox_schema::api;
 
 use proxmox_backup::api2;
@@ -35,6 +35,23 @@ async fn check(
     Ok(Value::Null)
 }
 
+#[api(
+    input: {
+        properties: {
+            "s3-endpoint-id": {
+                schema: S3_CLIENT_ID_SCHEMA,
+            },
+        },
+    },
+)]
+/// List buckets accessible by the given S3 client configuration
+async fn list_buckets(
+    s3_endpoint_id: String,
+    rpcenv: &mut dyn RpcEnvironment,
+) -> Result<Vec<S3BucketListItem>, Error> {
+    api2::config::s3::list_buckets(s3_endpoint_id, rpcenv).await
+}
+
 #[api(
     input: {
         properties: {
@@ -88,6 +105,12 @@ pub fn s3_commands() -> CommandLineInterface {
             CliCommand::new(&api2::config::s3::API_METHOD_DELETE_S3_CLIENT_CONFIG)
                 .arg_param(&["id"])
                 .completion_cb("id", pbs_config::s3::complete_s3_client_id),
+        )
+        .insert(
+            "list-buckets",
+            CliCommand::new(&API_METHOD_LIST_BUCKETS)
+                .arg_param(&["s3-endpoint-id"])
+                .completion_cb("s3-endpoint-id", pbs_config::s3::complete_s3_client_id),
         );
 
     let cmd_def = CliCommandMap::new()
-- 
2.47.2



_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


  parent reply	other threads:[~2025-07-30 14:39 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-30 14:39 [pbs-devel] [PATCH proxmox{, -backup} v2 00/10] s3: implement list buckets and use bucket selector for datastore creation Christian Ebner
2025-07-30 14:39 ` [pbs-devel] [PATCH proxmox v2 1/4] s3 client: restrict bucket template pattern to start of endpoint url Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox v2 2/4] s3 client: make bucket name optional in S3 client options Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox v2 3/4] s3 client: implement list buckets method Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox v2 4/4] s3 client: api types: add bucket list item type Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox-backup v2 1/6] ui: fix formatting issues using proxmox-biome Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox-backup v2 2/6] datastore: wrap bucket name, as in is now optional in the s3 client Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox-backup v2 3/6] api: admin s3: make store prefix for check command optional Christian Ebner
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox-backup v2 4/6] api: config s3: add bucket list api endpoint Christian Ebner
2025-07-30 14:40 ` Christian Ebner [this message]
2025-07-30 14:40 ` [pbs-devel] [PATCH proxmox-backup v2 6/6] ui: replace bucket field by bucket selector Christian Ebner
2025-07-31  9:28 ` [pbs-devel] [PATCH proxmox{, -backup} v2 00/10] s3: implement list buckets and use bucket selector for datastore creation Lukas Wagner
2025-07-31 10:14   ` Christian Ebner
2025-07-31 10:49 ` Christian Ebner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250730144008.642429-10-c.ebner@proxmox.com \
    --to=c.ebner@proxmox.com \
    --cc=pbs-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal