Project

General

Profile

Actions

Bug #24763

closed

Any error in CreateRssNotifications still marks the task as success and it doesn't re-schedule next event

Added by Ivan Necas over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Notifications
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

How to reproduce:

1. simulate error during the rss notification (add raise 'error' to https://github.com/theforeman/foreman/blob/44539bfc73f1c36e57985707032a1c91160af799/app/jobs/create_rss_notifications.rb#L8)
2. restart the dynflowd (or web server in case of development)
3. notice that the "Create RSS notifications" (when foreman-tasks is present) in still in stopped/success
4. further notice, there is not next task planned for the next check

It turns out the active_job is not handling `after_perform` callback when rescue bock is triggered, see https://github.com/rails/rails/blob/v5.2.0/activejob/lib/active_job/execution.rb#L33-L43


Related issues 1 (0 open1 closed)

Related to foreman-tasks - Bug #24761: Don't pause on active jobs tasksClosedIvan NecasActions
Actions #1

Updated by Ivan Necas over 5 years ago

  • Related to Bug #24761: Don't pause on active jobs tasks added
Actions #2

Updated by Ivan Necas over 5 years ago

  • Bugzilla link set to 1623151
Actions #3

Updated by The Foreman Bot over 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6016 added
Actions #4

Updated by Ivan Necas over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions #5

Updated by Marek Hulán over 5 years ago

  • Fixed in Releases 1.20.0 added
Actions #6

Updated by Tomer Brisker over 5 years ago

  • Category set to Notifications
Actions

Also available in: Atom PDF