Skip to content
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

Fix sdist verification in CI after we standardized packaging #37406

Merged
merged 1 commit into from
Feb 13, 2024

Commits on Feb 13, 2024

  1. Fix sdist verification in CI after we standardized packaging

    Airflow Sdist packages have been broken by apache#37340 and fixed by 37388,
    but we have not noticed it because CI check for sdist packages has
    been broken since apache#36537 where we standardized naming of the sdist
    packages to follow modern syntax (and we silently skipped installation
    because no providers were found),.
    
    This PR fixes it:
    
    * changes the naming format expected to follow the new standard
    * treats "no providers found as error"
    
    The "no providers" as success was useful at some point of time when we
    run sdist as part of regular PRs and some PRs resulted in "no providers
    changed" condition, however sdist verification only happens now in
    canary build (so all providers are affected) as well as we have if
    condition in the job itself to skip the step of installation if there
    are no providers.
    potiuk committed Feb 13, 2024
    Configuration menu
    Copy the full SHA
    22e47ef View commit details
    Browse the repository at this point in the history