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

Mark NoGIL Linux & MacOS builders as stable by itamaro · Pull Request #444 · python/buildmaster-config · GitHub

Repository navigation

Mark NoGIL Linux & MacOS builders as stable - #444

Merged
itamaro merged 1 commit into
python:mainfrom
itamaro:nogil-stable-builders
Nov 19, 2023
Merged

itamaro merged 1 commit into
python:mainfrom
itamaro:nogil-stable-builders

Conversation

itamaro commented Nov 18, 2023

Copy link
Copy Markdown
Contributor

these builders have been very stable over the last few weeks.
with the official acceptance of PEP 703, I propose we promote these builders to a stable status.

note that the M1 builders was previously wrongly marked as "tier-1 unstable". according to PEP 11, Apple aarch64 is still considered tier-2, so until that changes, I moved the M1 NoGIL builder to "tier-2 stable".

Comment thread master/custom/builders.py

# macOS x86-64 clang
("x86-64 macOS", "billenstein-macos", UnixBuild),
("x86-64 MacOS Intel NoGIL", "itamaro-macos-intel-aws", UnixNoGilBuild),

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

If free-threaded CIs are triggered anyway, Can we update this buildbot as refleak checker?

Copy link
Copy Markdown
Contributor Author

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

afaik the buildbot test suite is more comprehensive than the equivalent CI test suite (e.g. due to --fast-ci flag for example), so there's value in a buildbot builder with "equivalent configuration". I also believe the release manager considers the buildbot statuses when making release decisions, not GitHub Actions.

I also noticed there are no refleak builders for MacOS at all (for the default build) -- would it be valuable to add such builders? (for default and/or nogil)

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

afaik the buildbot test suite is more comprehensive than the equivalent CI test suite

I see!

I also noticed there are no refleak builders for MacOS at all (for the default build) -- would it be valuable to add such builders? (for default and/or nogil)

Oh, I didn't know that :( If possible for me, "yes"

Copy link
Copy Markdown
Contributor Author

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

If possible for me, "yes"

let me keep the existing builders as is (just marked as stable), and I'll look into adding refleak macos builders separately (probably marked "unstable" initially)

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

I also believe the release manager considers the buildbot statuses when making release decisions, not GitHub Actions.

Yes, it's here: https://peps.python.org/pep-0101/#how-to-make-a-release

  • Check the stable buildbots.

    Go to https://buildbot.python.org/all/#/release_status

    Look at the buildbots for the release you’re making. Ignore any that are offline (or inform the community so they can be restarted). If what remains are (mostly) green buildbots, you’re good to go. If you have non-offline red buildbots, you may want to hold up the release until they are fixed. Review the problems and use your judgement, taking into account whether you are making an alpha, beta, or final release.

Although GitHub Actions is kind of implicitly included, because all PRs should be green before merging in the first place.

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL