Skip to content

Commit

Permalink
Updating the force_tests_on_mac_re mode file to only contain the RE u…
Browse files Browse the repository at this point in the history
…se 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
  • Loading branch information
Uche Uba authored and facebook-github-bot committed Oct 24, 2024
1 parent 0ff2141 commit 50dd2de
Showing 1 changed file with 1 addition and 12 deletions.
13 changes: 1 addition & 12 deletions tests/re_utils.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand Down

0 comments on commit 50dd2de

Please sign in to comment.