From e26f82d07a929dbee0ec4c47710ce903884eb067 Mon Sep 17 00:00:00 2001
From: Karel van Klink <karel.vanklink@geant.org>
Date: Tue, 3 Dec 2024 12:52:08 +0100
Subject: [PATCH] prevent sending an empty extra_vars in iBGP check run

---
 gso/workflows/router/update_ibgp_mesh.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gso/workflows/router/update_ibgp_mesh.py b/gso/workflows/router/update_ibgp_mesh.py
index 1b43273c..23379ba0 100644
--- a/gso/workflows/router/update_ibgp_mesh.py
+++ b/gso/workflows/router/update_ibgp_mesh.py
@@ -155,12 +155,12 @@ def add_all_pe_to_p_real(subscription: dict[str, Any], tt_number: str, process_i
 
 
 @step("Verify iBGP session health")
-def check_ibgp_session(subscription: Router) -> LSOState:
+def check_ibgp_session(subscription: dict[str, Any]) -> LSOState:
     """Run a playbook using the provisioning proxy, to check the health of the new iBGP session."""
     return {
         "playbook_name": "gap_ansible/playbooks/check_ibgp.yaml",
-        "inventory": {"all": {"hosts": {subscription.router.router_fqdn: None}}},
-        "extra_vars": {},
+        "inventory": {"all": {"hosts": {subscription["router"]["router_fqdn"]: None}}},
+        "extra_vars": {"subscription": subscription},
     }
 
 
-- 
GitLab