- Issue created by @catch
- Status changed to Needs review
10 months ago 11:04am 12 January 2024 - Status changed to Needs work
10 months ago 2:29pm 12 January 2024 - Status changed to Needs review
10 months ago 3:54pm 12 January 2024 - 🇬🇧United Kingdom catch
Some debug crept in, green now.
From looking at the results of this, I found 📌 Avoid reading session from the database multiple times during a request Needs work and 📌 Route preloading doesn't run on dynamic page cache hits Needs review for potential optimizations.
- Status changed to RTBC
10 months ago 4:52pm 12 January 2024 - 🇺🇸United States smustgrave
Based on how other performance tests are written this looks inline.
Left a small comment but not a blocker.
- 🇬🇧United Kingdom catch
Found a third issue as a result of adding this and the open telemetry reports: 📌 Return early in AssetResolver::getJsAssets() and AssetResolver::getCssAssets() when there are no libraries to load Needs review .
- 🇬🇧United Kingdom longwave UK
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 45df42c3 on 10.2.x
- Status changed to Fixed
10 months ago 5:53pm 12 January 2024 -
longwave →
committed af726dbf on 11.x
Issue #3414261 by catch: Add authenticated user umami performance tests
-
longwave →
committed af726dbf on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.