Skip to content

Pin uvloop<0.22 and add sanity test for it#7432

Open
pedro-psb wants to merge 1 commit intopulp:mainfrom
pedro-psb:pin-uvloop-and-add-test
Open

Pin uvloop<0.22 and add sanity test for it#7432
pedro-psb wants to merge 1 commit intopulp:mainfrom
pedro-psb:pin-uvloop-and-add-test

Conversation

@pedro-psb
Copy link
Member

0.22.1 is known to broke some existing behavior. Maybe there is a fix in our code, but we'll wait if they can revert/fix it on their side.
See MagicStack/uvloop#702.

Closes: #7213

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

0.22.1 is known to broke some existing behavior. Maybe there is a fix in
our code, but we'll wait if they can revert it.
See <MagicStack/uvloop#702>.

Closes: pulp#7213
@pedro-psb pedro-psb force-pushed the pin-uvloop-and-add-test branch from 7934d3b to b7b015f Compare March 9, 2026 19:41
@pedro-psb
Copy link
Member Author

Maybe the post_before_script isnt the best place to put it (since now it's labelled as "Dump Metadata"), but I wanted it to run before the other tests. This is mostly for dependabot, I think.

I've tested that it correctly fails without the bump, and that other errors (like missing setuptools) causes the script to fail too.

https://github.com/pulp/pulpcore/actions/runs/22871481521/job/66352488860?pr=7432#step:13:752

@mdellweg
Copy link
Member

How about using a test scenario to run with uv?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PULP-1124] pulpcore-content 3.92 doesn't start with uvloop

2 participants