Skip to content

[3006.x] Python 3.11 #68018

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

Open
wants to merge 17 commits into
base: 3006.x
Choose a base branch
from
Open

[3006.x] Python 3.11 #68018

wants to merge 17 commits into from

Conversation

dwoz
Copy link
Contributor

@dwoz dwoz commented May 13, 2025

What does this PR do?

Updates the python shipped with 3006 release from 3.10 to 3.11

Why would we make this change in 3006.x?

We've had to postpone several milestones on the way to a 3008 LTS release. This means that we'll now be on the hook for maintaining 3006 LTS with security update longer than Python 3.10 will receive security updates.

@dwoz dwoz requested a review from a team as a code owner May 13, 2025 01:14
@dwoz dwoz changed the title [wip] [3006.x] Try 3.11 3006.x [wip] [3006.x] Python 3.11 May 13, 2025
@dwoz dwoz added the test:full Run the full test suite label May 13, 2025
@twangboy
Copy link
Contributor

Cherry-picked ff296ea to fix the failing Windows tests

@@ -12,6 +12,8 @@
from tests.support.mixins import SaltReturnAssertsMixin
from tests.support.runtests import RUNTIME_VARS

KEY = "AAAAC3NzaC1lZDI1NTE5AAAAIEmY2ZOSneFGN+b7+S9eLPGOqPF2yl9AgTCkZHuWKjC5"
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't it bad to create a global variable in the test suite

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe it's not very pytest-ee. Though, the alternative of making a fixture also creates a global variable. I.E. Function definitions are also variables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test:full Run the full test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants