From 50dd2de6fe2f98b899e6df6539733f56b4df2e74 Mon Sep 17 00:00:00 2001 From: Uche Uba Date: Wed, 23 Oct 2024 22:51:20 -0700 Subject: [PATCH] Updating the force_tests_on_mac_re mode file to only contain the RE use case informtion Summary: With the goal of having less mode files in arvr. The mode file to force mac tests to run on RE will eventually be removed as this diff sets up the toolchain to ensure that tests can run on RE if there is a platform mismatch D62482846. Currently the use case is being used to track the increase in mac capacity while jobs are being migrated over from mac-arm to linux. Once the migration has been completed the mode file will be deleted, and the use case RE capacity for `rl-mac-default` will be merged with that of `re_tests` Reviewed By: 8Keep Differential Revision: D64796922 fbshipit-source-id: c52bd3176edd3fe70f526afa369daab9df786615 --- tests/re_utils.bzl | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/tests/re_utils.bzl b/tests/re_utils.bzl index d5ef36611..b9f2134a1 100644 --- a/tests/re_utils.bzl +++ b/tests/re_utils.bzl @@ -57,18 +57,7 @@ def get_re_executors_from_props(ctx: AnalysisContext) -> ([CommandExecutorConfig re_props = _get_re_arg(ctx).re_props if re_props == None: - # If no RE args are set and an RE config is specified - if bool(read_config("tpx", "force_re_props")): - re_props = { - "capabilities": { - "platform": read_config("remoteexecution", "platform"), - "subplatform": read_config("remoteexecution", "subplatform"), - }, - "use_case": read_config("remoteexecution", "use_case"), - } - - else: - return None, {} + return None, {} re_props_copy = dict(re_props) capabilities = re_props_copy.pop("capabilities")