Commit graph

139 commits

Author SHA1 Message Date
Adam Piasecki
18d4aae59f Matching all branches instead of Epel only 2022-06-23 21:15:10 +00:00
Adam Piasecki
771aff027b Added Epel version to the tar ball name 2022-06-23 21:15:10 +00:00
Adam Piasecki
005da070b3 Added archive EPEL -gt '6' branches 2022-06-23 21:15:10 +00:00
Kevin Fenzi
f9849b7316 git/hooks: everything using this role should be python3 now.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2021-05-05 16:28:06 -07:00
Pierre-Yves Chibon
bd6180dcd0 git/hooks: decode the output of the subprocess
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2021-01-25 10:54:51 +01:00
Miro Hrončok
400c6ecf47 post-receive-fedora-messaging: Don't attempt to decode unicode strings
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>
2020-10-03 22:48:10 +02:00
Pierre-Yves Chibon
155094f884 git/hooks: Update the gnome-post-receive-email hook to use the git-multimail gook instead
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-10-02 18:39:08 +00:00
Pierre-Yves Chibon
3a7fa049d9 batcave: Fix the two git hooks run in the ansible git repo in the batcave
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-09-10 21:33:27 +02:00
Kevin Fenzi
460b28e0c5 git/hooks: Only install the batcave certs on batcave, not pkgs
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-09-09 11:32:02 -07:00
Pierre-Yves Chibon
a65d4589d0 git/hooks: ensures the revision sent to fedora-messaging are strings not oid objects
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>
2020-08-20 10:33:47 +02:00
Nils Philippsen
74752c3a3c Cope with dangling symlinks
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>
2020-06-25 10:30:33 +02:00
Nils Philippsen
34d165ab0d Fix hook links to Pagure hookrunner script, too
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>
2020-06-25 07:26:50 +00:00
Nils Philippsen
5175246b89 Fix wrong argument help string
Signed-off-by: Nils Philippsen <nils@redhat.com>
2020-06-25 07:26:50 +00:00
Nils Philippsen
a87c762569 Convert to pathlib API for easy path manipulation
The only left over is os.readlink() which was only introduced as
Path.readlink() in Python 3.9.

