Auto merge of #722 - saethlin:flaky-crates, r=Mark-Simulacrum

Ignore some more crates

Adding all the spurious failures I identified in rust-lang/rust#121282 (comment)

I'm also not sure what `blacklist.md` is for, but it hasn't been updated in 6 years so I imagine it's not important.

Author: bors

Cargo.lock

@@ -135,6 +135,24 @@ heap-vec = { skip-tests = true } # UB, sometimes segfaults
 k2_tree = { skip-tests = true } # UB, sometimes segfaults
 stabby-macros = { skip = true } # broken rustc version parsing
 stabby = { skip = true } # broken rustc version parsing
+knetrs = { skip-tests = true } # UB
+yices2 = { skip = true } # Sometimes times out, sometimes doesn't
+scheduled-executor = { skip-tests = true } # UB, allocator corruption
+skippy-rs = { skip-tests = true } # UB, out-of-bounds get_unchecked
+pepe-telemetry = { skip-test = true } # flaky test (concurrency)
+puid = { skip-tests = true } # flaky test (timing)
+watchable = { skip-tests = true } # flaky test (timing)
+lispi = { skip-tests = true } # flaky test (data races)
+cargo-ramdisk = { skip-tests = true } # flaky test (concurrency)
+njord = { skip-tests = true } # flaky test
+matecito = { skip-tests = true } # flaky test (timing)
+namaste = { skip-tests = true} # flaky test
+faktory = { skip-tests = true } # UB via atomic-option
+gazebo = { skip-tests = true } # UB but only in the test suite
+context_bind = { skip-tests = true } # UB
+unbounded-spsc = { skip-tests = true } # UB
+vtable_gen = { skip-tests = true } # UB
+crt0stack = { skip-tests = true } # UB
 
 [github-repos]
 # "org_name/repo_name" = { option = true }
@@ -175,5 +193,26 @@ stabby = { skip = true } # broken rustc version parsing
 "mistrpopo/MandelbrotAnimation" = { skip-tests = true } # second build seems to always fail
 "bastibl/futuresdr" = { skip = true } # unstable features
 "FutureSDR/FutureSDR" = { skip = true } # unstable features
+"yamgent/rusty-jam-2-dog-chicken" = { skip-tests = true } # UB
+"Liorst4/liorforth" = { skip-tests = true } # UB
+"perlindgren/fastmem" = { skip-tests = true } # UB
+"aharisu/navi" = { skip-tests = true } # UB, allocator corruption
+"fkjogu/stochasticsampling" = { skip-tests = true } # UB, allocator corruption
+"senrust/toy_compiler2" = { skip-tests = true } # UB when tests run concurrently
+"Yogaflre/leetcode" = { skip-tests = true } # UB
+"UndeadRat22/kzg10-rust" = { skip-tests = true } # Wraps a non-thread-safe C++ library
+"JakeDawkins/graphql-client-302-repro" = { skip-tests = true } # Stack overflow in a proc macro
+"cdstanford/hydroflow-hackathon" = { skip-tests = true } # UB in a proc macro
+"hydro-project/compute-pi" = { skip-tests = true } # UB in a proc macro
+"KatsukiFujimoto/git-rs" = { skip-tests = true } # flaky test (HashSet iteration)
+"MarkChuCarroll/schism" = { skip-tests = true } # flaky test (concurrency)
+"Osrepnay/algae" = { skip-tests = true } # flaky test (timing)
+"Yichangcs/Ruscheme" = { skip-tests = true } # flaky test (concurrency)
+"bakaq/rust-blackhole" = { skip-tests = true } # flaky test (rng)
+"diegoasanch/advent-of-code-2023" = { skip-tests = true } # flaky test (HashMap iteration)
+"grnmeira/Snake" = { skip-tests = true } # flaky test (rng)
+"wojciechkepka/pkger" = { skip-tests = true } # flaky test (concurrency)
+"xffxff/muzero-rs" = { skip-tests = true } # flaky test (rng)
+"maxjeffos/rs_dynamic_args" = { skip-tests = true } # flaky test (concurrency)
 
 [local-crates]

blacklist.md

@@ -13,7 +13,6 @@ use crate::utils::disk_usage::DiskUsage;
 use failure::Error;
 use rustwide::Workspace;
 use std::collections::BTreeSet;
-use std::iter::FromIterator;
 use std::ops;
 use std::sync::atomic::{AtomicBool, Ordering};
 use std::thread;

config.toml

@@ -1,5 +1,5 @@
 use crate::prelude::*;
-use mime::{self, Mime};
+use mime::Mime;
 use serde::Serialize;
 use std::borrow::Cow;
 use std::collections::HashMap;

src/agent/mod.rs

@@ -34,7 +34,7 @@ use std::time::Duration;
 pub struct Ex(String);
 
 #[derive(Debug, Clone)]
