Project

General

Profile

Actions
Copy link

Bug #18510

closed

Loading directive's page is very slow

Bug #18510: Loading directive's page is very slow

Added by Elaad FURREEDAN over 5 years ago. Updated over 5 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Performance and scalability
Target version:
6.2.0~rc1
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

Description

The loading of directive's page take ~3s to load.
Tested on Firefox 82.0.2 and Chromium 83.0.4103.116

Files

Download all files
clipboard-202011091718-4bprj.png (5.72 KB) clipboard-202011091718-4bprj.png François ARMAND, 2020-11-09 17:18
clipboard-202011091720-0qywu.png (4.48 KB) clipboard-202011091720-0qywu.png François ARMAND, 2020-11-09 17:20
difference-61-62.txt (53.1 KB) difference-61-62.txt François ARMAND, 2020-11-09 18:04
clipboard-202011091718-4bprj.png
clipboard-202011091720-0qywu.png

Related issues 1 (0 open — 1 closed)

Related to Rudder - User story #18076: Use new UI template in Directives pageReleasedVincent MEMBRÉActions

Updated by Elaad FURREEDAN over 5 years ago Actions
Copy link
 #1

Updated by François ARMAND over 5 years ago Actions
Copy link
 #2

I can confirm that

Updated by François ARMAND over 5 years ago Actions
Copy link Download all files
 #3

clipboard-202011091718-4bprj.png
clipboard-202011091720-0qywu.png

The problem is the size of the page that did x20: from 18KB (156 unzipped) to 334KB (2.1MB unzipped)!

- 6.1:

- 6.2:

What happened to that page?

Updated by Nicolas CHARLES over 5 years ago Actions
Copy link
 #4

also it computes on the server side a lot of compliance to load this page
On the stress server, it takes 22s before starting to transfer data

Updated by François ARMAND over 5 years ago Actions
Copy link
 #5

  • File difference-61-62.txt added

For the page size, the elements for technique nodes (the one like that) is much biger now:

      <ul>
        <li class="techniqueNode " data-jstree="{ &quot;type&quot; : &quot;template&quot; , &quot;state&quot; : { &quot;disabled&quot; : false } }">
        --- here the changed part ---

See attached file.

Updated by François ARMAND over 5 years ago Actions
Copy link
 #6

  • File deleted (difference-61-62.txt)

Updated by François ARMAND over 5 years ago Actions
Copy link
 #7

The technique nodes contains the technique description, which are not lazy loaded anymore (see new attached file where the HTML entities are replaced for lisibility)

Updated by François ARMAND over 5 years ago Actions
Copy link
 #8

  • Status changed from New to In progress
  • Assignee set to François ARMAND

Updated by François ARMAND over 5 years ago Actions
Copy link
 #9

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/3352

Updated by François ARMAND over 5 years ago Actions
Copy link
 #10

  • Status changed from Pending technical review to Pending release

Updated by François ARMAND over 5 years ago Actions
Copy link
 #11

  • Fix check changed from To do to Checked

Updated by Vincent MEMBRÉ over 5 years ago Actions
Copy link
 #12

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.2.0~rc1 which was released today.

Actions
Copy link

Also available in: PDF Atom