Merge pull request #25 from ethpandaops/feat/default-ntp-server

feat(sentry): set ntp default server to gcp
ethpandaops · Nov 24, 2022 · 01ed43e · 01ed43e
2 parents 913cd89 + 4a9fb50
commit 01ed43e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/example_sentry.yaml b/example_sentry.yaml
@@ -14,7 +14,8 @@ ethereum:
 #   time.aws.com - AWS
 #   time.windows.com - Azure
 #   time.google.com - GCP
-ntp_server: pool.ntp.org
+#   pool.ntp.org - https://www.pool.ntp.org/zone/@
+ntp_server: time.google.com
 
 outputs:
 - name: basic

diff --git a/pkg/sentry/config.go b/pkg/sentry/config.go
@@ -25,7 +25,7 @@ type Config struct {
 	Labels map[string]string `yaml:"labels"`
 
 	// NTP Server to use for clock drift correction
-	NTPServer string `yaml:"ntp_server" default:"pool.ntp.org"`
+	NTPServer string `yaml:"ntp_server" default:"time.google.com"`
 }
 
 func (c *Config) Validate() error {

diff --git a/pkg/sentry/sentry.go b/pkg/sentry/sentry.go
@@ -210,7 +210,7 @@ func (s *Sentry) syncClockDrift(ctx context.Context) error {
 	}
 
 	s.clockDrift = response.ClockOffset
-	s.log.WithField("drift", s.clockDrift).Debug("Updated clock drift")
+	s.log.WithField("drift", s.clockDrift).Info("Updated clock drift")
 
 	return err
 }