Bug #20532
closedcandlepin event listener does not release messages after error
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1440235
Description of problem:
The candlepin event listener will call 'acknowledge' on messages that it processes, but does not call 'release' or 'reject' on messages that it is unable to process.
This can cause some messages to become stuck in the katello_event queue, since they are being held but will never be released.
The best behavior may be to just log the message and then reject it, so potentially bad messages do not get reprocessed over and over.
Version-Release number of selected component (if applicable): 6.2.8
How reproducible: not sure how to repro yet
note: I put this under the 'hosts' component but I'm not sure if that is the best place for this to live. It is related to the ListenOnCandlepinEvents task.
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/6900 added
Updated by Pavel Moravec over 7 years ago
Nondeterministc reproducer (not real user story, but worth for developers/testers):
1) stop foreman-tasks service (to populate katello_event_queue a bit)
2) generate several hundreds candlepin events (i.e. (un)register a Content Host with an activation key, or remove all and attach back a subscription pool to another Host) - do that in a loop until katello_event_queue has few hundreds of messages
3) start foreman-tasks service - leave step 2) running (at least I did so, it might but not need to be important)
4) Once ListenOnCandlepinEvents task consumes the backlog, check if katello_event_queue has zero queue depth (see #c7)
5) if there some messages constantly acquired but not acknowledged, you got it. Otherwise, goto step 1).
Updated by Justin Sherrill over 7 years ago
- Assignee changed from Eric Helms to Justin Sherrill
Updated by Justin Sherrill over 7 years ago
- Translation missing: en.field_release set to 286
Updated by Brad Buckingham over 7 years ago
- Target version changed from 208 to 213
Updated by Eric Helms over 7 years ago
- Translation missing: en.field_release changed from 286 to 295
Updated by Justin Sherrill over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|6f9ecbfb02154370067d47237509e4827acf7c7f.