-pub struct DockerEnv(String);
+pub struct DockerEnv(#[allow(unused)] String);
 impl FromStr for Ex {
     type Err = Error;
 

src/assets.rs

@@ -6,7 +6,6 @@ use crate::prelude::*;
 use cargo_metadata::PackageId;
 use percent_encoding::{percent_decode_str, utf8_percent_encode, NON_ALPHANUMERIC};
 use rustwide::Crate as RustwideCrate;
-use std::convert::TryFrom;
 use std::fmt;
 use std::path::Path;
 use std::str::FromStr;

src/cli.rs

@@ -1,8 +1,5 @@
 use crate::prelude::*;
-use rand::{
-    self,
-    distributions::{Alphanumeric, DistString},
-};
+use rand::distributions::{Alphanumeric, DistString};
 use rusqlite::{Connection, Transaction};
 use std::collections::HashSet;
 
@@ -388,7 +385,7 @@ pub fn execute(db: &mut Connection) -> Fallible<()> {
     };
 
     for &(name, ref migration) in &migrations() {
-        if !executed_migrations.contains(&name.to_string()) {
+        if !executed_migrations.contains(name) {
             let t = db.transaction()?;
             match migration {
                 MigrationKind::SQL(sql) => t.execute_batch(sql),

src/crates/mod.rs

@@ -8,12 +8,11 @@ use crate::results::{EncodedLog, EncodingType, FailureReason, ReadResults, TestR
 use crate::toolchain::Toolchain;
 use crate::utils;
 use crates_index::GitIndex;
-use mime::{self, Mime};
+use mime::Mime;
 use percent_encoding::{utf8_percent_encode, AsciiSet};
 use std::borrow::Cow;
 #[cfg(test)]
 use std::collections::HashMap;
-use std::convert::AsRef;
 use std::fmt::{self, Display};
 use std::fs;
 use std::path::{Path, PathBuf};

src/db/migrations.rs

@@ -13,7 +13,6 @@ use rustwide::cmd::{CommandError, ProcessLinesActions, SandboxBuilder};
 use rustwide::logging::LogStorage;
 use rustwide::{Build, PrepareError};
 use std::collections::{BTreeSet, HashMap, HashSet};
-use std::convert::TryFrom;
 
 fn failure_reason(err: &Error) -> FailureReason {
     for cause in err.iter_chain() {

src/report/mod.rs

@@ -7,7 +7,7 @@ use regex::Regex;
 use rust_team_data::v1 as team_data;
 use std::collections::{HashMap, HashSet};
 use std::sync::{Arc, RwLock};
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 lazy_static! {
     static ref GIT_REVISION_RE: Regex =

src/runner/test.rs

@@ -17,12 +17,12 @@ use crate::server::agents::Agents;
 use crate::server::auth::ACL;
 use crate::server::github::{GitHub, GitHubApi};
 use crate::server::tokens::{BotTokens, Tokens};
-use http::{self, header::HeaderValue, Response};
+use http::{header::HeaderValue, Response};
 use hyper::Body;
 use metrics::Metrics;
 use std::net::SocketAddr;
 use std::sync::{Arc, Mutex};
-use warp::{self, Filter};
+use warp::Filter;
 
 lazy_static! {
     static ref SERVER_HEADER: String =

src/server/auth.rs

@@ -14,7 +14,7 @@ use hyper::Body;
 use std::collections::HashMap;
 use std::sync::{Arc, Condvar, Mutex};
 use std::time::Instant;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 #[derive(Deserialize)]
 #[serde(rename_all = "kebab-case")]

src/server/mod.rs

@@ -5,7 +5,7 @@ use http::{Response, StatusCode};
 use hyper::Body;
 use prometheus::{Encoder, TextEncoder};
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 pub fn routes(
     data: Arc<Data>,

src/server/routes/agent.rs

@@ -6,7 +6,7 @@ use http::{Response, StatusCode};
 use hyper::Body;
 use serde::Serialize;
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 mod agents;
 mod experiments;

src/server/routes/metrics.rs

@@ -1,6 +1,5 @@
 use crate::experiments::{Assignee, CapLints, DeferredCrateSelect, Mode};
 use crate::toolchain::Toolchain;
-use failure::{self, Fallible};
 
 #[derive(Debug, thiserror::Error)]
 #[cfg_attr(test, derive(PartialEq, Eq))]

src/server/routes/ui/mod.rs

@@ -12,7 +12,7 @@ use http::{HeaderMap, Response, StatusCode};
 use hyper::Body;
 use std::str::FromStr;
 use std::sync::Arc;
-use warp::{self, Filter, Rejection};
+use warp::{Filter, Rejection};
 
 fn process_webhook(
     payload: &[u8],

src/server/routes/webhooks/args.rs

@@ -1,5 +1,5 @@
 use crate::prelude::*;
-use failure::{AsFail, Backtrace, Fail};
+use failure::{AsFail, Backtrace};
 use percent_encoding::{AsciiSet, CONTROLS};
 use std::any::Any;
 use std::fmt;

src/server/routes/webhooks/mod.rs

@@ -1,5 +1,4 @@
 use serde::ser::{Serialize, SerializeSeq, Serializer};
-use std::iter::IntoIterator;
 
 pub fn to_vec<S, T>(data: T, serializer: S) -> Result<S::Ok, S::Error>
 where