[pbs-devel] [PATCH proxmox-backup v8 38/45] ui: expose s3 refresh button for datastores backed by object store

[Lukas Wagner]

On  2025-07-15 14:53, Christian Ebner wrote:
> Allows to trigger a refresh of the local datastore contents from
> the WebUI.
> 
> Signed-off-by: Christian Ebner <c.ebner at proxmox.com>
> ---
> changes since version 7:
> - no changes
> 
>  www/datastore/Summary.js | 44 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 44 insertions(+)
> 
> diff --git a/www/datastore/Summary.js b/www/datastore/Summary.js
> index cdb34aea3..d8f59ebc5 100644
> --- a/www/datastore/Summary.js
> +++ b/www/datastore/Summary.js
> @@ -301,6 +301,31 @@ Ext.define('PBS.DataStoreSummary', {
>                  });
>              },
>          },
> +        {
> +            xtype: 'button',
> +            text: gettext('S3 Refresh'),
> +            hidden: true,
> +            itemId: 's3RefreshButton',
> +            reference: 's3RefreshButton',
> +            handler: function () {
> +                let me = this;
> +                let datastore = me.up('panel').datastore;
> +                Proxmox.Utils.API2Request({
> +                    url: `/admin/datastore/${datastore}/s3-refresh`,
> +                    method: 'PUT',
> +                    failure: (response) => Ext.Msg.alert(gettext('Error'), response.htmlStatus),
> +                    success: function (response, options) {
> +                        Ext.create('Proxmox.window.TaskViewer', {
> +                            upid: response.result.data,
> +                            taskDone: () => {
> +                                me.up('panel').statusStore.load();
> +                                Ext.ComponentQuery.query('navigationtree')[0]?.reloadStore();
> +                            },
> +                        }).show();
> +                    },
> +                });
> +            },
> +        },

Already discussed off-list:

The "S3 Refresh" button should in my opinion be not placed as prominently as it is now.
In normal operation, a user should not need it. I'd maybe create some drop-down menu
in the "Content" page and use also a bit more text, e.g. "Refresh backup snapshots from S3 bucket"
or the like.


-- 
- Lukas