Actions
Bug #20622
closedPuppet 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 about 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 about 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 about 7 years ago
- Translation missing: en.field_release set to 287
Requesting 1.15 as well.
Updated by Lukas Zapletal about 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset be6706162e9705611827aa0a8e875a9f3175d2e9.
Actions