shahar1 commented on code in PR #69671:
URL: https://github.com/apache/airflow/pull/69671#discussion_r3557846956
##########
.github/workflows/additional-prod-image-tests.yml:
##########
@@ -329,34 +325,17 @@ jobs:
e2e_test_mode: "go_sdk"
if: inputs.canary-run == 'true' || inputs.run-go-sdk-e2e-tests == 'true'
- test-providers-e2e-tests-openlineage:
- name: "Provider E2E tests openlineage"
- uses: ./.github/workflows/providers-e2e-tests.yml
- with:
- provider: "openlineage"
- provider-display-name: "OpenLineage"
- runners: ${{ inputs.runners }}
- platform: ${{ inputs.platform }}
- default-python-version: "${{ inputs.default-python-version }}"
- use-uv: ${{ inputs.use-uv }}
- if: inputs.canary-run == 'true' ||
inputs.run-providers-e2e-tests-openlineage == 'true'
-
- test-providers-e2e-tests-openlineage-compat:
Review Comment:
It was an AI overusage side-effect, apologies for that :)
I will restore the coverage for older versions, hopefully Jarek won't be
disappointed by increase in number of lines.
However, just wondering if it's really useful to run it for **all** old 3.X
versions (3.0, 3.1, 3.2, 3.3), and not only the oldest one (3.0). We're
currently in shortage of CI resources, so each job adds its own overhead
(multiply by number of PRs per time unit) - which we try to reduce in these
days.
What do you think?
--
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]
