- Issue created by @catch
- Merge request !6385Increase assertion range to stop random test failures. → (Closed) created by catch
- Status changed to Needs review
11 months ago 10:19am 30 January 2024 - 🇬🇧United Kingdom catch
The last six performance test runs didn't reproduce this, only seeing it on full test runs:
Quick way to check is where it says the number of spans - i.e. '62 spans' on each of these.
https://grafana.prod.cluster.tag1.io/explore?panes=%7B%22d4B%22:%7B%22da...
https://grafana.prod.cluster.tag1.io/explore?panes=%7B%22hFh%22:%7B%22da...
https://grafana.prod.cluster.tag1.io/explore?panes=%7B%22b1m%22:%7B%22da...
https://grafana.prod.cluster.tag1.io/explore?panes=%7B%22AZk%22:%7B%22da...
https://grafana.prod.cluster.tag1.io/explore?panes=%7B%225GG%22:%7B%22da... - Status changed to RTBC
11 months ago 8:35pm 30 January 2024 - 🇺🇸United States smustgrave
If it helps capture more info sure. Only one that I was going back n forth on was the range 66-67 but may just be lack of coffee at the moment. Don't see any breaking feature though.
- 🇬🇧United Kingdom catch
Thinking now it might be worth postponing this on 📌 Allow needs_destruction services to run on page cache hits Needs review just in case that issue makes things more reliable overall.
- Status changed to Postponed
11 months ago 9:56pm 30 January 2024 - Status changed to Needs review
11 months ago 10:49pm 31 January 2024 - 🇬🇧United Kingdom catch
Starting again here since that issue hasn't removed the variation but all the numbers have changed now we're recording differently. @andypost noticed one on https://git.drupalcode.org/issue/drupal-839444/-/jobs/725475
- 🇬🇧United Kingdom catch
Not seeing any variation in the traces from https://grafana.prod.cluster.tag1.io/d/teMVIdjVz/umami?orgId=1&refresh=3...
It's possible this is infrequent so just isn't showing up in the last six traces, or it's possible that it only shows up on a full test run. If it's the latter, that will be very annoying to track down.
-
longwave →
committed 110c3f05 on 11.x
Issue #3417989 by catch: Add ranges to more performance test assertions
-
longwave →
committed 110c3f05 on 11.x
- Status changed to Fixed
11 months ago 6:45pm 3 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.