[pve-devel] [PATCH pve-network v2 1/1] sdn: api: rename parameter from skip-frr to regenerate-frr
Stefan Hanreich
s.hanreich at proxmox.com
Mon Aug 4 16:01:49 CEST 2025
Parameter has moved from skip_frr to regenerate-frr in pve-manager.
Also invert the check for generating the boolean, since the logic is
inverted now.
Signed-off-by: Stefan Hanreich <s.hanreich at proxmox.com>
---
src/PVE/API2/Network/SDN.pm | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/PVE/API2/Network/SDN.pm b/src/PVE/API2/Network/SDN.pm
index 7c91902..16a8e2c 100644
--- a/src/PVE/API2/Network/SDN.pm
+++ b/src/PVE/API2/Network/SDN.pm
@@ -90,12 +90,10 @@ __PACKAGE__->register_method({
});
my $create_reload_network_worker = sub {
- my ($nodename, $skip_frr) = @_;
+ my ($nodename, $regenerate_frr) = @_;
my @command = ('pvesh', 'set', "/nodes/$nodename/network");
- if ($skip_frr) {
- push(@command, '--skip_frr');
- }
+ push(@command, '--regenerate-frr') if $regenerate_frr;
# FIXME: how to proxy to final node ?
my $upid;
@@ -301,14 +299,14 @@ __PACKAGE__->register_method({
$lock_token,
);
- my $skip_frr = !($previous_config_has_frr || $new_config_has_frr);
+ my $regenerate_frr = $previous_config_has_frr || $new_config_has_frr;
my $code = sub {
$rpcenv->{type} = 'priv'; # to start tasks in background
PVE::Cluster::check_cfs_quorum();
my $nodelist = PVE::Cluster::get_nodelist();
for my $node (@$nodelist) {
- my $pid = eval { $create_reload_network_worker->($node, $skip_frr) };
+ my $pid = eval { $create_reload_network_worker->($node, $regenerate_frr) };
warn $@ if $@;
}
--
2.47.2
More information about the pve-devel
mailing list