Skip to content
Snippets Groups Projects
Verified Commit 5b9b2ddb authored by Karel van Klink's avatar Karel van Klink :smiley_cat:
Browse files

Rename conditional for reader clarity

parent c1a27d19
No related branches found
No related tags found
1 merge request!247Skip IP trunk validation for trunks that are Juniper on both sides
Pipeline #88294 passed
...@@ -195,7 +195,7 @@ def validate_iptrunk() -> StepList: ...@@ -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. 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( lambda state: all(
side.iptrunk_side_node.vendor == Vendor.JUNIPER side.iptrunk_side_node.vendor == Vendor.JUNIPER
for side in Iptrunk.from_subscription(state["subscription_id"]).iptrunk.iptrunk_sides for side in Iptrunk.from_subscription(state["subscription_id"]).iptrunk.iptrunk_sides
...@@ -205,7 +205,7 @@ def validate_iptrunk() -> StepList: ...@@ -205,7 +205,7 @@ def validate_iptrunk() -> StepList:
return ( return (
begin begin
>> store_process_subscription(Target.SYSTEM) >> store_process_subscription(Target.SYSTEM)
>> is_legacy_trunk(done) >> skip_legacy_trunks(done)
>> unsync >> unsync
>> verify_ipam_records >> verify_ipam_records
>> verify_netbox_entries >> verify_netbox_entries
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment