Project

General

Profile

Actions

Bug #6075

closed

The way software inventories are saved is totally inefficient

Added by François ARMAND almost 10 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
1 (highest)
Category:
Performance and scalability
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

When inventories are added to Rudder, we check for the list of software, to see if we already know them.

The way the check is done can't be much more inefficient than how it is done.

Actions #1

Updated by François ARMAND almost 10 years ago

With the proposed enhancement, the performance goes (on my machine, with a fairly big number of nodes (>1000)) from ~15s to add or update an inventory to <1s to add a new inventory and <400ms to update one (with only a few modification).

So that's at least a 15x improvment.
For example, adding 1000 entries for test go down from 4 hours to 15 minutes.

Actions #2

Updated by François ARMAND almost 10 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/ldap-inventory/pull/58
Actions #3

Updated by François ARMAND almost 10 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset ldap-inventory:commit:e2c8154dc4a8d091db6feae2d7e70febee9ed0f6.

Actions #4

Updated by Nicolas CHARLES almost 10 years ago

Applied in changeset ldap-inventory:commit:32b23680dcf2f7a9e83e32e0885622f5b1705203.

Actions #5

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.9 to 2.10.10
Actions #6

Updated by Vincent MEMBRÉ almost 10 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.10.10 and 2.11.7, which were these days.

Actions

Also available in: Atom PDF