Re: [pbs-devel] [PATCH v3 proxmox-backup 14/14] remove old verification scheduling from proxmox-backup-proxy.rs

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

On 13.10.20 11:33, Hannes Laimer wrote:
> Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
> ---
>  src/bin/proxmox-backup-proxy.rs | 115 --------------------------------
>  1 file changed, 115 deletions(-)
> 
> diff --git a/src/bin/proxmox-backup-proxy.rs b/src/bin/proxmox-backup-proxy.rs
> index e8047ecc..7993247c 100644
> --- a/src/bin/proxmox-backup-proxy.rs
> +++ b/src/bin/proxmox-backup-proxy.rs
> @@ -197,7 +197,6 @@ async fn schedule_tasks() -> Result<(), Error> {
>  
>      schedule_datastore_garbage_collection().await;
>      schedule_datastore_prune().await;
> -    schedule_datastore_verification().await;
>      schedule_datastore_sync_jobs().await;
>      schedule_datastore_verify_jobs().await;
>      schedule_task_log_rotate().await;
> @@ -471,120 +470,6 @@ async fn schedule_datastore_prune() {
>      }
>  }
>  
> -async fn schedule_datastore_verification() {
> -    use proxmox_backup::backup::{DataStore, verify_all_backups};
> -    use proxmox_backup::server::{WorkerTask};

would also need a rebase, but see my other replies.

Also, please do *not* but source code file endings in the commit message or subject,
makes no sense and just uglier/harder to read. This one should have rather been:

> proxmox-backup-proxy: remove old verification scheduling

or 

> api proxy: remove old verification scheduling