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 #1

Updated by Nicolas CHARLES over 7 years ago

  • Status changed from New to In progress
Actions #2

Updated by Nicolas CHARLES over 7 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1135
Actions #3

Updated by Jonathan CLARKE over 7 years ago

  • Status changed from Pending technical review to New
Actions #4

Updated by Jonathan CLARKE over 7 years ago

  • Assignee deleted (Alexis Mousset)
Actions #5

Updated by Jonathan CLARKE over 7 years ago

  • Category changed from System techniques to Performance and scalability
  • Status changed from New to Pending technical review
  • Assignee set to Alexis Mousset
  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 17
Actions #6

Updated by Nicolas CHARLES over 7 years ago

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

Updated by Nicolas CHARLES over 7 years ago

  • Related to Bug #10752: Invalid generation of inventory at install added
Actions #8

Updated by Vincent MEMBRÉ over 7 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.1.20, 4.0.5 and 4.1.2 which were released today.

Actions

Also available in: Atom PDF