From 9759a1b241c11203f6630634d8cb3e38aefd8aaa Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Wed, 7 Nov 2018 15:21:04 -0800 Subject: removed accidental submodule update --- third_party/toolchains/BUILD | 37 ++++++++++++++++++----- third_party/toolchains/RBE_USE_MACHINE_TYPE_LARGE | 1 - third_party/toolchains/machine_size/BUILD | 31 +++++++++++++++++++ 3 files changed, 61 insertions(+), 8 deletions(-) delete mode 100644 third_party/toolchains/RBE_USE_MACHINE_TYPE_LARGE create mode 100644 third_party/toolchains/machine_size/BUILD (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 02cd87a7b9..efb52736fe 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -16,36 +16,59 @@ licenses(["notice"]) # Apache v2 package(default_visibility = ["//visibility:public"]) -exports_files(["RBE_USE_MACHINE_TYPE_LARGE",]) - # Latest RBE Ubuntu16_04 container # Update every time when a new container is released. alias( name = "rbe_ubuntu1604", - actual = ":rbe_ubuntu1604_r328903", + actual = ":rbe_ubuntu1604_r340178", ) -# RBE Ubuntu16_04 r328903 +alias( + name = "rbe_ubuntu1604_large", + actual = ":rbe_ubuntu1604_r340178_large", +) + +# RBE Ubuntu16_04 r340178 platform( - name = "rbe_ubuntu1604_r328903", + name = "rbe_ubuntu1604_r340178", constraint_values = [ "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "//third_party/toolchains/machine_size:standard", ], remote_execution_properties = """ properties: { name: "container-image" - value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:59bf0e191a6b5cc1ab62c2224c810681d1326bad5a27b1d36c9f40113e79da7f" + value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:9bd8ba020af33edb5f11eff0af2f63b3bcb168cd6566d7b27c6685e717787928" } properties: { name: "gceMachineType" # Small machines for majority of tests. value: "n1-highmem-2" } + """, +) + +# RBE Ubuntu16_04 r340178 large +platform( + name = "rbe_ubuntu1604_r340178_large", + constraint_values = [ + "@bazel_tools//platforms:x86_64", + "@bazel_tools//platforms:linux", + "@bazel_tools//tools/cpp:clang", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "//third_party/toolchains/machine_size:large", + ], + remote_execution_properties = """ + properties: { + name: "container-image" + value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:9bd8ba020af33edb5f11eff0af2f63b3bcb168cd6566d7b27c6685e717787928" + } properties: { - name: "gceMachineType_LARGE" # Large machines for a small set of resource-consuming tests such as combiner_tests under TSAN. + name: "gceMachineType" # Small machines for majority of tests. value: "n1-standard-8" } """, diff --git a/third_party/toolchains/RBE_USE_MACHINE_TYPE_LARGE b/third_party/toolchains/RBE_USE_MACHINE_TYPE_LARGE deleted file mode 100644 index b1120238d7..0000000000 --- a/third_party/toolchains/RBE_USE_MACHINE_TYPE_LARGE +++ /dev/null @@ -1 +0,0 @@ -# This file is a sentinel and is meant to be empty. diff --git a/third_party/toolchains/machine_size/BUILD b/third_party/toolchains/machine_size/BUILD new file mode 100644 index 0000000000..cc962946c3 --- /dev/null +++ b/third_party/toolchains/machine_size/BUILD @@ -0,0 +1,31 @@ +# Copyright 2018 gRPC authors. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +licenses(["notice"]) # Apache v2 + +package(default_visibility = ["//visibility:public"]) + +constraint_setting(name = "machine_size") + +constraint_value( + name = "large", + constraint_setting = ":machine_size", +) + +constraint_value( + name = "standard", + constraint_setting = ":machine_size", +) + +# Add other constraint values as needed (tiny, huge, etc.) in the future. -- cgit v1.2.3 From d069bb2444a38021caf3ab4eee489fa2310fe7a8 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Wed, 14 Nov 2018 16:25:31 -0800 Subject: more destination fixes --- third_party/toolchains/BUILD | 8 ++++---- tools/remote_build/rbe_common.bazelrc | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index efb52736fe..f5767f212d 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -35,8 +35,8 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "@bazel_toolchains//constraints:xenial", + "@bazel_toolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:standard", ], remote_execution_properties = """ @@ -58,8 +58,8 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "@bazel_toolchains//constraints:xenial", + "@@bazel_toolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:large", ], remote_execution_properties = """ diff --git a/tools/remote_build/rbe_common.bazelrc b/tools/remote_build/rbe_common.bazelrc index 583781effc..f17ed10f57 100644 --- a/tools/remote_build/rbe_common.bazelrc +++ b/tools/remote_build/rbe_common.bazelrc @@ -18,8 +18,8 @@ startup --host_jvm_args=-Dbazel.DigestFunction=SHA256 -build --crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain -build --extra_toolchains=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/cpp:cc-toolchain-clang-x86_64-default +build --crosstool_top=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain +build --extra_toolchains=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/cpp:cc-toolchain-clang-x86_64-default # Use custom execution platforms defined in third_party/toolchains build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large build --host_platform=//third_party/toolchains:rbe_ubuntu1604 @@ -59,9 +59,9 @@ build:msan --cxxopt=--stdlib=libc++ # setting LD_LIBRARY_PATH is necessary # to avoid "libc++.so.1: cannot open shared object file" build:msan --action_env=LD_LIBRARY_PATH=/usr/local/lib -build:msan --host_crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain +build:msan --host_crosstool_top=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain # override the config-agnostic crosstool_top -build:msan --crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/msan:toolchain +build:msan --crosstool_top=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/msan:toolchain # thread sanitizer: most settings are already in %workspace%/.bazelrc # we only need a few additional ones that are Foundry specific @@ -76,7 +76,7 @@ build:ubsan --copt=-gmlt # TODO(jtattermusch): use more reasonable test timeout build:ubsan --test_timeout=3600 # override the config-agnostic crosstool_top ---crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/experimental/ubuntu16_04_clang/1.1/bazel_0.16.1/ubsan:toolchain +--crosstool_top=@bazel_toolchains//configs/experimental/ubuntu16_04_clang/1.1/bazel_0.16.1/ubsan:toolchain # TODO(jtattermusch): remove this once Foundry adds the env to the docker image. # ubsan needs symbolizer to work properly, otherwise the suppression file doesn't work # and we get test failures. -- cgit v1.2.3 From caf56447d5b67092b99ea74dbe5a7ec9cc7c6b1d Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 10:52:41 -0800 Subject: added custom platform --- third_party/toolchains/BUILD | 20 ++++++++++++++++++++ tools/remote_build/rbe_common.bazelrc | 2 +- 2 files changed, 21 insertions(+), 1 deletion(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index f5767f212d..b9a8cfd46a 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -73,3 +73,23 @@ platform( } """, ) + +# This target is auto-generated from release/cpp.tpl and should not be +# modified directly. +toolchain( + name = "cc-toolchain-clang-x86_64-default", + exec_compatible_with = [ + "@bazel_tools//platforms:linux", + "@bazel_tools//platforms:x86_64", + "@bazel_tools//tools/cpp:clang", + "//constraints:xenial", + ], + target_compatible_with = [ + "//third_party/toolchains:rbe_ubuntu1604", + "//third_party/toolchains:rbe_ubuntu1604_large", + "@bazel_tools//platforms:linux", + "@bazel_tools//platforms:x86_64", + ], + toolchain = "//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", + toolchain_type = "@bazel_tools//tools/cpp:toolchain_type", +) diff --git a/tools/remote_build/rbe_common.bazelrc b/tools/remote_build/rbe_common.bazelrc index f17ed10f57..82b59355cb 100644 --- a/tools/remote_build/rbe_common.bazelrc +++ b/tools/remote_build/rbe_common.bazelrc @@ -19,7 +19,7 @@ startup --host_jvm_args=-Dbazel.DigestFunction=SHA256 build --crosstool_top=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain -build --extra_toolchains=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/cpp:cc-toolchain-clang-x86_64-default +build --extra_toolchains=//third_party/toolchains:cc-toolchain-clang-x86_64-default # Use custom execution platforms defined in third_party/toolchains build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large build --host_platform=//third_party/toolchains:rbe_ubuntu1604 -- cgit v1.2.3 From 4d84165bb00b18bad657a6f8b09a97a655a08a29 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 13:56:30 -0800 Subject: added machine size to toolchain constraints --- third_party/toolchains/BUILD | 2 ++ 1 file changed, 2 insertions(+) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index b9a8cfd46a..2699cf391d 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -83,6 +83,8 @@ toolchain( "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", "//constraints:xenial", + "//third_party/toolchains/machine_size:standard", + "//third_party/toolchains/machine_size:large", ], target_compatible_with = [ "//third_party/toolchains:rbe_ubuntu1604", -- cgit v1.2.3 From 17ce7e0d94d147040c86cd3d92b03d46f591f681 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 14:26:30 -0800 Subject: addressed Nick's comments --- third_party/toolchains/BUILD | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 2699cf391d..27a62b7495 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -59,7 +59,7 @@ platform( "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", "@bazel_toolchains//constraints:xenial", - "@@bazel_toolchains//constraints/sanitizers:support_msan", + "@bazel_toolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:large", ], remote_execution_properties = """ @@ -82,7 +82,8 @@ toolchain( "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", - "//constraints:xenial", + "@bazel_toolchains//constraints:xenial", + "@bazel_toolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:standard", "//third_party/toolchains/machine_size:large", ], -- cgit v1.2.3 From 8575588fde9cfb7c8ab1bcd550e13b2f635a9292 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 14:32:35 -0800 Subject: added path prefix to bazel_toolchains repo --- third_party/toolchains/BUILD | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 27a62b7495..e7ef03f5db 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -93,6 +93,6 @@ toolchain( "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", ], - toolchain = "//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", + toolchain = "@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", toolchain_type = "@bazel_tools//tools/cpp:toolchain_type", ) -- cgit v1.2.3 From 3f70672827018b97e7ce13be826d91e8455ce19f Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 14:39:29 -0800 Subject: reverted bazel-toolchains alias to its original form --- third_party/toolchains/BUILD | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index e7ef03f5db..4b075eb7dc 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -35,8 +35,8 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - "@bazel_toolchains//constraints:xenial", - "@bazel_toolchains//constraints/sanitizers:support_msan", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:standard", ], remote_execution_properties = """ @@ -58,8 +58,8 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - "@bazel_toolchains//constraints:xenial", - "@bazel_toolchains//constraints/sanitizers:support_msan", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:large", ], remote_execution_properties = """ @@ -82,8 +82,8 @@ toolchain( "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", - "@bazel_toolchains//constraints:xenial", - "@bazel_toolchains//constraints/sanitizers:support_msan", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", "//third_party/toolchains/machine_size:standard", "//third_party/toolchains/machine_size:large", ], @@ -93,6 +93,6 @@ toolchain( "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", ], - toolchain = "@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", + toolchain = "@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", toolchain_type = "@bazel_tools//tools/cpp:toolchain_type", ) -- cgit v1.2.3 From 09b0b0b71207e716b990ed0fb8d1cfae435fbc43 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 14:50:05 -0800 Subject: Attempting Eric Burnett's fix --- third_party/toolchains/BUILD | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 4b075eb7dc..8b16b669df 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -88,8 +88,8 @@ toolchain( "//third_party/toolchains/machine_size:large", ], target_compatible_with = [ - "//third_party/toolchains:rbe_ubuntu1604", - "//third_party/toolchains:rbe_ubuntu1604_large", + # "//third_party/toolchains:rbe_ubuntu1604", + # "//third_party/toolchains:rbe_ubuntu1604_large", "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", ], -- cgit v1.2.3 From 6cbbefc6c6f03bf6d48816f36f2d496768c39a12 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Thu, 15 Nov 2018 16:47:51 -0800 Subject: temporarily removed some constraints --- third_party/toolchains/BUILD | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 8b16b669df..bf32ae4786 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -35,9 +35,9 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", - "//third_party/toolchains/machine_size:standard", + # "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + # "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + # "//third_party/toolchains/machine_size:standard", ], remote_execution_properties = """ properties: { @@ -82,10 +82,10 @@ toolchain( "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", - "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", - "//third_party/toolchains/machine_size:standard", - "//third_party/toolchains/machine_size:large", + # "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + # "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + # "//third_party/toolchains/machine_size:standard", + # "//third_party/toolchains/machine_size:large", ], target_compatible_with = [ # "//third_party/toolchains:rbe_ubuntu1604", -- cgit v1.2.3 From 8fad166c854d9685233457c1848661bc48e4986f Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Mon, 19 Nov 2018 16:11:57 -0800 Subject: added autoconfigured --- third_party/toolchains/BUILD | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index bf32ae4786..bbaecae565 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -35,9 +35,9 @@ platform( "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", "@bazel_tools//tools/cpp:clang", - # "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - # "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", - # "//third_party/toolchains/machine_size:standard", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "//third_party/toolchains/machine_size:standard", ], remote_execution_properties = """ properties: { @@ -79,17 +79,18 @@ platform( toolchain( name = "cc-toolchain-clang-x86_64-default", exec_compatible_with = [ + "@bazel_tools//platforms:autoconfigured", "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", - # "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - # "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", - # "//third_party/toolchains/machine_size:standard", - # "//third_party/toolchains/machine_size:large", + "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", + "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", + "//third_party/toolchains/machine_size:standard", + "//third_party/toolchains/machine_size:large", ], target_compatible_with = [ - # "//third_party/toolchains:rbe_ubuntu1604", - # "//third_party/toolchains:rbe_ubuntu1604_large", + "//third_party/toolchains:rbe_ubuntu1604", + "//third_party/toolchains:rbe_ubuntu1604_large", "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", ], -- cgit v1.2.3 From 4a9feff1fd9a0e789f10ea7b556a7af8f878e70d Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Tue, 20 Nov 2018 16:15:42 -0800 Subject: removed autoconfigured constraint --- third_party/toolchains/BUILD | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index bbaecae565..2a68f3e7cc 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -71,7 +71,7 @@ platform( name: "gceMachineType" # Small machines for majority of tests. value: "n1-standard-8" } - """, + """, ) # This target is auto-generated from release/cpp.tpl and should not be @@ -79,7 +79,6 @@ platform( toolchain( name = "cc-toolchain-clang-x86_64-default", exec_compatible_with = [ - "@bazel_tools//platforms:autoconfigured", "@bazel_tools//platforms:linux", "@bazel_tools//platforms:x86_64", "@bazel_tools//tools/cpp:clang", -- cgit v1.2.3 From 3391245e739f587ded080aa62b104af1d66f0b79 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Tue, 20 Nov 2018 16:18:17 -0800 Subject: removed all constraints from custom toolchain --- third_party/toolchains/BUILD | 11 ----------- 1 file changed, 11 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 2a68f3e7cc..7506c6a1fb 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -79,19 +79,8 @@ platform( toolchain( name = "cc-toolchain-clang-x86_64-default", exec_compatible_with = [ - "@bazel_tools//platforms:linux", - "@bazel_tools//platforms:x86_64", - "@bazel_tools//tools/cpp:clang", - "@com_github_bazelbuild_bazeltoolchains//constraints:xenial", - "@com_github_bazelbuild_bazeltoolchains//constraints/sanitizers:support_msan", - "//third_party/toolchains/machine_size:standard", - "//third_party/toolchains/machine_size:large", ], target_compatible_with = [ - "//third_party/toolchains:rbe_ubuntu1604", - "//third_party/toolchains:rbe_ubuntu1604_large", - "@bazel_tools//platforms:linux", - "@bazel_tools//platforms:x86_64", ], toolchain = "@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:cc-compiler-k8", toolchain_type = "@bazel_tools//tools/cpp:toolchain_type", -- cgit v1.2.3 From 70f9992d77e67f90234b079eabaa33210d866c87 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Mon, 26 Nov 2018 10:10:53 -0800 Subject: modified execution platform registration, added host platform --- WORKSPACE | 4 ---- third_party/toolchains/BUILD | 22 ++++++++++++++++++++++ tools/remote_build/rbe_common.bazelrc | 3 ++- 3 files changed, 24 insertions(+), 5 deletions(-) (limited to 'third_party') diff --git a/WORKSPACE b/WORKSPACE index cd2718204d..1340ba0c6b 100644 --- a/WORKSPACE +++ b/WORKSPACE @@ -6,10 +6,6 @@ grpc_deps() grpc_test_only_deps() -register_execution_platforms( - "//third_party/toolchains:all", -) - register_toolchains( "//third_party/toolchains:all", ) diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 7506c6a1fb..39699f3112 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -74,6 +74,28 @@ platform( """, ) +platform( + name = "host_platform-large", + constraint_values = [ + "//third_party/toolchains/machine_size:large", + ], + cpu_constraints = [ + "@bazel_tools//platforms:x86_32", + "@bazel_tools//platforms:x86_64", + "@bazel_tools//platforms:ppc", + "@bazel_tools//platforms:arm", + "@bazel_tools//platforms:aarch64", + "@bazel_tools//platforms:s390x", + ], + host_platform = True, + os_constraints = [ + "@bazel_tools//platforms:osx", + "@bazel_tools//platforms:freebsd", + "@bazel_tools//platforms:linux", + "@bazel_tools//platforms:windows", + ], +) + # This target is auto-generated from release/cpp.tpl and should not be # modified directly. toolchain( diff --git a/tools/remote_build/rbe_common.bazelrc b/tools/remote_build/rbe_common.bazelrc index 77ce96b8d4..c1ee428082 100644 --- a/tools/remote_build/rbe_common.bazelrc +++ b/tools/remote_build/rbe_common.bazelrc @@ -21,7 +21,7 @@ startup --host_jvm_args=-Dbazel.DigestFunction=SHA256 build --crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain build --extra_toolchains=//third_party/toolchains:cc-toolchain-clang-x86_64-default # Use custom execution platforms defined in third_party/toolchains -build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large +build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large,//third_party/toolchains:host_platform-large build --host_platform=//third_party/toolchains:rbe_ubuntu1604 build --platforms=//third_party/toolchains:rbe_ubuntu1604 @@ -69,6 +69,7 @@ build:tsan --copt=-gmlt # TODO(jtattermusch): use more reasonable test timeout build:tsan --test_timeout=3600 build:tsan --test_tag_filters=-qps_json_driver,-json_run_localhost +build:tsan --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large # undefined behavior sanitizer: most settings are already in %workspace%/.bazelrc # we only need a few additional ones that are Foundry specific -- cgit v1.2.3 From 49528b9713d0e60cdb7806cb3c10e2e5b9a92f03 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Tue, 27 Nov 2018 14:13:28 -0800 Subject: trying highmem machine --- third_party/toolchains/BUILD | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 39699f3112..9481d3b137 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -69,7 +69,7 @@ platform( } properties: { name: "gceMachineType" # Small machines for majority of tests. - value: "n1-standard-8" + value: "n1-highmem-8" } """, ) -- cgit v1.2.3 From eff3b4ec306e4b7cdedab5b42fbbde8f800208aa Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Tue, 27 Nov 2018 15:13:33 -0800 Subject: trying standard-16 machine --- third_party/toolchains/BUILD | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 9481d3b137..7df0074b40 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -68,8 +68,8 @@ platform( value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:9bd8ba020af33edb5f11eff0af2f63b3bcb168cd6566d7b27c6685e717787928" } properties: { - name: "gceMachineType" # Small machines for majority of tests. - value: "n1-highmem-8" + name: "gceMachineType" # Large machines for some resource demanding tests (TSAN). + value: "n1-standard-16" } """, ) -- cgit v1.2.3 From e00b58ba5aa2e6683b156570eb785e679a628d3b Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Tue, 27 Nov 2018 15:25:30 -0800 Subject: trying latest image --- third_party/toolchains/BUILD | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 7df0074b40..2736543fcf 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -20,17 +20,17 @@ package(default_visibility = ["//visibility:public"]) # Update every time when a new container is released. alias( name = "rbe_ubuntu1604", - actual = ":rbe_ubuntu1604_r340178", + actual = ":rbe_ubuntu1604_r342117", ) alias( name = "rbe_ubuntu1604_large", - actual = ":rbe_ubuntu1604_r340178_large", + actual = ":rbe_ubuntu1604_r342117_large", ) -# RBE Ubuntu16_04 r340178 +# RBE Ubuntu16_04 r342117 platform( - name = "rbe_ubuntu1604_r340178", + name = "rbe_ubuntu1604_r342117", constraint_values = [ "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", @@ -42,7 +42,7 @@ platform( remote_execution_properties = """ properties: { name: "container-image" - value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:9bd8ba020af33edb5f11eff0af2f63b3bcb168cd6566d7b27c6685e717787928" + value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:f3120a030a19d67626ababdac79cc787e699a1aa924081431285118f87e7b375" } properties: { name: "gceMachineType" # Small machines for majority of tests. @@ -51,9 +51,9 @@ platform( """, ) -# RBE Ubuntu16_04 r340178 large +# RBE Ubuntu16_04 r342117 large platform( - name = "rbe_ubuntu1604_r340178_large", + name = "rbe_ubuntu1604_r342117_large", constraint_values = [ "@bazel_tools//platforms:x86_64", "@bazel_tools//platforms:linux", @@ -65,11 +65,11 @@ platform( remote_execution_properties = """ properties: { name: "container-image" - value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:9bd8ba020af33edb5f11eff0af2f63b3bcb168cd6566d7b27c6685e717787928" + value:"docker://gcr.io/cloud-marketplace/google/rbe-ubuntu16-04@sha256:f3120a030a19d67626ababdac79cc787e699a1aa924081431285118f87e7b375" } properties: { name: "gceMachineType" # Large machines for some resource demanding tests (TSAN). - value: "n1-standard-16" + value: "n1-standard-8" } """, ) -- cgit v1.2.3 From 527ddd99821cd0e0970fdd5b5b4988394f0ea078 Mon Sep 17 00:00:00 2001 From: Bill Feng Date: Wed, 28 Nov 2018 11:10:05 -0800 Subject: cleanup --- third_party/toolchains/BUILD | 22 ---------------------- tools/remote_build/rbe_common.bazelrc | 3 +-- 2 files changed, 1 insertion(+), 24 deletions(-) (limited to 'third_party') diff --git a/third_party/toolchains/BUILD b/third_party/toolchains/BUILD index 2736543fcf..e213461acc 100644 --- a/third_party/toolchains/BUILD +++ b/third_party/toolchains/BUILD @@ -74,28 +74,6 @@ platform( """, ) -platform( - name = "host_platform-large", - constraint_values = [ - "//third_party/toolchains/machine_size:large", - ], - cpu_constraints = [ - "@bazel_tools//platforms:x86_32", - "@bazel_tools//platforms:x86_64", - "@bazel_tools//platforms:ppc", - "@bazel_tools//platforms:arm", - "@bazel_tools//platforms:aarch64", - "@bazel_tools//platforms:s390x", - ], - host_platform = True, - os_constraints = [ - "@bazel_tools//platforms:osx", - "@bazel_tools//platforms:freebsd", - "@bazel_tools//platforms:linux", - "@bazel_tools//platforms:windows", - ], -) - # This target is auto-generated from release/cpp.tpl and should not be # modified directly. toolchain( diff --git a/tools/remote_build/rbe_common.bazelrc b/tools/remote_build/rbe_common.bazelrc index df72f129ab..aa3ddb050c 100644 --- a/tools/remote_build/rbe_common.bazelrc +++ b/tools/remote_build/rbe_common.bazelrc @@ -21,7 +21,7 @@ startup --host_jvm_args=-Dbazel.DigestFunction=SHA256 build --crosstool_top=@com_github_bazelbuild_bazeltoolchains//configs/ubuntu16_04_clang/1.1/bazel_0.16.1/default:toolchain build --extra_toolchains=//third_party/toolchains:cc-toolchain-clang-x86_64-default # Use custom execution platforms defined in third_party/toolchains -build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large,//third_party/toolchains:host_platform-large +build --extra_execution_platforms=//third_party/toolchains:rbe_ubuntu1604,//third_party/toolchains:rbe_ubuntu1604_large build --host_platform=//third_party/toolchains:rbe_ubuntu1604 build --platforms=//third_party/toolchains:rbe_ubuntu1604 @@ -37,7 +37,6 @@ build --verbose_failures=true build --experimental_strict_action_env=true build --action_env=BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=1 -build --toolchain_resolution_debug # don't use port server build --define GRPC_PORT_ISOLATED_RUNTIME=1 -- cgit v1.2.3