Project

General

Profile

Actions

Bug #20716

closed

Improve dynamic group computation speed and fix inverted searched

Added by Nicolas CHARLES about 2 years ago. Updated almost 2 years ago.

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

Description

A lot of computation is done by querying LDAP, while we have most of the data sitting in the NodeInfoCache
So dynamic group update can take several minutes for something that could be much faster

Also, searches on node state & node properties are post-filtered, but the order of operation between post-filter and inversion is not correct, so every search on properties that are inverted will not return anything


Related issues 3 (1 open2 closed)

Related to Rudder - Bug #20736: List of directive for the pending node is invalidReleasedFrançois ARMANDActions
Related to Rudder - Bug #20758: Improve group node computationPending releaseNicolas CHARLESActions
Related to Datasources - Bug #20813: tests broken with because of new method in NodeInfoServiceReleasedAlexis MoussetActions
Actions #1

Updated by Nicolas CHARLES about 2 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions #2

Updated by Nicolas CHARLES about 2 years ago

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

Updated by François ARMAND about 2 years ago

  • Target version changed from 7.0.1 to 6.2.13
Actions #4

Updated by Nicolas CHARLES about 2 years ago

  • Related to Bug #20736: List of directive for the pending node is invalid added
Actions #5

Updated by Nicolas CHARLES about 2 years ago

  • Subject changed from Improve dynamic group computation speed to Improve dynamic group computation speed and fix inverted searched
  • Description updated (diff)
Actions #6

Updated by Nicolas CHARLES about 2 years ago

  • Pull Request changed from https://github.com/Normation/rudder/pull/4155 to https://github.com/Normation/rudder/pull/4164
Actions #7

Updated by Nicolas CHARLES about 2 years ago

  • Related to Bug #20758: Improve group node computation added
Actions #8

Updated by Nicolas CHARLES about 2 years ago

  • Status changed from Pending technical review to Pending release
Actions #9

Updated by François ARMAND about 2 years ago

  • Related to Bug #20813: tests broken with because of new method in NodeInfoService added
Actions #11

Updated by François ARMAND about 2 years ago

  • Fix check changed from To do to Checked
Actions #12

Updated by Vincent MEMBRÉ almost 2 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.2.13, 7.0.2 and 7.1.0~rc1 which were released today.

Actions

Also available in: Atom PDF