Signed-off-by: Nils Philippsen <nils@redhat.com>
2020-06-25 07:26:50 +00:00
Pierre-Yves Chibon
dc92c9d924 git/hooks: fix typo isinstance != instance
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-06-12 14:15:54 +02:00
Pierre-Yves Chibon
b631c49d08 git/hook: apparently the rev can be either a string or the pygit2.Oid object
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-06-11 21:39:32 +02:00
Pierre-Yves Chibon
18a674d74d git/hook: parse the revision/hash rather than the Oid object
Fixes https://pagure.io/fedora-infrastructure/issue/9016

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-06-11 21:13:44 +02:00
Pierre-Yves Chibon
891dcaac2b distgit: give access to the fedora-messaging certs to the packager group
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>
2020-06-10 10:06:13 +02:00
Stephen Smoogen
0082dc0996 make sure pkgs02 has tar and python2 installed 2020-06-09 22:32:53 -04:00
Pierre-Yves Chibon
de0b5d6abe git/hooks: point the post-receive-chained-forks file to the proper git hook from pagure
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-06-10 00:15:18 +02:00
Pierre-Yves Chibon
6e82268613 git/hooks: Start making the hooks crash less on rhel8/python3
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-06-06 21:17:05 +02:00
Pierre-Yves Chibon
e0ed6697f4 distgit: make the iad2 host closer to the current staging and different from phx2
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-25 23:10:15 +02:00
Pierre-Yves Chibon
7acddaeca5 git/hooks: adjusts the post-receive hooks so it loads the correct fedora-messaging config
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-20 11:23:28 +02:00
Pierre-Yves Chibon
cef92fff61 git/hooks: looks like moreutils fails to install on rhel8 atm
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-20 11:22:01 +02:00
Pierre-Yves Chibon
dd6c85b1e2 Adjust the list of packages to install for RHEL8/python3
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-20 11:21:14 +02:00
Pierre-Yves Chibon
633866fdff git/hooks: install the fedora-messaging certs and config file
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-20 11:20:57 +02:00
Pierre-Yves Chibon
7ee8003ba9 git/hooks: port from fedmsg to fedora-messaging
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2020-05-20 11:13:37 +02:00
Kevin Fenzi
81fb4582e7 ansible: change when conditions to use == instead of is when checking strings.
Signed-off-by: Kevin Fenzi <kevin@scrye.com>
2020-04-24 21:34:10 +02:00
Karsten Hopp
c9ed62ac32 update ansible_distribution_major_version conditionals
Signed-off-by: Karsten Hopp <karsten@redhat.com>
2020-04-24 21:34:10 +02:00
Aurélien Bompard
d98a7392a5 git/hooks: update package command to not use items
Signed-off-by: Aurélien Bompard <aurelien@bompard.org>
2019-06-11 17:56:22 +02:00
Pierre-Yves Chibon
ef61cb0d11 Do not seng git.receive fedmsg message upon changes to the requests git repo
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-12-13 14:50:43 +01:00
Pierre-Yves Chibon
85bd191653 Update post-receive-chained to use the new post-receive hook from pagure
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-11-21 11:46:49 +01:00
Patrick Uiterwijk
03136b922b And use the new getlogin()
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-08-02 17:04:01 +00:00
Patrick Uiterwijk
7148f95bf8 With https pushing, we need to use the USER env var
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-08-02 17:03:15 +00:00
Patrick Uiterwijk
e592912070 Actually, Todd was right and the failed text was invalid. Let's ignore
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-07-19 00:00:59 +02:00
Patrick Uiterwijk
84141c0895 Pretty sure we want to encode
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-07-18 23:44:21 +02:00
Todd Zullinger
c45689142e git/hooks: remove fedmsg_hook from post-receive-chained-forks
The fedmsg_hook no longer exists.  It was removed from the non-fork
post-receive-chained hook in 9875b043d ("Drop the fedmsg_hook since it
no longer exists", 2018-05-03).

Fixes the following warning when pushing to a fork:

    remote: sh: /usr/lib/python2.7/site-packages/pagure/hooks/files/fedmsg_hook.py: No such file or directory

Signed-off-by: Todd Zullinger <tmz@pobox.com>
2018-05-25 11:39:24 -04:00
Pierre-Yves Chibon
9875b043d3 Drop the fedmsg_hook since it no longer exists
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-05-03 12:07:40 +02:00
Pierre-Yves Chibon
3b1e225f73 Add missing get_default_branch 2018-05-03 10:40:13 +02:00
Kevin Fenzi
9bd9110781 Re-enable fedmsg hooks now that there is a acl allowing access to key 2018-04-19 16:42:36 +00:00
Pierre-Yves Chibon
dc86b22f90 Turn off the pagure fedmsg hook for now
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 17:39:48 +02:00
Pierre-Yves Chibon
e864768c74 Typi typoes...
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 15:49:44 +02:00
Pierre-Yves Chibon
7b6407df81 Technically we only need os.walk() on forks
The other namespace won't have sub-folders

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 15:46:41 +02:00
Pierre-Yves Chibon
cab27352d2 Some more fixes to the script
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 15:26:02 +02:00
Pierre-Yves Chibon
c825ebf5a0 Another attempt at not going down the rabbit hole
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 15:22:17 +02:00
Pierre-Yves Chibon
8a180b87c0 Fix checking only a specific namespace
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 15:14:28 +02:00
Pierre-Yves Chibon
1320c7f424 Remove a debugging statement
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 14:06:21 +02:00
Pierre-Yves Chibon
da5d18632b Turn on the pagure fedmsg hook on all projects and forks
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 14:02:00 +02:00
Pierre-Yves Chibon
0b2079ca11 Add new script to check the post-receive hooks on dist-git
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 12:52:32 +02:00
Pierre-Yves Chibon
a09604d9b2 Add a custom post-receive-chained hook for forks
This hook doesn't send fedmsg messages on the git.receive topic
and does not notify the alternative arch folks about changes

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
2018-04-17 12:14:07 +02:00