Tests: Fix testcontainers. CI: Use uv across the board.#686
Conversation
WalkthroughThis pull request consolidates Python environment setup in GitHub Actions workflows by removing explicit Python installation steps and centralizing configuration within the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
b95d6ed to
a5bd00b
Compare
Integration tests became flaky again. Let's investigate later. kinesis.exceptions.StreamDoesNotExist: Stream 'testdrive' does not exist
87602d1 to
433524e
Compare
| "3.13", | ||
| # TODO: Integration tests became flaky again. Let's investigate later. | ||
| # kinesis.exceptions.StreamDoesNotExist: Stream 'testdrive' does not exist | ||
| # "3.13", |
There was a problem hiding this comment.
We reported this or a related problem already. Apparently, it is coming back, now on Python 3.13? We think it is most probably related to our integration test setup and related timing changes by cleaning up the testcontainers rig recently. As the comment says: Let's investigate later.
/cc @hampsterx
Just maintenance, effectively various improvements to the CI configuration.