Merge pull request #1676 from Kobzol/collect-busy-loop

Reduce needless sleeps in `collector/collect.sh`

Author: Kobzol

collector/collect.sh

@@ -22,9 +22,12 @@ while : ; do
         # Install measureme tooling
         cargo install --git https://github.com/rust-lang/measureme --branch stable flamegraph crox summarize
 
-        target/release/collector bench_next $SITE_URL --self-profile --bench-rustc --db $DATABASE;
-        echo finished run at `date`;
-
-        # Wait a little bit before the next run starts.
-        sleep 120
+        target/release/collector bench_next $SITE_URL --self-profile --bench-rustc --db $DATABASE
+        STATUS=$?
+        echo finished run at `date` with exit code $STATUS
+
+        # Wait a bit if the command has failed.
+        if [ $STATUS -ne 0 ]; then
+            sleep 120
+        fi
 done

collector/src/bin/collector.rs

@@ -747,6 +747,12 @@ fn main_result() -> anyhow::Result<i32> {
                 c
             } else {
                 println!("no artifact to benchmark");
+
+                // Sleep for a bit to avoid spamming the perf server too much
+                // This sleep serves to remove a needless sleep in `collector/collect.sh` when
+                // a benchmark was actually executed.
+                std::thread::sleep(Duration::from_secs(60 * 2));
+
                 // no missing artifacts
                 return Ok(0);
             };