From 1517f1a55d116e0e0cc27b220192748e6e65efc9 Mon Sep 17 00:00:00 2001 From: Evgeni Golov Date: Wed, 14 Feb 2024 09:11:03 +0100 Subject: [PATCH] don't lower() the packit pr data before manging it packit coprs are case sensitive (it's Katello, not katello), but the lower() was breaking that. the original idea was that the lower() catches GitHub.COM and other spellings, but it seems to do more harm than good, so let's remove it --- roles/packit/tasks/main.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/packit/tasks/main.yml b/roles/packit/tasks/main.yml index ee675a734..fa11a7647 100644 --- a/roles/packit/tasks/main.yml +++ b/roles/packit/tasks/main.yml @@ -5,7 +5,7 @@ chroot: "rhel-{{ ansible_distribution_major_version }}-x86_64" loop: "{{ packit_prs | default([]) }}" vars: - packit_data: "{{ item.lower().replace('https://github.com/', '').replace('/pull', '').split('/') }}" + packit_data: "{{ item.replace('https://github.com/', '').replace('/pull', '').split('/') }}" packit_org: "{{ packit_data[0] }}" packit_repo: "{{ packit_data[1] }}" packit_pr: "{{ packit_data[2] }}"