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

Please regenerate RBE toolchain configs #22758

Closed
Wyverald opened this issue Aug 18, 2022 · 15 comments · Fixed by #22862 or #23140
Closed

Please regenerate RBE toolchain configs #22758

Wyverald opened this issue Aug 18, 2022 · 15 comments · Fixed by #22862 or #23140
Assignees

Comments

@Wyverald
Copy link
Contributor

Problem: Envoy RBE is failing on Bazel@HEAD+Downstream pipeline due to bazelbuild/bazel@3469784 (example failure)

What needs to be done: envoy-build-tools needs to be updated to use the latest rbe_config_gen from bazel-toolchains (at least version 5.1.2 so that it contains bazelbuild/bazel-toolchains@8c717f8). Then the toolchain configs need to be regenerated, and this file in Envoy needs to be updated so that Envoy uses the latest envoy-build-tools.

@Wyverald Wyverald added the triage Issue requires triage label Aug 18, 2022
@phlax
Copy link
Member

phlax commented Aug 18, 2022

@Wyverald can you raise a PR in envoy-build-tools to update

@Wyverald
Copy link
Contributor Author

sorry, I don't have the time. If you also can't find the cycles to do this, we'll have to disable Envoy in the downstream pipeline.

@mattklein123
Copy link
Member

cc @lizan @keith

keith added a commit to envoyproxy/envoy-build-tools that referenced this issue Aug 19, 2022
envoyproxy/envoy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
keith added a commit to envoyproxy/envoy-build-tools that referenced this issue Aug 19, 2022
envoyproxy/envoy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith
Copy link
Member

keith commented Aug 19, 2022

working on this here envoyproxy/envoy-build-tools#181

@Wyverald
Copy link
Contributor Author

gentle ping. Envoy remains broken in the downstream pipeline, and if we can't resolve that soon, we'd need to disable Envoy on Bazel CI.

@keith
Copy link
Member

keith commented Aug 23, 2022

I submitted #22822 to unblock bazel CI

@keith
Copy link
Member

keith commented Aug 24, 2022

@Wyverald can you confirm that fixes the next run?

@Wyverald
Copy link
Contributor Author

The next run that includes that fix will be tonight CET, so I'll report back tomorrow. Thanks!

@Wyverald
Copy link
Contributor Author

For some reason, Envoy is still broken: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/2601#0182d854-183d-4bcd-b15e-f60f99173f16

@aranguyen do you know why that may be? On line 279 of the logs, it says Inherited 'build' options: [...] --incompatible_use_platforms_repo_for_constraints=false, yet later the build still fails on @bazel_tools//platforms:X usages (line 292).

@aranguyen
Copy link

aranguyen commented Aug 26, 2022

oh hmm..interesting..it looks like the flag did not get overriden somehow. The flag is in the workspace bazelrc file which should take precedence, unless there is something special about incompatible flag that prevent it from being overriden once flipped (I doubt it) that I do not know about. Let me take a look at that and file a bug with configurability team.

In the meantime, let me see if I can update the following to unblock the CI failures for envoy.

aranguyen-macbookpro:envoy aranguyen$ grep -r "@bazel_tools//platforms" /private/var/tmp/_bazel_aranguyen/d62ce45fc2cda7024a6d68c3c63b8002
/private/var/tmp/_bazel_aranguyen/d62ce45fc2cda7024a6d68c3c63b8002/external/io_bazel_rules_go/go/toolchain/toolchains.bzl:    @bazel_tools//platforms:default_platform will be used most of the time).
/private/var/tmp/_bazel_aranguyen/d62ce45fc2cda7024a6d68c3c63b8002/external/com_google_protobuf/third_party/googletest/BUILD.bazel:    constraint_values = ["@bazel_tools//platforms:windows"],
/private/var/tmp/_bazel_aranguyen/d62ce45fc2cda7024a6d68c3c63b8002/external/com_google_googletest/BUILD.bazel:    constraint_values = ["@bazel_tools//platforms:windows"],

keith added a commit to envoyproxy/envoy-build-tools that referenced this issue Aug 26, 2022
envoyproxy/envoy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith
Copy link
Member

keith commented Aug 26, 2022

gtest update is here: #22779, blocked on unrelated issues
bazel toolchains PR is here: envoyproxy/envoy-build-tools#181 was blocked on a windows issue that should be fixed now, so maybe that one will be mergeable today

@keith
Copy link
Member

keith commented Aug 26, 2022

#22862

@mattklein123 mattklein123 added area/build and removed triage Issue requires triage labels Aug 29, 2022
keith added a commit to keith/envoy that referenced this issue Sep 2, 2022
Fixes envoyproxy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
keith added a commit that referenced this issue Sep 2, 2022
Fixes #22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith keith reopened this Sep 2, 2022
keith added a commit to keith/envoy that referenced this issue Sep 2, 2022
This doesn't update to the latest version because of other new
incompatibilities, envoyproxy#22779

This should fix the last issue with envoyproxy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith
Copy link
Member

keith commented Sep 2, 2022

still needs #22978 or #22779

@meteorcloudy
Copy link
Contributor

image

https://buildkite.com/bazel/bazelisk-plus-incompatible-flags/builds/1256#_

Can we get the gtest fixes merged? Envoy is one of the few projects that're blocking the flip of bazelbuild/bazel#8622

keith added a commit to keith/envoy that referenced this issue Sep 15, 2022
This has landed upstream, but updating googletest proved harder than
expected. This fixes envoyproxy#22758 in
the meantime.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith
Copy link
Member

keith commented Sep 15, 2022

I've submitted #23140 to just try to patch it, since updating googletest has some breaking changes it seems like. @meteorcloudy as stated above we are disabling this feature

envoy/.bazelrc

Lines 17 to 18 in aa1c71d

# TODO: https://github.com/envoyproxy/envoy/issues/22758
build --incompatible_use_platforms_repo_for_constraints=false
which clearly isn't taking, which sounds a bit risky for actually shipping a bazel update where folks can't disable it

keith added a commit that referenced this issue Sep 16, 2022
This has landed upstream, but updating googletest proved harder than
expected. This fixes #22758 in
the meantime.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
oschaaf pushed a commit to maistra/envoy that referenced this issue Oct 26, 2022
Fixes envoyproxy/envoy#22758

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
oschaaf pushed a commit to maistra/envoy that referenced this issue Oct 26, 2022
This has landed upstream, but updating googletest proved harder than
expected. This fixes envoyproxy/envoy#22758 in
the meantime.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants