simulator: don't treat dial errors as errors

Leads to too many superfluous io-timeout messages.

Author: kmroz

simulator/miner.go

@@ -44,7 +44,8 @@ func (s *StratumMiner) Simulate(ctx context.Context, dst string) error {
 		conn.Close()
 	}
 
-	if isSoftError(err, "connect: connection refused") {
+	// TODO Dropping isDialError() as an error as it's causing too many io-timeout messages.
+	if isSoftError(err, "connect: connection refused") || isDialError(err) {
 		return nil
 	}
 	return err

simulator/simulator.go

@@ -72,9 +72,8 @@ func (s *TCPConnectSimulator) Simulate(ctx context.Context, dst string) error {
 	if conn != nil {
 		conn.Close()
 	}
-	// This will likely generate some superfluous io-timeout error messages, but if the
-	// user specifies a misconfigured interface, they'll see the simulation failing.
-	if err != nil && (!isSoftError(err, "connect: connection refused") || isDialError(err)) {
+	// TODO: Dropping isDialError() check, as it's causing too many io-timeout messages.
+	if err != nil && !isSoftError(err, "connect: connection refused") && !isDialError(err) {
 		return err
 	}
 	return nil
@@ -112,7 +111,8 @@ func (s *DNSResolveSimulator) Simulate(ctx context.Context, dst string) error {
 
 	_, err := r.LookupHost(ctx, host)
 	// Ignore "no such host".  Will ignore timeouts as well, so check for dial errors.
-	if err != nil && (!isSoftError(err, "no such host") || isDialError(err)) {
+	// TODO: Dropping isDialError() check, as it's causing too many io-timeout messages.
+	if err != nil && !isSoftError(err, "no such host") && !isDialError(err) {
 		return err
 	}
 

simulator/spambot.go

@@ -89,12 +89,11 @@ func (s *Spambot) Hosts(scope string, size int) ([]string, error) {
 	for n := 0; len(hosts) < size && n < len(idx); n++ {
 		ctx, cancel := context.WithTimeout(context.Background(), 300*time.Millisecond)
 		host := utils.FQDN(domains[idx[n]])
-		mx, err := rv.LookupMX(ctx, host)
+		mx, _ := rv.LookupMX(ctx, host)
 		cancel()
 		// Check error message for sign of resolver/routing issue.
-		if err != nil && isDialError(err) {
-			return hosts, err
-		}
+		// TODO: at some point we'll want to check dialer errors for a sign of resolver
+		// problems
 		if len(mx) > 0 {
 			host := strings.TrimSuffix(mx[0].Host, ".")
 			if !seen[host] {

simulator/tunnel-dns.go

@@ -57,7 +57,7 @@ func (s *Tunnel) Simulate(ctx context.Context, host string) error {
 		_, err := r.LookupTXT(ctx, fmt.Sprintf("%s.%s", label, host))
 
 		// Ignore "no such host".  Will ignore timeouts as well, so check for dial errors.
-		if err != nil && (!isSoftError(err, "no such host") || isDialError(err)) {
+		if err != nil && !isSoftError(err, "no such host") && !isDialError(err) {
 			return err
 		}