Project

General

Profile

Actions

Bug #18510

closed

Loading directive's page is very slow

Added by Elaad FURREEDAN about 4 years ago. Updated almost 4 years ago.

Status:
Released
Priority:
N/A
Category:
Performance and scalability
Target version:
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

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

Related issues 1 (0 open1 closed)

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

Updated by Elaad FURREEDAN about 4 years ago

Actions #2

Updated by François ARMAND about 4 years ago

I can confirm that

Updated by François ARMAND about 4 years ago

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?

Actions #4

Updated by Nicolas CHARLES about 4 years ago

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

Actions #5

Updated by François ARMAND about 4 years ago

  • 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.

Actions #6

Updated by François ARMAND about 4 years ago

  • File deleted (difference-61-62.txt)
Actions #7

Updated by François ARMAND about 4 years ago

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)

Actions #8

Updated by François ARMAND about 4 years ago

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

Updated by François ARMAND about 4 years ago

  • 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
Actions #10

Updated by François ARMAND about 4 years ago

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

Updated by François ARMAND about 4 years ago

  • Fix check changed from To do to Checked
Actions #12

Updated by Vincent MEMBRÉ almost 4 years ago

  • Status changed from Pending release to Released

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

Actions

Also available in: Atom PDF