From 187591fa23b0dcfadd8ba37ed2be442351851409 Mon Sep 17 00:00:00 2001 From: Nick Dokos Date: Tue, 6 Jun 2023 16:03:07 -0400 Subject: [PATCH] Last-minute fixes to ansible roles --- agent/ansible/collection/galaxy.yml | 2 +- .../collection/roles/pbench_repo_install/defaults/main.yml | 2 +- .../collection/roles/pbench_repo_install/tasks/main.yml | 3 ++- .../templates/etc/yum.repos.d/pbench.repo.j2 | 4 ++-- 4 files changed, 6 insertions(+), 5 deletions(-) diff --git a/agent/ansible/collection/galaxy.yml b/agent/ansible/collection/galaxy.yml index 7f4f4c53d8..328b82eb99 100644 --- a/agent/ansible/collection/galaxy.yml +++ b/agent/ansible/collection/galaxy.yml @@ -10,7 +10,7 @@ namespace: pbench name: agent # The version of the collection. Must be compatible with semantic versioning -version: 1.0.7 +version: 1.0.10 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection readme: README.md diff --git a/agent/ansible/collection/roles/pbench_repo_install/defaults/main.yml b/agent/ansible/collection/roles/pbench_repo_install/defaults/main.yml index f152c8851f..28a22c5636 100644 --- a/agent/ansible/collection/roles/pbench_repo_install/defaults/main.yml +++ b/agent/ansible/collection/roles/pbench_repo_install/defaults/main.yml @@ -17,7 +17,7 @@ repos: - tag: "{{ pbench_aux_repo_name }}" user: "{{ fedoraproject_username }}" - baseurl: "{{ pbench_repo_url_prefix }}/{{ pbench_aux_repo_name }}/{{distrodir}}" + baseurl: "{{ pbench_repo_url_prefix }}/{{ pbench_aux_repo_name }}/{{ distrodir }}" gpgkey: "{{ pbench_repo_url_prefix }}/{{ pbench_aux_repo_name }}/pubkey.gpg" gpgcheck: 1 enabled: "{{ enable_copr_aux_repo }}" diff --git a/agent/ansible/collection/roles/pbench_repo_install/tasks/main.yml b/agent/ansible/collection/roles/pbench_repo_install/tasks/main.yml index 2eb8e2f7ae..4bdb8f74b2 100644 --- a/agent/ansible/collection/roles/pbench_repo_install/tasks/main.yml +++ b/agent/ansible/collection/roles/pbench_repo_install/tasks/main.yml @@ -1,10 +1,11 @@ --- # Install pbench.repo - name: Assert pbench_repo_name is defined - assert: + ansible.builtin.assert: that: - pbench_repo_name is defined fail_msg: "Please specify the COPR repository name to use in the `pbench_repo_name` variable" + quiet: true - name: Ensure we have the pbench.repo file properly in place ansible.builtin.template: diff --git a/agent/ansible/collection/roles/pbench_repo_install/templates/etc/yum.repos.d/pbench.repo.j2 b/agent/ansible/collection/roles/pbench_repo_install/templates/etc/yum.repos.d/pbench.repo.j2 index 673c8bc1b2..08ddd8b2c5 100644 --- a/agent/ansible/collection/roles/pbench_repo_install/templates/etc/yum.repos.d/pbench.repo.j2 +++ b/agent/ansible/collection/roles/pbench_repo_install/templates/etc/yum.repos.d/pbench.repo.j2 @@ -1,7 +1,7 @@ {% for repo in repos %} -[copr-{{ repo.name }}-{{ repo.user }}] -name=COPR {{ repo.name }} ({{ repo.user }}) repo +[copr-{{ repo.tag }}-{{ repo.user }}] +name=COPR {{ repo.tag }} ({{ repo.user }}) repo baseurl={{ repo.baseurl }} gpgcheck={{ repo.gpgcheck }} gpgkey={{ repo.gpgkey }}