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

Add Python 3.13 testing #1379

Merged
merged 4 commits into from
Jul 23, 2024
Merged

Add Python 3.13 testing #1379

merged 4 commits into from
Jul 23, 2024

Conversation

Shrews
Copy link
Contributor

@Shrews Shrews commented Jul 1, 2024

Add testing environment for future Python 3.13 release.

  • Python 3.13 seems to cause the test_keepalive_setting() integration tests to fail on a fairly consistent basis at the assert for the number of keepalive events (we exceed the hardcoded value of 5). Speculation is that there is a change in this version that causes these tests to fail more consistently on a low-powered machine with test parallelization. Increasing the upper "wobble" value to account for this.
  • Fixes an issue exposed by Python 3.13 with the Base64IO context manager and an undeclared attribute within the close() method.

Co-authored-by: Adam Williamson [email protected]

@github-actions github-actions bot added the needs_triage New item that needs to be triaged label Jul 1, 2024
@Shrews Shrews force-pushed the py313 branch 2 times, most recently from 4332608 to 55c9d94 Compare July 10, 2024 17:43
@github-actions github-actions bot added the test Changes to test files label Jul 10, 2024
@Shrews Shrews marked this pull request as ready for review July 12, 2024 18:56
@Shrews Shrews requested a review from a team as a code owner July 12, 2024 18:56
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
test/requirements.txt Outdated Show resolved Hide resolved
@Shrews Shrews merged commit f744e99 into ansible:devel Jul 23, 2024
14 checks passed
@Shrews Shrews deleted the py313 branch July 23, 2024 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs_triage New item that needs to be triaged test Changes to test files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants