Re: [pbs-devel] [PATCH proxmox 1/1] pbs-api-types: jobs: verify: add worker-threads to VerificationJobConfig

[Christian Ebner <c.ebner@proxmox.com>]

Please mention in a short commit message what this new parameter will be 
used for, e.g. controls the number of parallel workers to read and 
verify chunks.

On 11/5/25 4:51 PM, Nicolas Frey wrote:
> Signed-off-by: Nicolas Frey <n.frey@proxmox.com>
> ---
>   pbs-api-types/src/jobs.rs | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/pbs-api-types/src/jobs.rs b/pbs-api-types/src/jobs.rs
> index 4dbbef2b..d904f797 100644
> --- a/pbs-api-types/src/jobs.rs
> +++ b/pbs-api-types/src/jobs.rs
> @@ -203,6 +203,13 @@ pub const VERIFICATION_OUTDATED_AFTER_SCHEMA: Schema =
>               optional: true,
>               schema: crate::NS_MAX_DEPTH_SCHEMA,
>           },
> +        "worker-threads": {
> +            type: Integer,
> +            optional: true,
> +            minimum: 1,
> +            maximum: 32,
> +            default: 1,
> +        },

Please define this as dedicated schema, so it can be reused for the PBS 
api as well (as exposed in patch 2).

>       }
>   )]
>   #[derive(Serialize, Deserialize, Updater, Clone, PartialEq)]
> @@ -221,6 +228,9 @@ pub struct VerificationJobConfig {
>       #[serde(skip_serializing_if = "Option::is_none")]
>       /// Reverify snapshots after X days, never if 0. Ignored if 'ignore_verified' is false.
>       pub outdated_after: Option<i64>,
> +    /// Set the number of worker threads to use for the job
> +    #[serde(skip_serializing_if = "Option::is_none")]
> +    pub worker_threads: Option<usize>,
>       #[serde(skip_serializing_if = "Option::is_none")]
>       pub comment: Option<String>,
>       #[serde(skip_serializing_if = "Option::is_none")]



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