Project

General

Profile

Actions

Bug #10641

closed

Unecessary call to inventory code to check its version during policy validation

Added by Nicolas CHARLES over 7 years ago. Updated over 7 years ago.

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Priority:
17
Name check:
Fix check:
Regression:

Description

During policy validation, we see pleinty of

root      9907  0.0  0.0  11628  1336 ?        S    12:38   0:00  |   \_ sh -c /opt/rudder/bin/run-inventory --version | /bin/grep '[0-9]\.[0-9]' | /bin/s
root      9911  0.0  0.0 115248  1700 ?        S    12:38   0:00  |       \_ /bin/sh /opt/rudder/bin/run-inventory --version
root      9981  0.0  0.0 156584 14988 ?        R    12:38   0:00  |       |   \_ /opt/rudder/bin/perl -I /opt/rudder/lib/perl5 /opt/rudder/bin/fusioninven
root      9912  0.0  0.0   9032   656 ?        S    12:38   0:00  |       \_ /bin/grep [0-9]\.[0-9]
root      9913  0.0  0.0  13260   700 ?        S    12:38   0:00  |       \_ /bin/sed s%.*(\([0-9]\+\)\.\([0-9]\+\).*).*%\1\2%

this is completly unuseful, and removing this part in case of cf-promises saves 90s for generation of 2000 nodes (420s instead of 513 s)


Related issues 1 (0 open1 closed)

Related to Rudder - Bug #10752: Invalid generation of inventory at installReleasedBenoît PECCATTEActions
Actions

Also available in: Atom PDF