Skip to content

Commit

Permalink
dns-controller: don't expose prometheus metrics by default
Browse files Browse the repository at this point in the history
  • Loading branch information
justinsb committed Apr 3, 2018
1 parent 0757961 commit 5474753
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions dns-controller/cmd/dns-controller/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ func main() {
flags.StringVar(&gossipSecret, "gossip-secret", gossipSecret, "Secret to use to secure gossip")
flags.StringVar(&watchNamespace, "watch-namespace", "", "Limits the functionality for pods, services and ingress to specific namespace, by default all")
flag.IntVar(&route53.MaxBatchSize, "route53-batch-size", route53.MaxBatchSize, "Maximum number of operations performed per changeset batch")
flag.StringVar(&metricsListen, "metrics-listen-addr", ":3999", "The address on which to listen for Prometheus metrics.")
flag.StringVar(&metricsListen, "metrics-listen-addr", "", "The address on which to listen for Prometheus metrics.")

// Trick to avoid 'logging before flag.Parse' warning
flag.CommandLine.Parse([]string{})
Expand All @@ -79,10 +79,12 @@ func main() {
flags.AddGoFlagSet(flag.CommandLine)
flags.Parse(os.Args)

go func() {
http.Handle("/metrics", promhttp.Handler())
log.Fatal(http.ListenAndServe(metricsListen, nil))
}()
if metricsListen != "" {
go func() {
http.Handle("/metrics", promhttp.Handler())
log.Fatal(http.ListenAndServe(metricsListen, nil))
}()
}

zoneRules, err := dns.ParseZoneRules(zones)
if err != nil {
Expand Down

0 comments on commit 5474753

Please sign in to comment.