fix: remove dep cycle with certreloader (#1410)

#1389 works well but caused a dependency cycle: core -> flagd -> core. I've moved the `certreloader` pkg to core with the existing telemetry stuff. cc @kevinschoonover Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
open-feature · Sep 23, 2024 · 5244f6f · 5244f6f
1 parent 8737f53
commit 5244f6f
Show file tree

Hide file tree

Showing 5 changed files with 109 additions and 4,546 deletions.
diff --git a/flagd/pkg/certreloader/certreloader.go → core/pkg/certreloader/certreloader.go b/flagd/pkg/certreloader/certreloader.go → core/pkg/certreloader/certreloader.go
diff --git a/flagd/pkg/certreloader/certreloader_test.go → core/pkg/certreloader/certreloader_test.go b/flagd/pkg/certreloader/certreloader_test.go → core/pkg/certreloader/certreloader_test.go
diff --git a/core/pkg/telemetry/builder.go b/core/pkg/telemetry/builder.go
@@ -10,8 +10,8 @@ import (
 
 	"connectrpc.com/connect"
 	"connectrpc.com/otelconnect"
+	"github.com/open-feature/flagd/core/pkg/certreloader"
 	"github.com/open-feature/flagd/core/pkg/logger"
-	"github.com/open-feature/flagd/flagd/pkg/certreloader"
 	"go.opentelemetry.io/otel"
 	"go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc"
 	"go.opentelemetry.io/otel/exporters/otlp/otlptrace"