From 184002a674f328a0c0f1ba1aa4b60b7d40235ea3 Mon Sep 17 00:00:00 2001
From: Jan Kamieth <jan@justtrack.io>
Date: Wed, 14 Feb 2024 09:15:28 +0100
Subject: [PATCH] more review fixes

---
 builder/dashboard.go                                   |  6 +++---
 builder/dashboard_test.go                              |  4 ++--
 builder/panel.go                                       |  4 ++--
 .../datasource_application_dashboard_definition.go     | 10 +++++-----
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/builder/dashboard.go b/builder/dashboard.go
index c9352d2..653772a 100644
--- a/builder/dashboard.go
+++ b/builder/dashboard.go
@@ -16,12 +16,12 @@ type Dashboard struct {
 }
 
 type DashboardBuilder struct {
-	resourceNames  ResourceNames
+	resourceNames  *ResourceNames
 	panelFactories []PanelFactory
 	orchestrator   string
 }
 
-func NewDashboardBuilder(resourceNames ResourceNames, orchestrator string) *DashboardBuilder {
+func NewDashboardBuilder(resourceNames *ResourceNames, orchestrator string) *DashboardBuilder {
 	return &DashboardBuilder{
 		resourceNames:  resourceNames,
 		panelFactories: make([]PanelFactory, 0),
@@ -175,7 +175,7 @@ func (d *DashboardBuilder) Build() Dashboard {
 
 func (d *DashboardBuilder) buildPanel(factory PanelFactory, x int, y int) Panel {
 	panelGridPos := NewPanelGridPos(PanelHeight, PanelWidth, x, y)
-	settings := newPaneSettings(d.resourceNames, panelGridPos, d.orchestrator)
+	settings := newPanelSettings(d.resourceNames, panelGridPos, d.orchestrator)
 	panel := factory(settings)
 
 	if panel.FieldConfig.Defaults.Custom.AxisPlacement == "" {
diff --git a/builder/dashboard_test.go b/builder/dashboard_test.go
index 6713c55..da23fd9 100644
--- a/builder/dashboard_test.go
+++ b/builder/dashboard_test.go
@@ -33,7 +33,7 @@ func TestEcsDashboardWithError(t *testing.T) {
 		},
 	}
 
-	resourceNames := builder.ResourceNames{
+	resourceNames := &builder.ResourceNames{
 		CloudwatchNamespace:                cloudwatchNamespace,
 		EcsCluster:                         ecsClusterName,
 		EcsService:                         ecsServiceName,
@@ -82,7 +82,7 @@ func TestKubernetesDashboardWithError(t *testing.T) {
 	kubernetesPod := "pod"
 	var targetGroups []builder.ElbTargetGroup
 
-	resourceNames := builder.ResourceNames{
+	resourceNames := &builder.ResourceNames{
 		CloudwatchNamespace:                cloudwatchNamespace,
 		EcsCluster:                         ecsClusterName,
 		EcsService:                         ecsServiceName,
diff --git a/builder/panel.go b/builder/panel.go
index 32424aa..0c01ed5 100644
--- a/builder/panel.go
+++ b/builder/panel.go
@@ -1,6 +1,6 @@
 package builder
 
-func newPaneSettings(resourceNames ResourceNames, gridPos PanelGridPos, orchestrator string) PanelSettings {
+func newPanelSettings(resourceNames *ResourceNames, gridPos PanelGridPos, orchestrator string) PanelSettings {
 	return PanelSettings{
 		resourceNames: resourceNames,
 		gridPos:       gridPos,
@@ -9,7 +9,7 @@ func newPaneSettings(resourceNames ResourceNames, gridPos PanelGridPos, orchestr
 }
 
 type PanelSettings struct {
-	resourceNames ResourceNames
+	resourceNames *ResourceNames
 	gridPos       PanelGridPos
 	orchestrator  string
 }
diff --git a/provider/datasource_application_dashboard_definition.go b/provider/datasource_application_dashboard_definition.go
index b2f12bc..b5ec9e0 100644
--- a/provider/datasource_application_dashboard_definition.go
+++ b/provider/datasource_application_dashboard_definition.go
@@ -89,7 +89,7 @@ func (a *ApplicationDashboardDefinitionDataSource) Read(ctx context.Context, req
 
 	var err error
 	var metadata *builder.MetadataApplication
-	var resourceNames builder.ResourceNames
+	var resourceNames *builder.ResourceNames
 
 	if metadata, err = a.metadataReader.ReadMetadata(state.AppId()); err != nil {
 		response.Diagnostics.AddError("can not get metadata", err.Error())
@@ -160,7 +160,7 @@ func (a *ApplicationDashboardDefinitionDataSource) Read(ctx context.Context, req
 	response.Diagnostics.Append(diags...)
 }
 
-func (a *ApplicationDashboardDefinitionDataSource) getResourceNames(ctx context.Context, state *ApplicationDashboardDefinitionData, response *tfsdk.ReadDataSourceResponse) (builder.ResourceNames, error) {
+func (a *ApplicationDashboardDefinitionDataSource) getResourceNames(ctx context.Context, state *ApplicationDashboardDefinitionData, response *tfsdk.ReadDataSourceResponse) (*builder.ResourceNames, error) {
 	var err error
 
 	// Always available
@@ -190,7 +190,7 @@ func (a *ApplicationDashboardDefinitionDataSource) getResourceNames(ctx context.
 		targetGroups, ecsTaskDefinitionName, err = a.getEc2AndEcsData(ctx, response, ecsClusterName, ecsServiceName)
 
 		if err != nil {
-			return builder.ResourceNames{}, err
+			return nil, err
 		}
 	case orchestratorKubernetes:
 		kubernetesNamespace = builder.Augment(a.resourceNamePatterns.KubernetesNamespace, state.AppId())
@@ -198,7 +198,7 @@ func (a *ApplicationDashboardDefinitionDataSource) getResourceNames(ctx context.
 		traefikServiceName = builder.Augment(a.resourceNamePatterns.TraefikServiceName, state.AppId())
 	}
 
-	resourceNames := builder.ResourceNames{
+	resourceNames := &builder.ResourceNames{
 		CloudwatchNamespace:                cloudwatchNamespace,
 		EcsCluster:                         ecsClusterName,
 		EcsService:                         ecsServiceName,
@@ -243,7 +243,7 @@ func (a *ApplicationDashboardDefinitionDataSource) getEc2AndEcsData(ctx context.
 	return targetGroups, *ecsTaskDefinitionName, err
 }
 
-func (a *ApplicationDashboardDefinitionDataSource) addHttpServers(metadata *builder.MetadataApplication, resourceNames builder.ResourceNames, db *builder.DashboardBuilder) {
+func (a *ApplicationDashboardDefinitionDataSource) addHttpServers(metadata *builder.MetadataApplication, resourceNames *builder.ResourceNames, db *builder.DashboardBuilder) {
 	if len(metadata.HttpServers) == 0 {
 		return
 	}