Race condition in error reporting? about exporter_exporter

[candlerb]

I found the problem.

When I run exporter_exporter, without specifying -web.listen-address, by default it tries to start a server on :9999. But I already had exporter_exporter running on this port, so the HTTP server fails to bind.

Hence the HTTP server immediately terminates, without printing an error but returning an err value. But when a member of an errgroup terminates with an error, all the other members of the group are cancelled. (Possibly the TLS goroutine was cancelled before it even started).

I tried using eg.Wait() but couldn't get it to work - it just hung. But converting the HTTP server to invoke Fatalf() sorts it:

root@ldex-mon2:~# ./exporter_exporter -config.file=/etc/prometheus/exporter_exporter.yml -config.dirs=/etc/prometheus/exporter_exporter.d -web.tls.listen-address ':9998' -web.tls.cert=/path/to/cert.pem -web.tls.key=/path/to/privkey.pem
FATA[0000] Failed starting HTTP server, listen tcp :9999: bind: address already in use  source="main.go:236"

I will submit a PR.