From 64c0f572e8ae2f5fc66337ed251d43ebc172fdc3 Mon Sep 17 00:00:00 2001 From: Wez Furlong Date: Tue, 20 Aug 2019 15:41:14 -0700 Subject: [PATCH] getdeps: fixup testpilot new test recording Summary: In the Facebook infrastructure the testpilot runner talks to a service to determine whether a given test should be run locally. The remote service was returning an error for some tests because they didn't have enough configuration specified. This diff associates some configuration with the tests based on the host type and makes things happier. Reviewed By: Ben0mega Differential Revision: D16894682 fbshipit-source-id: b6f59b112991160ef26cc126e98041d8908a6273 --- build/fbcode_builder/getdeps/builder.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/build/fbcode_builder/getdeps/builder.py b/build/fbcode_builder/getdeps/builder.py index e59683432..ac32b308a 100644 --- a/build/fbcode_builder/getdeps/builder.py +++ b/build/fbcode_builder/getdeps/builder.py @@ -465,6 +465,7 @@ if __name__ == "__main__": env.set("http_proxy", "") env.set("https_proxy", "") + machine_suffix = self.build_opts.host_type.as_tuple_string() runs = [] @@ -478,6 +479,9 @@ if __name__ == "__main__": self.build_opts.fbsource_dir, "--buck-test-info", buck_test_info_name, + "--test-config", + "platform=%s" % machine_suffix, + "buildsystem=getdeps", ] if schedule_type == "continuous":