Skip to content

Sync uv.lock and provider-deps checksum for #66612#68620

Merged
Lee-W merged 1 commit into
apache:mainfrom
FrankYang0529:minor-fix-generated
Jun 17, 2026
Merged

Sync uv.lock and provider-deps checksum for #66612#68620
Lee-W merged 1 commit into
apache:mainfrom
FrankYang0529:minor-fix-generated

Conversation

@FrankYang0529

@FrankYang0529 FrankYang0529 commented Jun 16, 2026

Copy link
Copy Markdown
Member

#66612 added SQL auto-lineage support to the Informatica provider and declared two new dependencies in
providers/informatica/pyproject.tomlsqlglot>=30.0.0 and the common-sql extra (apache-airflow-providers-common-sql). It updated generated/provider_dependencies.json, but the matching uv.lock and
generated/provider_dependencies.json.sha256sum were not regenerated, leaving the repository in an inconsistent state.

The task generate-tasksdk-datamodels in pre-commit-config runs uv run -p 3.12 --no-progress --active --group codegen --project apache-airflow-task-sdk --directory task-sdk -s dev/generate_task_sdk_models.py and updates uv.lock file.

related: #66612

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

Signed-off-by: PoAn Yang <payang@apache.org>
@eladkal

eladkal commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Fix error in Generate Datamodels for TaskSDK client.

related: #66612

Can you be more specific? traceback? reproduce steps?
The related issue does not clarify what is the problem you are trying to solve

@FrankYang0529 FrankYang0529 changed the title fix(informatica): add required packages fix(informatica): sync uv.lock and provider-deps checksum for #66612 Jun 16, 2026
@FrankYang0529 FrankYang0529 marked this pull request as ready for review June 16, 2026 11:39
@FrankYang0529

FrankYang0529 commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Can you be more specific? traceback? reproduce steps?
The related issue does not clarify what is the problem you are trying to solve

Sorry for that. I updated PR description for more background. I will avoid to give unclear context next time. I tried to fix OpenLineage issue for s3 (#68619) and cannot push commit successfully cause of uv.lock and generated/provider_dependencies.json.sha256sum. Use another PR for this because it's not related to the change in #68619.

@Lee-W Lee-W left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also encountered the same issue during development. and just verified it's still there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants