Re: [PATCH cluster v2 6/8] api: add token timeout and warning level to cluster join info

["Lukas Sichert" <l.sichert@proxmox.com>]

On 2026-04-20 18:43, [=?utf-8?q?Michael_K=C3=B6ppl?= <m.koeppl@proxmox.com>] wrote:

> The token timeout in seconds and the warning level provide additional
> information for users regarding the expected token timeout in seconds
> after adding an additional node and whether changing the token
> coefficient is recommended.
>
> Signed-off-by: Michael Köppl <m.koeppl@proxmox.com>
> ---
>  src/PVE/API2/ClusterConfig.pm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/src/PVE/API2/ClusterConfig.pm b/src/PVE/API2/ClusterConfig.pm
> index bbed40e..5739e4c 100644
> --- a/src/PVE/API2/ClusterConfig.pm
> +++ b/src/PVE/API2/ClusterConfig.pm
> @@ -571,6 +571,18 @@ __PACKAGE__->register_method({
>              preferred_node => get_standard_option('pve-node'),
>              totem => { type => 'object' },
>              config_digest => { type => 'string' },
> +            expected_timeout => {
> +                type => 'number',
> +                description =>
> +                    "Expected total timeout (in seconds) if an additional node is added.",
> +                optional => 1,
> +            },
> +            timeout_warning_level => {
> +                type => 'string',
> +                description => "Warning level for the expected total timeout.",
> +                optional => 1,
> +                enum => ['optimize', 'change-recommended', 'change-strongly-recommended'],
> +            },
>          },
>      },

As discussed in [1] new API variables use a dash instead of an
underscore. In this file and as far as I have seen in the complete repo
this stye has not been used yet. Maybe somebody with more knowledge
about the styleguide can give their opinion here. 

>      code => sub {
> @@ -599,12 +611,22 @@ __PACKAGE__->register_method({
>              $node->{pve_addr} = scalar(PVE::Cluster::remote_node_ip($name));
>          }
>  
> +        # Total timeout if additional node is added
> +        my $total_timeout_secs = PVE::Corosync::calculate_membership_recovery_timeout(
> +            $totem_cfg,
> +            scalar(keys %$nodelist) + 1,
> +        );
> +
> +        my $warning_level = PVE::Corosync::get_timeout_warning_level($total_timeout_secs);
> +
>          my $res = {
>              nodelist => [values %$nodelist],
>              preferred_node => $nodename,
>              totem => $totem_cfg,
>              config_digest => $corosync_config_digest,
> +            expected_timeout => $total_timeout_secs,
>          };
> +        $res->{timeout_warning_level} = $warning_level if defined($warning_level);
>  
>          return $res;
>      },
> -- 
> 2.47.3
[1] https://lore.proxmox.com/all/DHA3CMV4VBMI.14WCK95P7U5AW@proxmox.com/