Bug 698441 - fedora-easy-karma submits too many comments to bodhi when bodhi has a server problem
Summary: fedora-easy-karma submits too many comments to bodhi when bodhi has a server ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: bodhi
Version: 15
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Luke Macken
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-04-20 21:34 UTC by Christopher Aillon
Modified: 2016-09-20 02:41 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-10-25 15:54:39 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Christopher Aillon 2011-04-20 21:34:00 UTC
Not sure if this is a fedora-easy-karma bug or a bodhi bug, but filing here for now.

I started using fedora-easy-karma and during my usage, koji went down (around Wed Apr 20 2100 UTC)

I had submitted a comment to https://admin.fedoraproject.org/updates/FEDORA-2011-5638

fedora-easy-karma told me:
Comment? -1/0/1 ->karma, other -> skip> 1
Comment> yep
Warning: Server error: ServerError(https://admin.fedoraproject.org/updates/comment, 500, Internal Server Error)
Warning: Server error: ServerError(https://admin.fedoraproject.org/updates/comment, 500, Internal Server Error)
Warning: Server error: ServerError(https://admin.fedoraproject.org/updates/comment, 500, Internal Server Error)
Warning: Server error: ServerError(https://admin.fedoraproject.org/updates/comment, 500, Internal Server Error)
Warning: Comment not submitted: too many errors


However, as you can see on the update, it actually did submit the comment each time it told me it didn't (4 in total).

Comment 1 Christopher Aillon 2011-04-20 21:45:08 UTC
<lmacken> caillon: it's due to the koji outage
<lmacken> caillon: since your comment will trigger autokarma, it errors out in adding the -pending tag to the update

Ah, so the problem is that since the package has karma automatism set to 3, and I am the 3rd +1, it failed in moving the package over to the new tag.  Not really sure how to best handle it.  Perhaps bodhi should be taken offline during future koji outages?

Comment 2 Till Maas 2011-08-19 18:13:36 UTC
I do not see what I can do here. Fedora-easy-karma allows to limit the retries in case of an error:

--retries=RETRIES     Number if retries when submitting a comment in case of
                        an error, default: 3

Use --retries=0 if you want to avoid multiple comments in all cases.

Comment 3 Luke Macken 2011-10-25 15:54:39 UTC
This issue should be resolved with bodhi v0.8.3, which is currently deployed to
production. I made the addition/removal of pending tags more robust, so future koji outages shouldn't trigger this.


Note You need to log in before you can comment on or make changes to this bug.