From 800453513a3f3e3f77ba698b77a76b90848fa6d4 Mon Sep 17 00:00:00 2001 From: Aleksandr Kurbatov <ak@geant.org> Date: Wed, 4 Dec 2024 15:06:50 +0000 Subject: [PATCH 1/3] p-to-pe command_checks: add conditional for displaying results --- .../gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml b/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml index 4d95f8c4..8d3f40bb 100644 --- a/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml +++ b/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml @@ -6,5 +6,6 @@ display: json register: out - name: "Results of {{ item.command }}" + when: out.output['rpc-reply']['nokiaoper:results'] != None ansible.builtin.debug: msg: "{{out.output['rpc-reply']['nokiaoper:results']['nokiaoper:md-cli-output-block'].split('\n')}}" -- GitLab From 3d3f6b6d10a5f2d5c51b480a04c2c67cc3d462b9 Mon Sep 17 00:00:00 2001 From: Aleksandr Kurbatov <ak@geant.org> Date: Wed, 4 Dec 2024 17:04:04 +0000 Subject: [PATCH 2/3] p_to_pe: update in command checks --- .../gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml | 2 +- geant/gap_ansible/roles/promote_p_to_pe/tasks/main.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml b/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml index 8d3f40bb..06ad86e9 100644 --- a/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml +++ b/geant/gap_ansible/roles/promote_p_to_pe/tasks/command_checks.yaml @@ -6,6 +6,6 @@ display: json register: out - name: "Results of {{ item.command }}" - when: out.output['rpc-reply']['nokiaoper:results'] != None + when: "'nokiaoper:results' in out.output['rpc-reply']" ansible.builtin.debug: msg: "{{out.output['rpc-reply']['nokiaoper:results']['nokiaoper:md-cli-output-block'].split('\n')}}" diff --git a/geant/gap_ansible/roles/promote_p_to_pe/tasks/main.yml b/geant/gap_ansible/roles/promote_p_to_pe/tasks/main.yml index 16eabc11..4a68aacc 100644 --- a/geant/gap_ansible/roles/promote_p_to_pe/tasks/main.yml +++ b/geant/gap_ansible/roles/promote_p_to_pe/tasks/main.yml @@ -57,7 +57,7 @@ ansible.builtin.include_tasks: deploy.yaml - name: PRE checks - when: verb == "post_checks" + when: verb == "pre_checks" ansible.builtin.include_tasks: command_checks.yaml loop: "{{ pre_check_commands }}" -- GitLab From 763b3aaace62e378eb19abba5b51b0ffe4f66364 Mon Sep 17 00:00:00 2001 From: Aleksandr Kurbatov <ak@geant.org> Date: Wed, 4 Dec 2024 17:05:59 +0000 Subject: [PATCH 3/3] v1.1.20 --- geant/gap_ansible/CHANGELOG.md | 3 +++ geant/gap_ansible/galaxy.yml | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/geant/gap_ansible/CHANGELOG.md b/geant/gap_ansible/CHANGELOG.md index 13e8e952..e3a3177e 100644 --- a/geant/gap_ansible/CHANGELOG.md +++ b/geant/gap_ansible/CHANGELOG.md @@ -1,4 +1,7 @@ # GAP Ansible changelog +1.1.20 2024-12-04 +- Bugfixes in `p_to_pe_promotion` pre and post checks + 1.1.19 2024-12-03 - Added a list of roles for L3 service provisioning. - `sbp` - SBP interface configuration diff --git a/geant/gap_ansible/galaxy.yml b/geant/gap_ansible/galaxy.yml index 926db354..f2e294b5 100644 --- a/geant/gap_ansible/galaxy.yml +++ b/geant/gap_ansible/galaxy.yml @@ -8,7 +8,7 @@ namespace: geant name: gap_ansible # The version of the collection. Must be compatible with semantic versioning -version: 1.1.19 +version: 1.1.20 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection readme: README.md -- GitLab