We must know if the target and guest exists to avoid creation faulty jobs. --- PVE/API2/ReplicationConfig.pm | 9 +++++++++ 1 file changed, 9 insertions(+)
V2 remove the abstraction in pve-guest- common and check directly. diff --git a/PVE/API2/ReplicationConfig.pm b/PVE/API2/ReplicationConfig.pm index 69d56aed..9bc4fd3b 100644 --- a/PVE/API2/ReplicationConfig.pm +++ b/PVE/API2/ReplicationConfig.pm @@ -8,6 +8,7 @@ use PVE::Exception qw(raise_perm_exc raise_param_exc); use PVE::JSONSchema qw(get_standard_option); use PVE::RPCEnvironment; use PVE::ReplicationConfig; +use PVE::Cluster; use PVE::RESTHandler; @@ -109,6 +110,14 @@ __PACKAGE__->register_method ({ my $plugin = PVE::ReplicationConfig->lookup($type); my $id = extract_param($param, 'id'); + my $nodelist = PVE::Cluster::get_members(); + my $vmlist = PVE::Cluster::get_vmlist(); + + die "Guest does not exists\n" + if !defined($vmlist->{ids}->{$param->{guest}}); + die "Target does not exists\n" + if !defined($nodelist->{$param->{target}}); + my $code = sub { my $cfg = PVE::ReplicationConfig->new(); -- 2.11.0 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel