Project

General

Profile

Bug #17041

Orchestration not skipped when importing facts

Added by Timo Goebel over 4 years ago. Updated almost 3 years ago.

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

Description

When importing facts, orchestration is not skipped which causes the smart proxy to overload in larger environments (even though the lease file is not parsed on every request). All foreman passenger processes get blocked and the UI becomes unresponsive.


Related issues

Related to Foreman - Bug #15306: Orchestration does not roll back queued actions if DB error occursClosed2016-06-06
Related to Foreman - Bug #17118: Error pushing facts to foremanDuplicate2016-10-27

Associated revisions

Revision a390cfb7 (diff)
Added by Timo Goebel over 4 years ago

fixes #17041 - skips orchestration when importing facts

The orchestration callbacks only run if unless :skip_orchestration?,
however skip_orchestration? is not a method for Nic::Managed causing the
callbacks to always run, even if importing facts.

Revision 16385b4f (diff)
Added by Timo Goebel over 4 years ago

fixes #17041 - skips orchestration when importing facts

The orchestration callbacks only run if unless :skip_orchestration?,
however skip_orchestration? is not a method for Nic::Managed causing the
callbacks to always run, even if importing facts.

(cherry picked from commit a390cfb73dd55ed6735e9f8916699dc15415ce03)

History

#1 Updated by Timo Goebel over 4 years ago

  • Related to Bug #15306: Orchestration does not roll back queued actions if DB error occurs added

#2 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3958 added

#3 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 190

#4 Updated by Timo Goebel over 4 years ago

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

#5 Updated by Felix Barbeira over 4 years ago

  • Related to Bug #17118: Error pushing facts to foreman added

Also available in: Atom PDF