Skip to content

Commit 7bf0ad2

Browse files
ViniGupta08facebook-github-bot
authored andcommitted
Update eden running version threshhold from 30 to 45 days
Summary: As per the discussion in the document linked below, we will display EdenFS Health Report notifications for stale EdenFS versions only when the running version and installed version differ by more than 45 days. This change is part of the first proposal, which increases the gap in the Eden version from 30 days to 45 days. https://docs.google.com/document/d/1fTxlasZfWv58iVhRr6Um3nSIOk4R0e4J6lxsCzG4W9E/edit?usp=sharing Reviewed By: kavehahmadi60 Differential Revision: D74971406 fbshipit-source-id: ce01b40e991a14b6b67360f5e3988e9d62bbc784
1 parent ce3cf2e commit 7bf0ad2

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

eden/fs/cli/main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1296,7 +1296,7 @@ def remediation(self) -> str:
12961296

12971297
running_version: str = ""
12981298
# a value of 0 turns off the stale version check
1299-
running_version_days_threshold = 30
1299+
running_version_days_threshold = 45
13001300
version_info: VersionInfo = VersionInfo()
13011301
error_codes: Dict[ErrorCode, str] = {}
13021302

eden/fs/cli/test/health_report_test.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,12 @@
2222
from eden.test_support.temporary_directory import TemporaryDirectoryMixin
2323

2424
from ..main import HealthReportCmd
25-
from eden.fs.cli.main import HealthReportCmd
2625

2726
# The version "20241030-165642" is the latest version.
28-
# The version "20240928-144752" is over 30 days old, which is considered stale.
29-
# The version "20241010-189752" is less than 30 days old, which is considered acceptable.
27+
# The version "20240910-144752" is over 45 days old, which is considered stale.
28+
# The version "20241010-189752" is less than 45 days old, which is considered acceptable.
3029
LATEST_VERSION_STR = "20241030-165642"
31-
STALE_VERSION_STR = "20240928-144752"
30+
STALE_VERSION_STR = "20240910-144752"
3231
ACCEPTABLE_VERSION_STR = "20241010-189752"
3332

3433

@@ -279,7 +278,7 @@ def test_health_report_check_for_stale_eden_version_prompt_error(
279278
self.assertEqual(
280279
HealthReportCmd.error_codes,
281280
{
282-
HealthReportCmd.ErrorCode.STALE_EDEN_VERSION: "Running EdenFS version: 20240928-144752, installed EdenFS version: 20241030-165642. The running EdenFS daemon is over 30 days out-of-date."
281+
HealthReportCmd.ErrorCode.STALE_EDEN_VERSION: "Running EdenFS version: 20240910-144752, installed EdenFS version: 20241030-165642. The running EdenFS daemon is over 45 days out-of-date."
283282
},
284283
)
285284
self.assertEqual(result, 1)

eden/fs/config/EdenConfig.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1634,7 +1634,7 @@ class EdenConfig : private ConfigSettingManager {
16341634
*/
16351635
ConfigSetting<size_t> healthReportStaleVersionThresholdDays{
16361636
"notifications:health-report-stale-version-threshold-days",
1637-
30,
1637+
45,
16381638
this};
16391639

16401640
/**

0 commit comments

Comments
 (0)