Reduce needless sleeps in collector/collect.sh

Before, the collector would sleep for 2 minutes after every collection, both if something was benchmarked and if nothing was benchmarked because the queue was empty. When the queue is not empty, it's wasteful to sleep for two minutes.

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);
             };