Actions
Bug #10641
closedUnecessary call to inventory code to check its version during policy validation
Status:
Released
Priority:
N/A
Assignee:
Category:
Performance and scalability
Target version:
Pull Request:
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)
Actions