From fadd2303261f6d4ba8a691c5f71898eb2d465aa7 Mon Sep 17 00:00:00 2001
From: Saulius Grigaitis <saulius@grandine.io>
Date: Mon, 23 Dec 2024 17:19:55 +0200
Subject: [PATCH] Trying macos

---
 .github/workflows/ci.yml  | 7 ++++++-
 metrics/src/metric_sys.rs | 6 +++---
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 0bf6a8d5..5a8732c6 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -6,7 +6,7 @@ jobs:
   build:
     strategy:
       matrix:
-        os: [ubuntu-latest, windows-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
 
     runs-on: ${{ matrix.os }}
 
@@ -31,7 +31,12 @@ jobs:
     - name: Run Clippy (Linux)
       if: runner.os == 'Linux'
       run: scripts/ci/clippy.bash --deny warnings
+    # It's a known issue that a couple of networking tests doesn't pass
     - name: Run tests
+      if: runner.os == 'Macos'
+      run: cargo test --release --no-fail-fast -- --skip behaviour::tests::test_scoring_p1 --skip behaviour::tests::test_scoring_p7_grafts_before_backoff
+    - name: Run tests
+      if: runner.os != 'Macos'
       run: cargo test --release --no-fail-fast
     - name: Check consensus-spec-tests coverage (Linux)
       if: runner.os == 'Linux'
diff --git a/metrics/src/metric_sys.rs b/metrics/src/metric_sys.rs
index 5b88eb03..50db2b91 100644
--- a/metrics/src/metric_sys.rs
+++ b/metrics/src/metric_sys.rs
@@ -1,5 +1,5 @@
 use anyhow::Result;
-#[cfg(target_os = "linux")]
+#[cfg(not(target_os = "windows"))]
 use {
     anyhow::bail,
     psutil::{cpu, process::Process},
@@ -15,7 +15,7 @@ pub struct ProcessCpuMetric {
     pub memory_process_bytes: u64,
 }
 
-#[cfg(target_os = "linux")]
+#[cfg(not(target_os = "windows"))]
 pub fn get_process_cpu_metric() -> Result<ProcessCpuMetric> {
     #[expect(unused_assignments)]
     let mut cpu_process_seconds_total = 0;
@@ -78,7 +78,7 @@ pub struct CpuMetric {
 }
 
 // TODO maybe work for MacOS or wider Unix?
-#[cfg(target_os = "linux")]
+#[cfg(not(target_os = "windows"))]
 pub fn get_cpu_metric() -> Result<CpuMetric> {
     let cpu = cpu::cpu_times()?;
     let system_seconds = cpu.total().as_secs();