Revert "kola: Add --no-default-checks"

This reverts commit 0d07f4e.

This switch doesn't work. It relies on `getClusterSemver()` getting
called, which we don't do by default in FCOS.

mantle/cmd/kola/options.go

@@ -63,7 +63,6 @@ func init() {
 	bv(&kola.NoNet, "no-net", false, "Don't run tests that require an Internet connection")
 	ssv(&kola.Tags, "tag", []string{}, "Test tag to run. Can be specified multiple times.")
 	bv(&kola.Options.SSHOnTestFailure, "ssh-on-test-failure", false, "SSH into a machine when tests fail")
-	bv(&kola.Options.SuppressDefaultChecks, "no-default-checks", false, "Disable default checks for failed systemd units and SELinux AVC denials")
 	sv(&kola.Options.Stream, "stream", "", "CoreOS stream ID (e.g. for Fedora CoreOS: stable, testing, next)")
 	sv(&kola.Options.CosaWorkdir, "workdir", "", "coreos-assembler working directory")
 	sv(&kola.Options.CosaBuildId, "build", "", "coreos-assembler build ID")

mantle/kola/harness.go

@@ -883,12 +883,9 @@ func getClusterSemver(flight platform.Flight, outputDir string) (*semver.Version
 		return nil, err
 	}
 
-	cfg := &platform.RuntimeConfig{
-		OutputDir:        testDir,
-		AllowFailedUnits: Options.SuppressDefaultChecks,
-	}
-
-	cluster, err := flight.NewCluster(cfg)
+	cluster, err := flight.NewCluster(&platform.RuntimeConfig{
+		OutputDir: testDir,
+	})
 	if err != nil {
 		return nil, errors.Wrapf(err, "creating cluster for semver check")
 	}

mantle/platform/platform.go

@@ -185,8 +185,7 @@ type Options struct {
 	// inside of RenderUserData
 	OSContainer string
 
-	SSHOnTestFailure      bool
-	SuppressDefaultChecks bool
+	SSHOnTestFailure bool
 }
 
 // RuntimeConfig contains cluster-specific configuration.