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

Windows: Recent bump to rules_go breaks Windows + RBE #11657

Open
sunjayBhatia opened this issue Jun 19, 2020 · 8 comments
Open

Windows: Recent bump to rules_go breaks Windows + RBE #11657

sunjayBhatia opened this issue Jun 19, 2020 · 8 comments
Assignees
Labels
area/build area/windows bug no stalebot Disables stalebot from closing an issue

Comments

@sunjayBhatia
Copy link
Member

sunjayBhatia commented Jun 19, 2020

We have found that the recent bump to rules_go v0.23.3 breaks Windows building via RBE. This is not a specific issue with rules_go, but rather a bug in Bazel/RBE (to be further determined the exact cause). We are intending to patch rules_go with a small patch in the interim while the underlying issue gets sorted out.

The failure manifests itself in Envoy as:

ERROR: C:/_eb/external/go_sdk/BUILD.bazel:43:1: GoToolchainBinaryCompile external/go_sdk/builder.exe.a failed (Exit 1): builder.exe.bat failed: error executing command 
  cd C:/_eb/execroot/envoy
  SET GOROOT=external/go_sdk
  bazel-out/host/bin/external/go_sdk/builder.exe.bat
Execution platform: @rbe_windows_msvc_cl//config:platform
'bazel-out' is not recognized as an internal or external command,
operable program or batch file.
Target //source/exe:envoy-static failed to build

However, this is not an Envoy/rules_go specific issue, we have been able to reproduce this separately, see link below

Links

@sunjayBhatia
Copy link
Member Author

cc @wrowe @davinci26 @nigriMSFT

@sunjayBhatia
Copy link
Member Author

/assign @sunjayBhatia

/assign @wrowe

@sunjayBhatia
Copy link
Member Author

See bazelbuild/bazel#11636 for Bazel issue

lizan pushed a commit that referenced this issue Jun 27, 2020
Commit Message:
Enable RBE build in Windows CI

- Bumps envoy-build-tools and bazel-toolchains
- Bumps envoy-build-ubuntu image tag
  - Running Windows CI via docker uses the same mechanism as Linux to find the image tag, the tags are the same for Windows and Linux images
- Temporarily adds a patch to fix rules_go for Windows RBE issue executing batch scripts, see #11657
- Tags/fixes some additional tests that fail in CI

Additional Description: N/A
Risk Level: Low, but we may observe differences in performance of Windows CI running remotely, we may need to adjust the worker pool accordingly
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Fixes #10619

Signed-off-by: Sunjay Bhatia <[email protected]>
Co-authored-by: William A Rowe Jr <[email protected]>
@sunjayBhatia
Copy link
Member Author

looks like there are some volunteers to fix the issue in Bazel: bazelbuild/bazel#11636

@stale
Copy link

stale bot commented Aug 23, 2020

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or other activity occurs. Thank you for your contributions.

@stale stale bot added the stale stalebot believes this issue/PR has not been touched recently label Aug 23, 2020
@wrowe wrowe added no stalebot Disables stalebot from closing an issue and removed stale stalebot believes this issue/PR has not been touched recently labels Aug 24, 2020
@wrowe
Copy link
Contributor

wrowe commented May 7, 2021

@sunjayBhatia Would you go ahead and check status on this?

@sunjayBhatia
Copy link
Member Author

Doesn't look like this is solved yet in Bazel upstream, also given a new issue opened on the topic, referencing our Bazel issue: bazelbuild/remote-apis#187

@sunjayBhatia
Copy link
Member Author

I'm going to remove myself from this issue as I'm not actively working on it

@sunjayBhatia sunjayBhatia removed their assignment Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build area/windows bug no stalebot Disables stalebot from closing an issue
Projects
None yet
Development

No branches or pull requests

3 participants