Project

General

Profile

Bug #21802

Recurring tasks need to be more resilient

Added by Adam Ruzicka about 2 years ago. Updated over 1 year ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1518576

Description of problem:

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. Create a recurring task
2. Cancel the task, while leaving the recurring logic running

Actual results:
The recurring logic stays active forever without actually spawning any more tasks.

Expected results:
The recurring logic either cancels itself or tries to recreate the cancelled task.

Additional info:


Related issues

Related to Foreman - Bug #23008: rss notifications: no implicit conversion of Time into StringDuplicate2018-03-25
Related to foreman-tasks - Bug #23849: Current user is not set when triggering repetitions of recurring actionsClosed2018-06-07
Copied to Foreman Remote Execution - Bug #22700: Use the facilities from foreman-tasks for more resilient recurring jobsClosed2017-11-29

Associated revisions

Revision 6d0d9618 (diff)
Added by Adam Růžička over 1 year ago

Fixes #21802 - Make recurring tasks more resilient

  • Fixes #21802 - Trigger next repetition when cancelling

Before this fix cancelling a task belonging to a recurring logic would
make the recurring logic be active forever without triggering another
repetition.

Additionally, it also fixes time loading from dynflow after schema normalization.

History

#1 Updated by The Foreman Bot almost 2 years ago

  • Assignee set to Adam Ruzicka
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-tasks/pull/311 added

#2 Updated by Adam Ruzicka almost 2 years ago

  • Copied to Bug #22700: Use the facilities from foreman-tasks for more resilient recurring jobs added

#3 Updated by Anonymous over 1 year ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

#4 Updated by Marek Hulán over 1 year ago

  • Related to Bug #23008: rss notifications: no implicit conversion of Time into String added

#5 Updated by Ivan Necas over 1 year ago

  • Related to Bug #23849: Current user is not set when triggering repetitions of recurring actions added

Also available in: Atom PDF