From 5b9b2ddbd6320818db8dbdca7fb355655b179f04 Mon Sep 17 00:00:00 2001
From: Karel van Klink <karel.vanklink@geant.org>
Date: Tue, 6 Aug 2024 11:38:05 +0200
Subject: [PATCH] Rename conditional for reader clarity
---
gso/workflows/iptrunk/validate_iptrunk.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gso/workflows/iptrunk/validate_iptrunk.py b/gso/workflows/iptrunk/validate_iptrunk.py
index 4ffe3deb..bdfc4d3b 100644
--- a/gso/workflows/iptrunk/validate_iptrunk.py
+++ b/gso/workflows/iptrunk/validate_iptrunk.py
@@ -195,7 +195,7 @@ def validate_iptrunk() -> StepList:
If a trunk has a Juniper router on both sides, it is considered legacy and does not require validation.
"""
- is_legacy_trunk = conditional(
+ skip_legacy_trunks = conditional(
lambda state: all(
side.iptrunk_side_node.vendor == Vendor.JUNIPER
for side in Iptrunk.from_subscription(state["subscription_id"]).iptrunk.iptrunk_sides
@@ -205,7 +205,7 @@ def validate_iptrunk() -> StepList:
return (
begin
>> store_process_subscription(Target.SYSTEM)
- >> is_legacy_trunk(done)
+ >> skip_legacy_trunks(done)
>> unsync
>> verify_ipam_records
>> verify_netbox_entries
--
GitLab