Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

service: don't use sysinfo on Linux for metric collection #6805

Closed
andresilva wants to merge 3 commits into from
Closed

service: don't use sysinfo on Linux for metric collection #6805

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c52cafd
service: don't track metrics of all processes in system
andresilva Aug 3, 2020
79d3d05
service: don't use sysinfo on Linux
andresilva Aug 4, 2020
18e66e5
service: fix wasm compilation
andresilva Aug 4, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
service: fix wasm compilation
  • Loading branch information
@andresilva
andresilva committed Aug 4, 2020
commit 18e66e563ad4d498938e4e2e7eb59fad00bde994
6 changes: 3 additions & 3 deletions client/service/src/metrics.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -383,9 +383,9 @@ mod sysinfo {
}
}

#[cfg(any(target_os = "android", target_os = "ios"))]
#[cfg(not(all(any(unix, windows), not(target_os = "android"), not(target_os = "ios"))))]
mod noop {
use super::ProcessInfo;
use super::{LoadAverage, ProcessInfo};

pub(crate) struct System;

Expand Down Expand Up @@ -413,7 +413,7 @@ type System = linux::System;
not(target_os = "linux")
))]
type System = sysinfo::System;
#[cfg(any(target_os = "android", target_os = "ios"))]
#[cfg(not(all(any(unix, windows), not(target_os = "android"), not(target_os = "ios"))))]
type System = noop::System;

pub struct MetricsService {
Expand Down