Project

General

Profile

Bug #12725

We are missing a lot of timing measurement in policy generation

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

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
Severity:
User visibility:
Effort required:
Priority:
0

Description

In the policy generation, we have a big black box in term of time measurement: writeTemplate
This method uses the biggest part of the generation time (more than 80%), but we have little to no detail of which steps are slow

We need to add real timing info in this method, to measure what is slow

First studies give a very interesting result
[2018-05-31 16:15:49] DEBUG policy.generation - Promises move to final position in 11317 ms out of [2018-05-31 16:16:05] DEBUG com.normation.rudder.services.policies.PromiseGenerationServiceImpl - Write node configurations : 41423 ms

The logger used for that is named: policy.generation:

  <logger name="policy.generation" level="debug"/>

Associated revisions

Revision 8c4429fc (diff)
Added by Nicolas CHARLES 6 months ago

Fixes #12725: We are missing a lot of timing measurement in policy generation

History

#1 Updated by Nicolas CHARLES 6 months ago

  • Status changed from New to In progress

#2 Updated by Nicolas CHARLES 6 months 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/1954

#3 Updated by Normation Quality Assistant 6 months ago

  • Assignee changed from François ARMAND to Nicolas CHARLES

#4 Updated by Nicolas CHARLES 6 months ago

  • Status changed from Pending technical review to Pending release

#5 Updated by François ARMAND 5 months ago

  • Description updated (diff)

#6 Updated by François ARMAND 5 months ago

  • Description updated (diff)

#7 Updated by Vincent MEMBRÉ 5 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.13, 4.2.7 and 4.3.3 which were released today.

Also available in: Atom PDF