Issue #3923: fedmsg enablement - replace the genacls script with a fedmsg listener - fedora-infrastructure

fedora-infrastructure

#3923 fedmsg enablement - replace the genacls script with a fedmsg listener

Closed: Fixed None Opened 10 years ago by ralph.

when a fedmsg message has arrived that someone's pkgdb acls have changed, we should fire up a script to ask pkgdb if this is really true, and if it is we should run the genacls script so that packagers rights are reflected on pkgs.fp.o "immediately".

This could be done either with the new fedmsg-trigger command slated for the 0.7.0 release or with a consumer to the fedmsg-hub daemon.

ralph commented 10 years ago

Assigned to myself for bookkeeping. Please take it from me if you want to work on it! ;)

ralph commented 10 years ago

Proof of concept of the wait-and-delay functionality in a fedmsg consumer:

https://github.com/fedora-infra/fedmsg-genacls

janeznemanic, can you take it from here?

janeznemanic commented 10 years ago

I gonna try. In two weeks I will inform you what's the status and if I can complete the job.

pingou commented 10 years ago

Should we expend this to update bugzilla directly as well? This would avoid running the cron job (cc'ing toshio)

pingou commented 10 years ago

@janez, I think you can avoid using pkgdb-cli and directly call the API of pkgdb2: http://209.132.184.188/api/

This means it needs to wait for pkgdb2 but I hope this won't take too long.

ralph commented 10 years ago

Replying to [comment:5 pingou]:

Should we expend this to update bugzilla directly as well? This would avoid running the cron job (cc'ing toshio)

Can we make that a separate project? It could use the same starter-formula that I came up with for janeznemanic.

This is a first of its kind tool for updating a part of our existing systems with fedmsg. Let's keep it less complicated at least the first time around, please.

pingou commented 10 years ago

Replying to [comment:7 ralph]:

This is a first of its kind tool for updating a part of our existing systems with fedmsg. Let's keep it less complicated at least the first time around, please.

Sure, of course, it was just an idea when I came across this ticket :)

janeznemanic commented 10 years ago

Here is what I have at the moment. Give me as much comments as possible. This hasn't been fully tested so there are errors in code. Advises how to remove code errors and how to improve code are highly desirable.

janeznemanic commented 10 years ago

attachment
fedmsg_genacls.py

ralph commented 10 years ago

janeznemanic, I think its a good next step.

can you push it directly to the develop branch of this repo? https://github.com/fedora-infra/fedmsg-genacls

It will be easier to edit it back and forth there, I think.

janeznemanic commented 10 years ago

I pushed it to github. Let me know what needs to be improved.

ralph commented 10 years ago

I packaged it up and put it up for review: https://bugzilla.redhat.com/show_bug.cgi?id=1080094

ralph commented 10 years ago

This is pushed out to staging and production now as of puppet commit 34451c1.