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

x/build/env/windows/arm64: add Windows arm64 builders provided by Microsoft #53541

Closed
cagedmantis opened this issue Jun 24, 2022 · 6 comments
Closed
Assignees
Labels
arch-arm64 Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. new-builder OS-Windows
Milestone

Comments

@cagedmantis
Copy link
Contributor

Based on the comment left in #53383, we should work to setup reliable hardware and builders for the windows/arm64 port.

Microsoft dev here. We would like to help promoting windows/arm64 to be a first-class port. If it is just a problem of having reliable hardware, we can discuss providing reliable windows/arm64 hosts running in our own infrastructure. Would that be enough? If not, what else would be missing?

Originally posted by @qmuntal in #53383 (comment)

@qmuntal How would you like this discussion to proceed (GitHub discussion/email thread)?
@golang/release

@cagedmantis cagedmantis added Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jun 24, 2022
@cagedmantis cagedmantis added this to the Backlog milestone Jun 24, 2022
@qmuntal
Copy link
Contributor

qmuntal commented Jun 24, 2022

Let's discuss the internals via email thread and report out when we have something solid. Please send it to golangdev@microsoft.com.

@qmuntal
Copy link
Contributor

qmuntal commented Jul 12, 2022

Quick update: We are still trying to figure out the logistics on our side. Getting a Windows ARM64 machine available 24/7 is not straightforward at this point, but we will get there.

@toothrot toothrot added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jul 13, 2022
@dmitshur dmitshur removed the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Aug 10, 2022
@heschi heschi self-assigned this Aug 23, 2022
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/460296 mentions this issue: internal/releasetargets: remove windows-arm64 for now

gopherbot pushed a commit to golang/build that referenced this issue Jan 3, 2023
We haven't gotten the new builder up yet.

Updates golang/go#57408, golang/go#53541

Change-Id: I4f8b1d96d3f7f7a3080532f5e69b2fa0cf6992e0
Reviewed-on: https://go-review.googlesource.com/c/build/+/460296
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Heschi Kreinick <heschi@google.com>
Run-TryBot: Heschi Kreinick <heschi@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/461255 mentions this issue: internal/releasetargts: add windows-arm64 again

gopherbot pushed a commit to golang/build that referenced this issue Jan 9, 2023
This adds the windows-arm64-11 builder as a release target which was
removed in CL 460296.

Updates golang/go#57408
Updates golang/go#53541

Change-Id: I449859081350a36b05b85a7b87b412d43b19cb27
Reviewed-on: https://go-review.googlesource.com/c/build/+/461255
Run-TryBot: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/462655 mentions this issue: env/windows-arm64/aws: delete

gopherbot pushed a commit to golang/build that referenced this issue Jan 18, 2023
The Windows ARM64 builders on Azure have replaced the Windows ARM64
builders on AWS by now. Remove unused environment configuration; we
can bring it back or look it up in history if needed.

Updates golang/go#53541.
Updates golang/go#42604.
Updates golang/go#57408.

Change-Id: I080e4f5c53712c0470332a7d26d00c939136e57c
Reviewed-on: https://go-review.googlesource.com/c/build/+/462655
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Than McIntosh <thanm@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
@thanm
Copy link
Contributor

thanm commented Jan 31, 2023

Doesn't look like anything left to do on this issue, closing. Please reopen if you see fit, thanks.

@thanm thanm closed this as completed Jan 31, 2023
@dmitshur dmitshur modified the milestones: Backlog, Unreleased Jan 31, 2023
@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. new-builder and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jan 31, 2023
@golang golang locked and limited conversation to collaborators Jan 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-arm64 Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. new-builder OS-Windows
Projects
Archived in project
Development

No branches or pull requests

8 participants