public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Stefan Hanreich <s.hanreich@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [PATCH pve-network v8 07/25] tests: fix prefix list tests by including seq property
Date: Mon, 11 May 2026 15:36:26 +0200	[thread overview]
Message-ID: <20260511133650.310040-8-s.hanreich@proxmox.com> (raw)
In-Reply-To: <20260511133650.310040-1-s.hanreich@proxmox.com>

Since the seq property is now required for prefix list entries, fix
the tests s.t. they always include the sequence number for every
entry.

Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
---
 .../bgp_evpn_routemap_prefix_list/expected_controller_config  | 4 ++--
 src/test/zones/evpn/bgp_evpn_routemap_prefix_list/sdn_config  | 4 ++--
 .../evpn/routemap_prefix_list/expected_controller_config      | 4 ++--
 src/test/zones/evpn/routemap_prefix_list/sdn_config           | 4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/expected_controller_config b/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/expected_controller_config
index 7caf35ba..557fc96a 100644
--- a/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/expected_controller_config
+++ b/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/expected_controller_config
@@ -51,8 +51,8 @@ router bgp 65002 vrf vrf_myzone
  exit-address-family
 exit
 !
-ip prefix-list some_list deny 192.0.2.0/24 le 25
-ip prefix-list some_list deny 198.51.100.0/25 le 26 ge 25
+ip prefix-list some_list seq 44 deny 192.0.2.0/24 le 25
+ip prefix-list some_list seq 123 deny 198.51.100.0/25 le 26 ge 25
 ip prefix-list some_list seq 22 permit 203.0.113.0/24
 !
 route-map MAP_VTEP_IN permit 1
diff --git a/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/sdn_config b/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/sdn_config
index 24ee624f..eab8da10 100644
--- a/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/sdn_config
+++ b/src/test/zones/evpn/bgp_evpn_routemap_prefix_list/sdn_config
@@ -42,8 +42,8 @@
                 id => 'some_list',
                 type => 'prefix-list',
                 entries => [
-                    'action=deny,prefix=192.0.2.0/24,le=25',
-                    'action=deny,prefix=198.51.100.0/25,ge=25,le=26',
+                    'action=deny,prefix=192.0.2.0/24,le=25,seq=44',
+                    'action=deny,prefix=198.51.100.0/25,ge=25,le=26,seq=123',
                     'action=permit,prefix=203.0.113.0/24,seq=22',
                 ]
             }
diff --git a/src/test/zones/evpn/routemap_prefix_list/expected_controller_config b/src/test/zones/evpn/routemap_prefix_list/expected_controller_config
index 2e3a5914..cb912dc8 100644
--- a/src/test/zones/evpn/routemap_prefix_list/expected_controller_config
+++ b/src/test/zones/evpn/routemap_prefix_list/expected_controller_config
@@ -34,8 +34,8 @@ router bgp 65000 vrf vrf_myzone
  no bgp graceful-restart notification
 exit
 !
-ip prefix-list some_list deny 192.0.2.0/24 le 25
-ip prefix-list some_list deny 198.51.100.0/25 le 26 ge 25
+ip prefix-list some_list seq 11 deny 192.0.2.0/24 le 25
+ip prefix-list some_list seq 123 deny 198.51.100.0/25 le 26 ge 25
 ip prefix-list some_list seq 22 permit 203.0.113.0/24
 !
 route-map MAP_VTEP_IN permit 1
diff --git a/src/test/zones/evpn/routemap_prefix_list/sdn_config b/src/test/zones/evpn/routemap_prefix_list/sdn_config
index 8f1179a2..f60559e0 100644
--- a/src/test/zones/evpn/routemap_prefix_list/sdn_config
+++ b/src/test/zones/evpn/routemap_prefix_list/sdn_config
@@ -34,8 +34,8 @@
                 id => 'some_list',
                 type => 'prefix-list',
                 entries => [
-                    'action=deny,prefix=192.0.2.0/24,le=25',
-                    'action=deny,prefix=198.51.100.0/25,ge=25,le=26',
+                    'action=deny,prefix=192.0.2.0/24,le=25,seq=11',
+                    'action=deny,prefix=198.51.100.0/25,ge=25,le=26,seq=123',
                     'action=permit,prefix=203.0.113.0/24,seq=22',
                 ]
             }
-- 
2.47.3





  parent reply	other threads:[~2026-05-11 13:39 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-11 13:36 [PATCH manager/network/proxmox{-ve-rs,-perl-rs} v8 00/25] Add support for route maps / prefix lists to SDN Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-ve-rs v8 01/25] sdn: prefix lists: refactor section config and api format Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-ve-rs v8 02/25] prefix lists: implement validation for prefix lists Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-perl-rs v8 03/25] sdn: prefix lists: refactor existing API endpoint Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-perl-rs v8 04/25] sdn: prefix lists: add crud methods for prefix list entries Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-perl-rs v8 05/25] sdn: prefix lists: validate prefix lists Stefan Hanreich
2026-05-11 13:36 ` [PATCH proxmox-perl-rs v8 06/25] sdn: route maps: add route map list method Stefan Hanreich
2026-05-11 13:36 ` Stefan Hanreich [this message]
2026-05-11 13:36 ` [PATCH pve-network v8 08/25] api: refactor route map api structure Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-network v8 09/25] api: refactor prefix list " Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 10/25] ui: sdn: add route map selector Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 11/25] ui: sdn: add prefix list selector Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 12/25] ui: sdn: add panel for managing prefix lists Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 13/25] ui: sdn: add panel for managing route map entries Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 14/25] ui: sdn: bgp controller: allow configuring route maps Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 15/25] ui: sdn: evpn " Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 16/25] ui: sdn: openfabric: add route filter Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 17/25] ui: sdn: ospf: add route filter setting Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 18/25] ui: sdn: prefix list: add missing subjects Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 19/25] sdn: do not fail rendering record data if pending property is missing Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 20/25] ui: sdn: prefix list: adapt to changed api structure Stefan Hanreich
2026-05-11 16:11   ` Lukas Sichert
2026-05-11 16:24     ` Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 21/25] ui: sdn: route maps: adapt to new route map " Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 22/25] ui: sdn: prefix lists: route maps: use integerfields for numbers Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 23/25] ui: sdn: prefix list panel: reload data on deleting prefix list entry Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 24/25] ui: prefix list panel: delete empty le and get properties Stefan Hanreich
2026-05-11 13:36 ` [PATCH pve-manager v8 25/25] ui: prefix list entry panel: make prefix required Stefan Hanreich
2026-05-12  1:51 ` applied: [PATCH manager/network/proxmox{-ve-rs,-perl-rs} v8 00/25] Add support for route maps / prefix lists to SDN Thomas Lamprecht
2026-05-12  8:22   ` Stefan Hanreich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260511133650.310040-8-s.hanreich@proxmox.com \
    --to=s.hanreich@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal