Skip to content

Replace oss2 with alibabacloud-oss-v2 - #64361

Merged
eladkal merged 7 commits into
apache:mainfrom
sunny7712:chore/update-alibaba-oss-sdk
Mar 29, 2026
Merged

eladkal merged 7 commits into
apache:mainfrom
sunny7712:chore/update-alibaba-oss-sdk

Conversation

@sunny7712

@sunny7712 sunny7712 commented Mar 28, 2026 •

Copy link
Copy Markdown
Contributor

Changes are backwards compatible.
oss2 library is not maintained and replaced by alibabacloud-oss-v2


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

  • 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.

@boring-cyborg

boring-cyborg Bot commented Mar 28, 2026

Copy link
Copy Markdown

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
Here are some useful points:

  • Pay attention to the quality of your code (ruff, mypy and type annotations). Our prek-hooks will help you with that.
  • In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
  • Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
  • Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
  • Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
  • Be sure to read the Airflow Coding style.
  • Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
    Apache Airflow is a community-driven project and together we are making it better 🚀.
    In case of doubts contact the developers at:
    Mailing List: dev@airflow.apache.org
    Slack: https://s.apache.org/airflow-slack

@eladkal eladkal added the all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs label Mar 28, 2026
@eladkal eladkal changed the title Chore/update alibaba oss sdk Replace oss2 with alibabacloud-oss-v2 Mar 28, 2026
@eladkal
eladkal self-requested a review March 28, 2026 13:48

@eladkal eladkal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
Pending fixing tests

…cies

- Updated test cases in test_oss.py to utilize alibabacloud_oss_v2 instead of oss2.
- Changed mock patches to reflect the new structure of OSSHook methods.
- Removed dependencies on the old oss2 package and added alibabacloud-oss-v2 to the project.
- Updated the lock file to reflect the new versioning.
@sunny7712
sunny7712 force-pushed the chore/update-alibaba-oss-sdk branch from 513417a to 7211ff0 Compare March 29, 2026 06:46
@sunny7712
sunny7712 force-pushed the chore/update-alibaba-oss-sdk branch from 7211ff0 to 7532d58 Compare March 29, 2026 06:48
@eladkal

eladkal commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Merging. Failures are due to broken main

@eladkal
eladkal merged commit 272e50a into apache:main Mar 29, 2026
162 of 167 checks passed
@boring-cyborg

boring-cyborg Bot commented Mar 29, 2026

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

aaron-y-chen pushed a commit to aaron-y-chen/airflow that referenced this pull request Mar 30, 2026
* Refactor OSSHook tests to use alibabacloud_oss_v2 and update dependencies

- Updated test cases in test_oss.py to utilize alibabacloud_oss_v2 instead of oss2.
- Changed mock patches to reflect the new structure of OSSHook methods.
- Removed dependencies on the old oss2 package and added alibabacloud-oss-v2 to the project.
- Updated the lock file to reflect the new versioning.

* Add validation for missing bucket_name parameter in OSSHook

* Update alibabacloud-oss-v2 dependency version to 1.2.4 in configuration files

* Update exclude-newer date and prek dependency version in uv.lock

* Update alibabacloud-oss-v2 dependency version to 1.2.0 in configuration files

* Add 'alibabacloud' to spelling wordlist

* regenerated uv lock file
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
* Refactor OSSHook tests to use alibabacloud_oss_v2 and update dependencies

- Updated test cases in test_oss.py to utilize alibabacloud_oss_v2 instead of oss2.
- Changed mock patches to reflect the new structure of OSSHook methods.
- Removed dependencies on the old oss2 package and added alibabacloud-oss-v2 to the project.
- Updated the lock file to reflect the new versioning.

* Add validation for missing bucket_name parameter in OSSHook

* Update alibabacloud-oss-v2 dependency version to 1.2.4 in configuration files

* Update exclude-newer date and prek dependency version in uv.lock

* Update alibabacloud-oss-v2 dependency version to 1.2.0 in configuration files

* Add 'alibabacloud' to spelling wordlist

* regenerated uv lock file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs area:providers kind:documentation provider:alibaba

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants