Problem/Motivation

We have node and front page anonymous performance tests for Umami, we should add authenticated tests too.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3414261

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

catch created an issue. See original summary.

catch opened merge request !6141

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

seems to have failures

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs work » Needs review

Some debug crept in, green now.

From looking at the results of this, I found #3414287: Avoid reading session from the database multiple times during a big pipe request and #3414373: Route preloading doesn't run on dynamic page cache hits for potential optimizations.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Based on how other performance tests are written this looks inline.

Left a small comment but not a blocker.

catch’s picture

catch
he/him
Primary language English
commented

Found a third issue as a result of adding this and the open telemetry reports: #3414398: AssetResolver::getJsAssets() and AssetResolver::getCssAssets() can repeatedly try to calculate the same set of assets.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

Backported to 10.2.x to keep things in sync, the performance tests are the same there so why not add this one as well.

Committed and pushed af726dbfa5 to 11.x and 45df42c385 to 10.2.x. Thanks!

  • longwave committed 45df42c3 on 10.2.x 
    Issue #3414261 by catch: Add authenticated user umami performance tests...

  • longwave committed af726dbf on 11.x 
    Issue #3414261 by catch: Add authenticated user umami performance tests

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.