From fcc894f4a99e5325f20c60ad5a977ae8d0de6b86 Mon Sep 17 00:00:00 2001
From: Neda Moeini <neda.moeini@geant.org>
Date: Fri, 12 Jul 2024 15:20:45 +0200
Subject: [PATCH] Fixed the migration down revision after rebasing with develop
and update the code base to make the linter happy.
---
.../2023-09-22_6c986f219e3f_add_router_validation_task.py | 5 ++---
.../2024-06-19_fvd7mfcfbs1q_update_modify_isis_workflow.py | 2 +-
gso/utils/workflow_steps.py | 2 +-
gso/workflows/router/validate_router.py | 3 +--
4 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/gso/migrations/versions/2023-09-22_6c986f219e3f_add_router_validation_task.py b/gso/migrations/versions/2023-09-22_6c986f219e3f_add_router_validation_task.py
index 54a07175..266db20a 100644
--- a/gso/migrations/versions/2023-09-22_6c986f219e3f_add_router_validation_task.py
+++ b/gso/migrations/versions/2023-09-22_6c986f219e3f_add_router_validation_task.py
@@ -1,16 +1,15 @@
"""Add router validation task.
Revision ID: 6c986f219e3f
-Revises: 1ec810b289c0
+Revises: f6a38f9e9e18
Create Date: 2023-09-22 17:13:01.223133
"""
-import sqlalchemy as sa
from alembic import op
# revision identifiers, used by Alembic.
revision = '6c986f219e3f'
-down_revision = '1ec810b289c0'
+down_revision = 'f6a38f9e9e18'
branch_labels = None
depends_on = None
diff --git a/gso/migrations/versions/2024-06-19_fvd7mfcfbs1q_update_modify_isis_workflow.py b/gso/migrations/versions/2024-06-19_fvd7mfcfbs1q_update_modify_isis_workflow.py
index 96a18957..d38ddcb4 100644
--- a/gso/migrations/versions/2024-06-19_fvd7mfcfbs1q_update_modify_isis_workflow.py
+++ b/gso/migrations/versions/2024-06-19_fvd7mfcfbs1q_update_modify_isis_workflow.py
@@ -1,7 +1,7 @@
""" Modify ISIS metric workflow description
Revision ID: fvd7mfcfbs1q
-Revises:
+Revises: 1ec810b289c0
Create Date: 2024-06-19 11:17:08.539591
"""
diff --git a/gso/utils/workflow_steps.py b/gso/utils/workflow_steps.py
index f6255824..176b8325 100644
--- a/gso/utils/workflow_steps.py
+++ b/gso/utils/workflow_steps.py
@@ -124,7 +124,7 @@ def prompt_sharepoint_checklist_url(checklist_url: str) -> FormGenerator:
@step("[DRY RUN] Add all PE routers to P router iBGP group")
-def add_all_pe_to_p_dry(subscription: dict[str, Any], callback_route: str, process_id: UUIDstr) -> None:
+def add_all_pe_to_p_dry(subscription: dict[str, Any], callback_route: str) -> None:
"""Perform a dry run of adding the list of all PE routers to the new P router."""
extra_vars = {
"dry_run": True,
diff --git a/gso/workflows/router/validate_router.py b/gso/workflows/router/validate_router.py
index 81ceee92..f5f078db 100644
--- a/gso/workflows/router/validate_router.py
+++ b/gso/workflows/router/validate_router.py
@@ -12,7 +12,6 @@ from orchestrator.workflows.utils import wrap_modify_initial_input_form
from pydantic_forms.types import State, UUIDstr
from gso.products.product_types.router import Router
-from gso.services import infoblox
from gso.services import infoblox, lso_client, subscriptions
from gso.services.librenms_client import LibreNMSClient
from gso.services.lso_client import anonymous_lso_interaction, execute_playbook
@@ -53,7 +52,7 @@ def check_netbox_entry_exists(subscription: Router) -> None:
@step("Verify BGP configuration on P router")
-def verify_p_ibgp(subscription: dict[str, Any], callback_route: str, process_id: UUIDstr) -> None:
+def verify_p_ibgp(subscription: dict[str, Any], callback_route: str) -> None:
"""Perform a dry run of adding the list of all PE routers to the new P router."""
extra_vars = {
"dry_run": True,
--
GitLab