This will unify all the handlers to use first uppercase letter for
ansible-lint to stop complaining.
I went through all `notify:` occurrences and fixed them by running
```
set TEXT "text_to_replace"; set REPLACEMENT "replacement_text"; git grep
-rlz "$TEXT" . | xargs -0 sed -i "s/$TEXT/$REPLACEMENT/g"
```
Then I went through all the changes and removed the ones that wasn't
expected to be changed.
Fixes https://pagure.io/fedora-infrastructure/issue/12391
Signed-off-by: Michal Konecny <mkonecny@redhat.com>
fix 1900 failures of the following case issue:
`name[casing]: All names should start with an uppercase letter.`
Signed-off-by: Ryan Lerch <rlerch@redhat.com>
This reverts commit 26358b62a2.
So, this broke the toddler that handles package retirements. ;(
It seems it was commited on feb 20th, but not actually pushed out until
March 4th. At that point, retirements stop. ;(
I think it's getting the new git pagure messages, but they don't
contain the information it expects so it just drops them.
We will need to adjust toddlers before we can re-drop this.
Just scratching an itch with `s/ExcludesArch/ExcludeArch/`, making the
script do what it was supposed to do all those years. Call it The Seven
Year Itch if you like. :-P
This reverts commit 4bfc46ce37.
The Packit folks are lisening to the git messages, they will migrate to
the pagure messages but ask us to re-enable them while they do the
migration.
We already removed the old previous days version here, so trying to
remove it again results in a cron email with:
rm: cannot remove '/srv/cache/lookaside//rpm-specs-epel7*.tar.xz': No such file or directory
rm: cannot remove '/srv/cache/lookaside//rpm-specs-epel8*.tar.xz': No such file or directory
rm: cannot remove '/srv/cache/lookaside//rpm-specs-epel9*.tar.xz': No such file or directory
So, drop that here so we don't get these daily emails.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
Install the 'git-hooks' cert for use with git commits and package
uploads on pkgs. Before this we were... just not installing those
anymore and they had expired.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
On Python 3, we can only decode byte sequences.
subprocess.Popen(universal_newlines=True) always returns unicode stdout/sderr.
Fixes https://pagure.io/fedora-infrastructure/issue/9366
Signed-off-by: Miro Hrončok <miro@hroncok.cz>
The issue was triggered when multiple commits were pushed at once at
which point the list of revisions contained a list of OID objects
that are not JSON serializable but can be converted to text via
str() or by accessing their .hex attribute.
However, when a single commit is pushed, the list of revisions
contains just the hash of the commit pushed and thus is already
a string. So relying on .hex will not work then.
Using str() becomes the solution that works for both scenarios.
Fixes https://pagure.io/fedora-infrastructure/issue/9247
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
The exists() method uses os.stat() instead of os.lstat() and so doesn't
work for symlinks pointing to a non-existent path. A corresponding
lexists() method doesn't exist yet.
Signed-off-by: Nils Philippsen <nils@redhat.com>
This doesn't attempt to prefix the base directory if a supplied `target`
is an absolute path (for debugging).
Fixes: fedora-infrastructure#9000
Signed-off-by: Nils Philippsen <nils@redhat.com>
This solves the permission denied error packager would see otherwise
when pushing to dist-git as the hooks would fail to send notifications
on the fedora-messaging bus.
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
