all lists on 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal