Project

General

Profile

Bug #10641

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

Added by Nicolas CHARLES 5 months ago. Updated 4 months ago.

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
Target version (plugin):
Severity:
Minor - inconvenience | misleading | easy workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Priority:
17

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

Related to Rudder - Bug #10752: Invalid generation of inventory at install Released

Associated revisions

Revision c6053c58
Added by Nicolas CHARLES 5 months ago

Fixes #10641: Unecessary call to inventory code to check its version during policy validation

History

#1 Updated by Nicolas CHARLES 5 months ago

  • Status changed from New to In progress

#2 Updated by Nicolas CHARLES 5 months 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

#3 Updated by Jonathan CLARKE 5 months ago

  • Status changed from Pending technical review to New

#4 Updated by Jonathan CLARKE 5 months ago

  • Assignee deleted (Alexis MOUSSET)

#5 Updated by Jonathan CLARKE 5 months ago

  • Category changed from Initial promises & sys tech 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

#6 Updated by Nicolas CHARLES 5 months ago

  • Status changed from Pending technical review to Pending release

#7 Updated by Nicolas CHARLES 4 months ago

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

#8 Updated by Vincent MEMBRÉ 4 months 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.

Also available in: Atom PDF