diff options
| author | Adam Rogers <arogers0311@gmail.com> | 2026-09-21 06:30:17 +1000 |
|---|---|---|
| committer | Adam Rogers <arogers0311@gmail.com> | 2026-09-21 07:55:38 +1000 |
| commit | a9bd2b76ea4ca248d50cc2648b6ab8b19ce5dbf0 (patch) | |
| tree | a89fc0bc5afe0f35b5e2f812ab93b7d9f56c515c /src/migration-scripts | |
| parent | 92c2d6ec5ebd1a721f725473c92a24a737de9cfe (diff) | |
| download | vyos-1x-a9bd2b76ea4ca248d50cc2648b6ab8b19ce5dbf0.tar.gz vyos-1x-a9bd2b76ea4ca248d50cc2648b6ab8b19ce5dbf0.zip | |
smoketest: T9332: add GeoIP source/destination coverage
test_firewall.py and test_policy_route.py's test_geoip hardcoded the
pre-fix GeoIP set names (e.g. GEOIP_CC_name_smoketest_1), which broke
against the new per-side _s/_d suffix regardless of whether a rule
used one side or both - this is what failed CI on this PR.
Update those assertions, and add a dedicated ipv4 and ipv6 rule to
each test that matches GeoIP on both source and destination together,
verifying they resolve to distinct sets rather than colliding on a
shared name (the actual regression this PR fixes), per review
feedback from sarthurdev.
Diffstat (limited to 'src/migration-scripts')
0 files changed, 0 insertions, 0 deletions
