Skip to content

Commit 661106e

Browse files
committed
MB-48432: Disk usage analyzer alerts should be windows only
Ensure that calls to is_stale won't crash on windows, by making accesses to the ns_disksup ets table gated behind macos and linux only. Change-Id: Iafcbe858a0158dc57d5006519de2a60c12a636bc Reviewed-on: http://review.couchbase.org/c/ns_server/+/161532 Well-Formed: Restriction Checker Well-Formed: Build Bot <[email protected]> Tested-by: Build Bot <[email protected]> Tested-by: Bryan McCoid <[email protected]> Reviewed-by: Aliaksey Artamonau <[email protected]>
1 parent d3beffc commit 661106e

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

src/ns_disksup.erl

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,16 @@ get_disk_data() ->
4949
end.
5050

5151
is_stale() ->
52-
case get_latest_entry() of
53-
{none, _} ->
54-
true;
55-
{Ts, _, Timeout} ->
56-
check_staleness(Ts, Timeout * 2)
52+
case misc:is_linux() orelse misc:is_macos() of
53+
true ->
54+
case get_latest_entry() of
55+
{none, _} ->
56+
true;
57+
{Ts, _, Timeout} ->
58+
check_staleness(Ts, Timeout * 2)
59+
end;
60+
false ->
61+
false
5762
end.
5863

5964
%%----------------------------------------------------------------------

0 commit comments

Comments
 (0)