Actions
Bug #20622
closed
Puppet reports recalculate all statuses
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Description
It appears that puppet reports cause all statuses to be updated:
https://github.com/theforeman/foreman/blob/develop/app/services/report_importer.rb#L42
On a heavily loaded system this could mean this is being called 100s of times per second. This is a fairly slow operation to be doing when not necessary.
Puppet reports should only refresh statuses that are relevant to them.
Updated by Marek Hulán over 7 years ago
I agree, Host::Managed#refresh_statuses is being used from many places (report import, openscap report import and I can imagine other plugins too). Maybe it should accept set of statuses that were changed and should be refreshed. Btw Justin was looking into this recently too.
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Assignee set to Lukas Zapletal
- Pull request https://github.com/theforeman/foreman/pull/4749 added
Updated by Lukas Zapletal over 7 years ago
- Translation missing: en.field_release set to 287
Requesting 1.15 as well.
Updated by Lukas Zapletal over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset be6706162e9705611827aa0a8e875a9f3175d2e9.
Actions