From 1d511e439b429cd06ca08e89f31409a6865aca77 Mon Sep 17 00:00:00 2001
From: Colin Simmons <5785492+crsimmons@users.noreply.github.com>
Date: Thu, 17 Mar 2022 23:42:36 +0000
Subject: [PATCH] remove redundant variable assignment

---
 concourse/deploy.go | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/concourse/deploy.go b/concourse/deploy.go
index 9e78ab076..7bd72a807 100644
--- a/concourse/deploy.go
+++ b/concourse/deploy.go
@@ -134,21 +134,6 @@ func (client *Client) deployBoshAndPipeline(c config.ConfigView, tfOutputs terra
 	// When we are deploying for the first time rather than updating
 	// ensure that the pipeline is set _after_ the concourse is deployed
 
-	// TODO: Surely this can be removed as it is overriden on line 153 anyway?
-	bp := BoshParams{
-		CredhubPassword:          c.GetCredhubPassword(),
-		CredhubAdminClientSecret: c.GetCredhubAdminClientSecret(),
-		CredhubCACert:            c.GetCredhubCACert(),
-		CredhubURL:               c.GetCredhubURL(),
-		CredhubUsername:          c.GetCredhubUsername(),
-		ConcourseUsername:        c.GetConcourseUsername(),
-		ConcoursePassword:        c.GetConcoursePassword(),
-		GrafanaPassword:          c.GetGrafanaPassword(),
-		DirectorUsername:         c.GetDirectorUsername(),
-		DirectorPassword:         c.GetDirectorPassword(),
-		DirectorCACert:           c.GetDirectorCACert(),
-	}
-
 	bp, err := client.deployBosh(c, tfOutputs, false)
 	if err != nil {
 		return bp, err