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

gh-83714: Only use STATX_MNT_ID & STATX_SUBVOL if they're defined by encukou · Pull Request #140446 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-83714: Only use STATX_MNT_ID & STATX_SUBVOL if they're defined - #140446

Merged
encukou merged 3 commits into
python:mainfrom
encukou:guard-subvol
Oct 22, 2025
ConversationCommits (3)ChecksFiles changed
Merged

gh-83714: Only use STATX_MNT_ID & STATX_SUBVOL if they're defined#140446
encukou merged 3 commits into
python:mainfrom
encukou:guard-subvol

Conversation

encukou commented Oct 22, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

bedevere-app Bot mentioned this pull request Oct 22, 2025
Closed

encukou commented Oct 22, 2025

Copy link
Copy Markdown
Member Author

!buildbot AMD64.CentOS9.NoGIL

bedevere-bot commented Oct 22, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @encukou for commit e879800 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F140446%2Fmerge

The command will test the builders whose names match following regular expression: AMD64.CentOS9.NoGIL

The builders matched are:

  • AMD64 CentOS9 NoGIL PR
  • AMD64 CentOS9 NoGIL Refleaks PR

encukou commented Oct 22, 2025

Copy link
Copy Markdown
Member Author

!buildbot AMD64.CentOS9.NoGIL$

bedevere-bot commented Oct 22, 2025

Copy link
Copy Markdown

The regex 'AMD64.CentOS9.NoGIL$' did not match any buildbot builder. Is the requested builder in the list of stable builders?

encukou commented Oct 22, 2025

Copy link
Copy Markdown
Member Author

!buildbot AMD64.CentOS9.NoGIL

bedevere-bot commented Oct 22, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @encukou for commit f0d5edb 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F140446%2Fmerge

The command will test the builders whose names match following regular expression: AMD64.CentOS9.NoGIL

The builders matched are:

  • AMD64 CentOS9 NoGIL PR
  • AMD64 CentOS9 NoGIL Refleaks PR

vstinner approved these changes Oct 22, 2025
View reviewed changes

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. I wrote a similar change before seeing yours :-)

vstinner commented Oct 22, 2025

Copy link
Copy Markdown
Member

Oh sorry, I created a conflict when I merged my other statx() change.

encukou commented Oct 22, 2025

Copy link
Copy Markdown
Member Author

!buildbot AMD64.CentOS9.NoGIL

bedevere-bot commented Oct 22, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @encukou for commit 0c95f7a 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F140446%2Fmerge

The command will test the builders whose names match following regular expression: AMD64.CentOS9.NoGIL

The builders matched are:

  • AMD64 CentOS9 NoGIL PR
  • AMD64 CentOS9 NoGIL Refleaks PR

encukou commented Oct 22, 2025

Copy link
Copy Markdown
Member Author

Could you please run buildbots for any other statx changes?

encukou merged commit e53e9eb into python:main Oct 22, 2025
45 checks passed
encukou deleted the guard-subvol branch October 22, 2025 10:42

vstinner commented Oct 22, 2025

Copy link
Copy Markdown
Member

Could you please run buildbots for any other statx changes?

Ok, I will. We ran buildbots on the change causing the regression, but only some buildbots.

StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
pythongh-83714: Only use STATX_MNT_ID & STATX_SUBVOL if they're defin…
4d159c6 
…ed (pythonGH-140446)
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

vstinner vstinner approved these changes

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL