Changeset 58754

Timestamp:

07/18/2024 02:09:50 PM (3 months ago)

Author:

desrosj

Message:

Build/Test Tools: Remove regular 6.2 & 6.3 branch performance testing.

Prior to WordPress 6.4, performance testing was performed using Puppeteer instead of Playwright (converted in [56926]). Because of the flaky nature of the workflow using Puppeteer, it was not converted to the reusable pattern implemented through #61213. It was instead removed from 6.2 and 6.3 in [58301] and [58330], respectively.

This removes the workflow for these branches from the strategy matrix when testing old branches to avoid errors and adds an expanded note about why those branches are missing.

See #61213, #61564.

File:

• trunk/.github/workflows/test-old-branches.yml (modified) (2 diffs)

trunk/.github/workflows/test-old-branches.yml

@@ -93 +93 @@
             workflow: 'end-to-end-tests.yml'
 
-          # Performance testing was introduced in 6.2.
+          # Performance testing was introduced in 6.2 using Puppeteer but was overhauled to use Playwright instead in 6.4.
+          # Since the workflow frequently failed for 6.2 and 6.3 due to the flaky nature of the Puppeteer tests,
+          # the workflow was removed from those two branches.
           - branch: '6.6'
             workflow: 'performance.yml'
@@ -99 +101 @@
             workflow: 'performance.yml'
           - branch: '6.4'
-            workflow: 'performance.yml'
-          - branch: '6.3'
-            workflow: 'performance.yml'
-          - branch: '6.2'
             workflow: 'performance.yml'