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

Add code comments to trunk migration logic

parent 3e940367
No related branches found
No related tags found
No related merge requests found
Pipeline #85624 failed
......@@ -8,6 +8,7 @@ repos:
args:
- --fix
- --preview
- --ignore=PLR0917,PLR0914
# Run the formatter.
- id: ruff-format
args:
......
......@@ -7,7 +7,6 @@ configured to run from A to C. B is then no longer associated with this IP trunk
import copy
import json
import re
from typing import NoReturn
from uuid import uuid4
from orchestrator import step, workflow
......@@ -44,7 +43,7 @@ from gso.utils.helpers import (
from gso.utils.workflow_steps import set_isis_to_90000
def initial_input_form_generator(subscription_id: UUIDstr) -> FormGenerator:
def initial_input_form_generator(subscription_id: UUIDstr) -> FormGenerator: # type: ignore[PLR0914]
"""Gather input from the operator on the new router that the IP trunk should connect to."""
subscription = Iptrunk.from_subscription(subscription_id)
form_title = (
......@@ -88,7 +87,9 @@ def initial_input_form_generator(subscription_id: UUIDstr) -> FormGenerator:
not migrate_form_input.migrate_to_different_site
and current_router_site.subscription_id != old_side_site.owner_subscription_id
):
# We want to stay on the same site, so all routers that are in different sites get skipped.
continue
# If migrate_to_different_site is true, we can add ALL routers to the result map
routers[str(router_id)] = router["description"]
new_router_enum = Choice("Select a new router", zip(routers.keys(), routers.items(), strict=True)) # type: ignore[arg-type]
......@@ -148,7 +149,7 @@ def initial_input_form_generator(subscription_id: UUIDstr) -> FormGenerator:
new_lag_member_interfaces: ae_members # type: ignore[valid-type]
@validator("new_lag_interface", allow_reuse=True, pre=True, always=True)
def lag_interface_proper_name(cls, new_lag_interface: str) -> str | NoReturn:
def lag_interface_proper_name(cls, new_lag_interface: str) -> str:
if get_router_vendor(new_router) == RouterVendor.JUNIPER:
juniper_lag_re = re.compile("^ae\\d{1,2}$")
if not juniper_lag_re.match(new_lag_interface):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment