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

Remove requests limits and restore opensearch tests for latest boto #46289

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jan 30, 2025

The requests limits were removing opensearch during "latest boto" tests but this is not needed any more, since apache.beam latest version already supports requests 2.32.*. That also unblocks moving opensearch to a separate provider project.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

The requests limits were removing opensearch during "latest boto"
tests but this is not needed any more, since apache.beam latest
version already supports requests 2.32.*. That also unblocks moving
opensearch to a separate provider project.
@potiuk potiuk requested a review from ashb as a code owner January 30, 2025 13:41
@potiuk potiuk requested a review from amoghrajesh January 30, 2025 13:41
@potiuk potiuk requested a review from eladkal January 30, 2025 13:44
@potiuk potiuk added the canary When set on PR running from apache repo - behave as canary run label Jan 30, 2025
@potiuk potiuk closed this Jan 30, 2025
@potiuk potiuk reopened this Jan 30, 2025
@potiuk
Copy link
Member Author

potiuk commented Jan 30, 2025

Applied canary label to test latest boto tests.

@potiuk potiuk merged commit a1c8717 into apache:main Jan 30, 2025
101 of 125 checks passed
@potiuk potiuk deleted the remove-requests-exclusions branch January 30, 2025 15:30
@potiuk
Copy link
Member Author

potiuk commented Jan 30, 2025

Merged - failing tests are addressed in #46291

niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
…pache#46289)

The requests limits were removing opensearch during "latest boto"
tests but this is not needed any more, since apache.beam latest
version already supports requests 2.32.*. That also unblocks moving
opensearch to a separate provider project.
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
…pache#46289)

The requests limits were removing opensearch during "latest boto"
tests but this is not needed any more, since apache.beam latest
version already supports requests 2.32.*. That also unblocks moving
opensearch to a separate provider project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-tools canary When set on PR running from apache repo - behave as canary run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants