-
Notifications
You must be signed in to change notification settings - Fork 14.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Trigger provider openlineage tests after assets change #44026
Labels
Comments
Merged
dosubot
bot
added
area:CI
Airflow's tests and continious integration
provider:openlineage
AIP-53
labels
Nov 14, 2024
cc: @vatsrahul1001 @uranusjr => we might fix it to happen automatically (this test was broken by #41325) but kind request - until when you add new fields to assets etc, plese add "full tests needed" to the PR :) |
Yeah running openlineage tests would be good :) |
@potiuk I will look into this |
assigned you @vatsrahul1001 ! |
Yes. That's likely best idea |
github-project-automation
bot
moved this from In progress
to Done
in CI / CD planned work
Nov 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
When running selective checks, we should automatically trigger open-lineage provider tests (in main) in any PR that changes assets, as openlineage tests heavily rely on assets and are often broken after asset changes. Example #44025 fixing broken test:
The text was updated successfully, but these errors were encountered: