Skip to content

Commit 6e54512

Browse files
SBonesmeta-codesync[bot]
authored andcommitted
Remove get_server_build_info_legacy (migration complete)
Summary: Remove the legacy get_server_build_info_legacy function now that all callers have been migrated to the modern thrift-python implementation. Reviewed By: muirdm Differential Revision: D90630677 fbshipit-source-id: b398c2ace6b2eba929d7fa26871e124858307cee
1 parent 6b4102b commit 6e54512

3 files changed

Lines changed: 2 additions & 9 deletions

File tree

eden/fs/cli/config.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -490,7 +490,7 @@ def get_running_version_parts(self) -> Tuple[str, str]:
490490
491491
Throws an EdenNotRunningError if EdenFS does not currently appear to be running.
492492
"""
493-
bi = self.get_server_build_info_legacy()
493+
bi = self.get_server_build_info()
494494
return (
495495
bi.get("build_package_version", ""),
496496
bi.get("build_package_release", ""),
@@ -1314,10 +1314,6 @@ def _get_client_dir_for_mount_point(self, path: Path) -> Path:
13141314
def _get_clients_dir(self) -> Path:
13151315
return self._config_dir / CLIENTS_DIR
13161316

1317-
def get_server_build_info_legacy(self) -> Dict[str, str]:
1318-
with self.get_thrift_client_legacy(timeout=3) as client:
1319-
return client.getRegexExportedValues("^build_.*")
1320-
13211317
def get_server_build_info(self) -> Dict[str, str]:
13221318
with self.get_thrift_client(timeout=3) as client:
13231319
return dict(client.getRegexExportedValues("^build_.*"))

eden/fs/cli/doctor/test/lib/fake_eden_instance.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -285,9 +285,6 @@ def check_health(self) -> HealthStatus:
285285
def check_privhelper_connection(self) -> bool:
286286
return True
287287

288-
def get_server_build_info_legacy(self) -> Dict[str, str]:
289-
return dict(self._build_info)
290-
291288
def get_server_build_info(self) -> Dict[str, str]:
292289
return dict(self._build_info)
293290

eden/fs/service/BUCK

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -391,8 +391,8 @@ cpp_library(
391391
"//folly/chrono:conv",
392392
"//folly/coro:collect",
393393
"//folly/coro:invoke",
394-
"//folly/executors:cpu_thread_pool_executor",
395394
"//folly/executors/thread_factory:named_thread_factory",
395+
"//folly/executors:cpu_thread_pool_executor",
396396
"//folly/io/async:async_signal_handler",
397397
"//folly/json:dynamic",
398398
"//folly/logging:logging",

0 commit comments

Comments
 (0)