potiuk commented on code in PR #58901:
URL: https://github.com/apache/airflow/pull/58901#discussion_r2603004788


##########
.github/workflows/additional-prod-image-tests.yml:
##########
@@ -218,6 +222,17 @@ jobs:
       use-uv: ${{ inputs.use-uv }}
       e2e_test_mode: "remote_log"
 
+  test-ui-e2e-tests:

Review Comment:
   You do not even need to have matrix. What I propose here is to have three 
jobs here - separate job for every browser - named appropriately with the 
browser in name of the job and description reflecting the browser "UI e2e 
Chromium tests with PROD image"  - for long names, the "distinct" part of the 
name should be close to the left otherwise it is cut in the UI.
   
   
   This will significantly decrease elapsed time of execution for the e2e tests 
1/3 and it will not even increase the build time - simply more jobs to run with 
a little overhead for each job added.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to