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.

Sign up for GitHub

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

Jump to bottom

gh-125017: Fix refleak from GH-125636 #125664

Merged
merged 1 commit into from
Oct 17, 2024
Merged

gh-125017: Fix refleak from GH-125636 #125664

merged 1 commit into from
Oct 17, 2024

Conversation

zware
Copy link
Member

@zware zware commented Oct 17, 2024 •
edited by bedevere-app bot
Loading

@bedevere-app bedevere-app bot mentioned this pull request Oct 17, 2024
Closed
@zware
Copy link
Member Author

zware commented Oct 17, 2024

!buildbot Refleak

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @zware for commit 6d876da 🤖

The command will test the builders whose names match following regular expression: Refleak

The builders matched are:

  • PPC64LE Fedora Stable Refleaks PR
  • PPC64LE Fedora Rawhide Refleaks PR
  • aarch64 CentOS9 Refleaks PR
  • AMD64 Fedora Rawhide NoGIL refleaks PR
  • AMD64 FreeBSD Refleaks PR
  • AMD64 RHEL8 Refleaks PR
  • AMD64 Windows11 Refleaks PR
  • s390x RHEL8 Refleaks PR
  • AMD64 Fedora Stable Refleaks PR
  • AMD64 Fedora Rawhide Refleaks PR
  • AMD64 CentOS9 NoGIL Refleaks PR
  • ARM64 MacOS M1 Refleaks NoGIL PR
  • PPC64LE CentOS9 Refleaks PR
  • s390x RHEL9 Refleaks PR
  • aarch64 RHEL8 Refleaks PR
  • AMD64 CentOS9 Refleaks PR
  • aarch64 Fedora Stable Refleaks PR
  • aarch64 Fedora Rawhide Refleaks PR
  • PPC64LE RHEL8 Refleaks PR
  • PPC64LE Fedora Rawhide NoGIL refleaks PR
  • aarch64 Fedora Rawhide NoGIL refleaks PR

@zware zware requested a review from JelleZijlstra October 17, 2024 21:45
JelleZijlstra
JelleZijlstra approved these changes Oct 17, 2024
View reviewed changes
Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

Thanks, should have run the refleak bots on my PR.

@zware
Copy link
Member Author

zware commented Oct 17, 2024

If we ran the refleak bots on every PR that touches C code, they'd never catch up :)

@zware zware merged commit c3164ae into python:main Oct 17, 2024
42 of 44 checks passed
@zware zware deleted the fix_test_descr_refleak branch October 17, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JelleZijlstra JelleZijlstra JelleZijlstra 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
@zware @bedevere-bot @JelleZijlstra