Re: [PATCH pve-network 06/13] api2: add route map module

[Stefan Hanreich <s.hanreich@proxmox.com>]

On 3/26/26 4:06 PM, Hannes Laimer wrote:
> On 2026-03-25 10:44, Stefan Hanreich wrote:
>> This module contains the following API endpoint:
>>
>> GET /route-maps/<id> - lists all route map entries for the route map
>> <id>
>>
>> Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
>> ---
> 
> [..]
> 
>> +__PACKAGE__->register_method({
>> +    name => 'list_route_map_entries',
>> +    path => '',
>> +    method => 'GET',
>> +    permissions => {
>> +        check => ['perm', '/sdn/route-maps/{route_map_id}', ['SDN.Audit']],
>> +    },
>> +    description => "List all entries for a given Route Map",
> 
> [..]
> 
>> +    code => sub {
>> +        my ($param) = @_;
>> +
>> +        my $pending = extract_param($param, 'pending');
>> +        my $running = extract_param($param, 'running');
>> +        my $route_map_id = extract_param($param, 'route-map-id');
>> +
>> +        my $digest;
>> +        my $route_map_entries;
>> +
>> +        if ($pending) {
>> +            my $current_config = PVE::Network::SDN::RouteMaps::config()->list_route_map($route_map_id);
>> +            my $running_config = PVE::Network::SDN::RouteMaps::config(1)->list_route_map($route_map_id);
>> +
>> +            my $pending_route_maps = PVE::Network::SDN::pending_config(
>> +                $running_config,
>> +                $current_config,
> 
> we should wrap these two, like pve-network 03/13 :P

fixed in the same vein as 3/13