FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-138744: GitHub Actions: pin to `windows-2022` by hugovk · Pull Request #138743 · python/cpython · GitHub

/ cpython Public

gh-138744: GitHub Actions: pin to windows-2022 - #138743

Merged
hugovk merged 2 commits into
python:mainfrom
hugovk:3.15-pin-windows-2022
Sep 10, 2025
Merged

hugovk merged 2 commits into
python:mainfrom
hugovk:3.15-pin-windows-2022

Conversation

hugovk commented Sep 10, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

GitHub Actions is in the process of migrating windows-latest from windows-2022 to windows-2025:

This change will be rolled out over a period of several weeks beginning 2025-09-02 and will complete on 2025-09-30

actions/runner-images#12677

This is causing failures for forks which have already been migrated, for example:

======================================================================
FAIL: test_function_entry_return (test.test_dtrace.DTraceNormalTests.test_function_entry_return)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 38, in normalize_trace_output
    result.sort(key=lambda row: int(row[0]))
    ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 38, in <lambda>
    result.sort(key=lambda row: int(row[0]))
                                ~~~^^^^^^^^
ValueError: invalid literal for int() with base 10: 'dtrace: failed to compile script D:\\a\\cpython\\cpython\\Lib\\test\\dtracedata\\call_stack.d: line 3: pid provider is not installed on this system'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 124, in test_function_entry_return
    self.run_case("call_stack")
    ~~~~~~~~~~~~~^^^^^^^^^^^^^^
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 119, in run_case
    actual_output, expected_output = self.backend.run_case(
                                     ~~~~~~~~~~~~~~~~~~~~~^
        name, optimize_python=self.optimize_python)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 53, in run_case
    actual_output = normalize_trace_output(self.trace_python(
        script_file=abspath(name + self.EXTENSION),
        python_file=abspath(name + ".py"),
        optimize_python=optimize_python))
  File "D:\a\cpython\cpython\Lib\test\test_dtrace.py", line 42, in normalize_trace_output
    raise AssertionError(
        "tracer produced unparsable output:\n{}".format(output)
    )
AssertionError: tracer produced unparsable output:
dtrace: failed to compile script D:\a\cpython\cpython\Lib\test\dtracedata\call_stack.d: line 3: pid provider is not installed on this system

https://github.com/picnixz/cpython/actions/runs/17610564159/job/50031376950#step:1:9

Upstream and my fork are still on 2022, but let's pin to windows-2022 so we can separately investigate and fix for 2025, and then pin to windows-2025.

hugovk added OS-windows infra CI, GitHub Actions, buildbots, Dependabot, etc. labels Sep 10, 2025
picnixz changed the title GitHub Actions: pin to windows-22.04 gh-138744: GitHub Actions: pin to windows-22.04 Sep 10, 2025
hugovk removed the skip issue label Sep 10, 2025

hugovk commented Sep 10, 2025

Copy link
Copy Markdown
Member Author

vstinner changed the title gh-138744: GitHub Actions: pin to windows-22.04 gh-138744: GitHub Actions: pin to windows-2022 Sep 10, 2025

vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM. It's a good idea to pin the Windows version and only upgrade it once it's well tested on newer Windows.

sobolevn left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks! Pinning is a good idea in any case.

Question: does this new version have os.getpid?

hugovk commented Sep 10, 2025

Copy link
Copy Markdown
Member Author

Question: does this new version have os.getpid?

Looks like they both do:

hugovk merged commit 6e78a53 into python:main Sep 10, 2025
107 of 108 checks passed
hugovk deleted the 3.15-pin-windows-2022 branch September 10, 2025 14:28

Copy link
Copy Markdown

Thanks @hugovk for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10, 3.11, 3.12, 3.13, 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 10, 2025
(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

Copy link
Copy Markdown

Sorry, @hugovk, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 6e78a539bfb406238ec251ba01b7a1819e5c303e 3.13

Copy link
Copy Markdown

Sorry, @hugovk, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 6e78a539bfb406238ec251ba01b7a1819e5c303e 3.12

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138752 is a backport of this pull request to the 3.14 branch.

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Sep 10, 2025

Copy link
Copy Markdown

Sorry, @hugovk, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 6e78a539bfb406238ec251ba01b7a1819e5c303e 3.11

Copy link
Copy Markdown

Sorry, @hugovk, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 6e78a539bfb406238ec251ba01b7a1819e5c303e 3.10

Copy link
Copy Markdown

Sorry, @hugovk, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 6e78a539bfb406238ec251ba01b7a1819e5c303e 3.9

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138753 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 only security fixes label Sep 10, 2025
hugovk added a commit to hugovk/cpython that referenced this pull request Sep 10, 2025
…GH-138743)

(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
hugovk added a commit to hugovk/cpython that referenced this pull request Sep 10, 2025
…GH-138743)

(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138754 is a backport of this pull request to the 3.12 branch.

bedevere-app Bot removed the needs backport to 3.12 only security fixes label Sep 10, 2025
hugovk added a commit to hugovk/cpython that referenced this pull request Sep 10, 2025
…GH-138743)

(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138755 is a backport of this pull request to the 3.11 branch.

bedevere-app Bot removed the needs backport to 3.11 only security fixes label Sep 10, 2025
hugovk added a commit to hugovk/cpython that referenced this pull request Sep 10, 2025
…GH-138743)

(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138757 is a backport of this pull request to the 3.10 branch.

bedevere-app Bot removed the needs backport to 3.10 only security fixes label Sep 10, 2025
hugovk added a commit to hugovk/cpython that referenced this pull request Sep 10, 2025
…H-138743)

(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

bedevere-app Bot commented Sep 10, 2025

Copy link
Copy Markdown

GH-138758 is a backport of this pull request to the 3.9 branch.

hugovk added a commit that referenced this pull request Sep 10, 2025
…138752)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
ambv pushed a commit that referenced this pull request Sep 13, 2025
ambv pushed a commit that referenced this pull request Sep 13, 2025
…H-138755)

* [3.11] gh-138744: GitHub Actions: pin to `windows-2022` (GH-138743)
(cherry picked from commit 6e78a53)

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

* Update .azure-pipelines/libffi-build.yml
ambv pushed a commit that referenced this pull request Sep 13, 2025
Yhg1s pushed a commit that referenced this pull request Sep 26, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra CI, GitHub Actions, buildbots, Dependabot, etc. OS-windows skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL