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.

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

Promote aarch64-apple-darwin to Tier 1 #3671

Merged

Conversation

shepmaster
Copy link
Member

@shepmaster shepmaster commented Jul 9, 2024

@shepmaster shepmaster added T-lang Relevant to the language team, which will review and decide on the RFC. T-infra Relevant to the infrastructure team, which will review and decide on the RFC. T-release Relevant to the release team, which will review and decide on the RFC. T-compiler Relevant to the compiler team, which will review and decide on the RFC. and removed T-lang Relevant to the language team, which will review and decide on the RFC. labels Jul 9, 2024
@clarfonthey
Copy link
Contributor

I'll be honest, I didn't realise this wasn't already tier 1. This seems highly uncontroversial.

@compiler-errors
Copy link
Member

I don't see any reason to not kick off this FCP immediately, given the number of people on the FCP.

Regarding the specific team set, this is the same set of teams used for the FCP for aarch64-unknown-linux-gnu.

@rfcbot fcp merge

@rfcbot
Copy link
Collaborator

rfcbot commented Jul 10, 2024

Team member @compiler-errors has proposed to merge this. The next step is review by the rest of the tagged team members:

No concerns currently listed.

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@rfcbot rfcbot added proposed-final-comment-period Currently awaiting signoff of all team members in order to enter the final comment period. disposition-merge This RFC is in PFCP or FCP with a disposition to merge it. labels Jul 10, 2024
@tmandry
Copy link
Member

tmandry commented Jul 10, 2024

Tier 1 status requires increased continuous integration resource usage which means increased cost to the project. However, the aarch64-apple-darwin target has been treated as Tier 1 since rust-lang/rust#120509 without causing financial concern.

Server availability and cost are my only reservation. Five months is long enough to have data, but we should make sure we are sustainable in the long run, even if our current sponsorships don't continue.

I expect the infra team will be able to weigh in on this after consulting with the foundation, so I'll check my box for now and trust that the due diligence will happen.

@rfcbot reviewed

@pietroalbini
Copy link
Member

Server availability and cost are my only reservation. Five months is long enough to have data, but we should make sure we are sustainable in the long run, even if our current sponsorships don't continue.

I expect the infra team will be able to weigh in on this after consulting with the foundation, so I'll check my box for now and trust that the due diligence will happen.

Honestly, I'd expect that as time goes by the problem will not be sourcing aarch64 apple builders, but rather x86_64 apple builders. For example, another CI provider (CircleCI) is already sunsetting x86 apple support, and I expect that GitHub Actions will over time retire their x86 apple runners.

@evelynharthbrooke
Copy link

evelynharthbrooke commented Jul 18, 2024

Server availability and cost are my only reservation. Five months is long enough to have data, but we should make sure we are sustainable in the long run, even if our current sponsorships don't continue.
I expect the infra team will be able to weigh in on this after consulting with the foundation, so I'll check my box for now and trust that the due diligence will happen.

Honestly, I'd expect that as time goes by the problem will not be sourcing aarch64 apple builders, but rather x86_64 apple builders. For example, another CI provider (CircleCI) is already sunsetting x86 apple support, and I expect that GitHub Actions will over time retire their x86 apple runners.

when and if github retires the x86_64 macOS runners, the question at that point in time would likely (if not definitely) become "should x86_64-apple-darwin still be supported as a tier 1 target?" and the answer to that question would most likely be a no as without the ability to run CI on 64-bit macOS, the rust project cannot adequately test and/or verify that the x86_64 macOS build of Rust will continue to work, unless enough money is collected to build and maintain a self-hosted GitHub Runner, but by that time its highly likely that apple will have dropped support for intel macs completely by then, and by that time it'll likely be time to retire rust's support for x86_64 macOS anyway. but if i had to guess, while macs based on apple silicon are likely the most popular type of mac at this point, github will likely drop their intel mac runners shortly or soon after the last intel-based mac receives its final software update (be it security or final point release or what have you).

that's my two cents on the matter, but for now, the intel macOS runner shouldn't be discontinued on github for a while yet.

@shepmaster
Copy link
Member Author

shepmaster commented Jul 18, 2024

The RFC already states

There are no concrete plans to demote x86_64-apple-darwin at this time and any such demotion would need its own well-publicized RFC.

No further discussion of removal / deprecation / demotion of x86_64-apple-darwin is needed on this RFC. I'll hide future comments on that topic as I see fit.

@rfcbot rfcbot added final-comment-period Will be merged/postponed/closed in ~10 calendar days unless new substational objections are raised. and removed proposed-final-comment-period Currently awaiting signoff of all team members in order to enter the final comment period. labels Jul 18, 2024
@rfcbot
Copy link
Collaborator

rfcbot commented Jul 18, 2024

🔔 This is now entering its final comment period, as per the review above. 🔔

@rfcbot rfcbot added finished-final-comment-period The final comment period is finished for this RFC. and removed final-comment-period Will be merged/postponed/closed in ~10 calendar days unless new substational objections are raised. labels Jul 28, 2024
@rfcbot
Copy link
Collaborator

rfcbot commented Jul 28, 2024

The final comment period, with a disposition to merge, as per the review above, is now complete.

As the automated representative of the governance process, I would like to thank the author for their work and everyone else who contributed.

This will be merged soon.

Copy link
Member

@compiler-errors compiler-errors left a comment

Choose a reason for hiding this comment

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

OK, this looks ready to go. I updated the tracking issue to point to rust-lang/rust#73908, since that seems like a convenient place to point this. Thanks for putting up this RFC, @shepmaster.

@compiler-errors compiler-errors merged commit a48e9d3 into rust-lang:master Aug 3, 2024
evelynharthbrooke added a commit to evelynharthbrooke/rust that referenced this pull request Aug 3, 2024
evelynharthbrooke added a commit to evelynharthbrooke/rust that referenced this pull request Aug 3, 2024
evelynharthbrooke added a commit to evelynharthbrooke/rust that referenced this pull request Aug 3, 2024
evelynharthbrooke added a commit to evelynharthbrooke/rust that referenced this pull request Aug 3, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 11, 2024
…imulacrum

Promote aarch64-apple-darwin to Tier 1

This promotes aarch64-apple-darwin to Tier 1 status as per rust-lang/rfcs#3671 and tracking issue rust-lang#73908. Not sure what else is necessary for this to impement the aforementioned RFC, however I figured I'd try. I did read in previous issues and PRs that the necessary infrastructure was already in place for the aarch64-apple-darwin target, and the RFC mentions the same. So this should be all thats necessary in order for the target to be promoted.

This is a recreation of my previous PR because I accidentally did an incorrect git rebase which caused unnecessary changes to various commit SHAs. So this PR is a recreation of my previous PR without said stumble. My bad.
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 11, 2024
…imulacrum

Promote aarch64-apple-darwin to Tier 1

This promotes aarch64-apple-darwin to Tier 1 status as per rust-lang/rfcs#3671 and tracking issue rust-lang#73908. Not sure what else is necessary for this to impement the aforementioned RFC, however I figured I'd try. I did read in previous issues and PRs that the necessary infrastructure was already in place for the aarch64-apple-darwin target, and the RFC mentions the same. So this should be all thats necessary in order for the target to be promoted.

This is a recreation of my previous PR because I accidentally did an incorrect git rebase which caused unnecessary changes to various commit SHAs. So this PR is a recreation of my previous PR without said stumble. My bad.
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Aug 11, 2024
Rollup merge of rust-lang#128592 - evelynharthbrooke:master, r=Mark-Simulacrum

Promote aarch64-apple-darwin to Tier 1

This promotes aarch64-apple-darwin to Tier 1 status as per rust-lang/rfcs#3671 and tracking issue rust-lang#73908. Not sure what else is necessary for this to impement the aforementioned RFC, however I figured I'd try. I did read in previous issues and PRs that the necessary infrastructure was already in place for the aarch64-apple-darwin target, and the RFC mentions the same. So this should be all thats necessary in order for the target to be promoted.

This is a recreation of my previous PR because I accidentally did an incorrect git rebase which caused unnecessary changes to various commit SHAs. So this PR is a recreation of my previous PR without said stumble. My bad.
RalfJung pushed a commit to RalfJung/miri that referenced this pull request Aug 12, 2024
RalfJung pushed a commit to RalfJung/miri that referenced this pull request Aug 12, 2024
Promote aarch64-apple-darwin to Tier 1

This promotes aarch64-apple-darwin to Tier 1 status as per rust-lang/rfcs#3671 and tracking issue #73908. Not sure what else is necessary for this to impement the aforementioned RFC, however I figured I'd try. I did read in previous issues and PRs that the necessary infrastructure was already in place for the aarch64-apple-darwin target, and the RFC mentions the same. So this should be all thats necessary in order for the target to be promoted.

This is a recreation of my previous PR because I accidentally did an incorrect git rebase which caused unnecessary changes to various commit SHAs. So this PR is a recreation of my previous PR without said stumble. My bad.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
disposition-merge This RFC is in PFCP or FCP with a disposition to merge it. finished-final-comment-period The final comment period is finished for this RFC. T-compiler Relevant to the compiler team, which will review and decide on the RFC. T-infra Relevant to the infrastructure team, which will review and decide on the RFC. T-release Relevant to the release team, which will review and decide on the RFC. to-announce
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants