feat: add logger to metrics for sqlinstance

Co-authored-by: christeredvartsen <[email protected]>

Author: tommytroen

internal/cmd/api/api.go

@@ -162,7 +162,7 @@ func run(ctx context.Context, cfg *Config, log logrus.FieldLogger) error {
 
 	userSyncRuns := usersync.NewRunsHandler(cfg.UserSync.RunsToPersist)
 	resourceUsageClient := resourceusage.NewClient(cfg.K8s.AllClusterNames(), db, log)
-	sqlinstanceMetrics, err := sqlinstance.NewMetrics(ctx)
+	sqlinstanceMetrics, err := sqlinstance.NewMetrics(ctx, log)
 	if err != nil {
 		return err
 	}

internal/sqlinstance/sqlinstance.go

@@ -14,6 +14,7 @@ import (
 
 type Metrics struct {
 	monitoring *monitoring.MetricClient
+	log        log.FieldLogger
 }
 
 const (
@@ -46,13 +47,16 @@ type MetricsOptions struct {
 
 type Option func(*MetricsOptions)
 
-func NewMetrics(ctx context.Context) (*Metrics, error) {
+func NewMetrics(ctx context.Context, log log.FieldLogger) (*Metrics, error) {
 	client, err := monitoring.NewMetricClient(ctx)
 	if err != nil {
 		return nil, err
 	}
 
-	return &Metrics{monitoring: client}, nil
+	return &Metrics{
+		monitoring: client,
+		log:        log,
+	}, nil
 }
 
 func WithQuery(metricType MetricType, databaseId string) Option {
@@ -150,6 +154,7 @@ func (m *Metrics) ListTimeSeries(ctx context.Context, projectID string, opts ...
 		metric, err := it.Next()
 		// TODO: handle error?
 		if err != nil {
+			m.log.WithError(err).Error("error when fetching time series")
 			break
 		}
 		timeSeries = append(timeSeries, metric)