From 842ca824870fc6cec1aa0f6f2b3ff74051fe6ad3 Mon Sep 17 00:00:00 2001
From: shreddedbacon <b@benjackson.email>
Date: Fri, 23 Feb 2024 15:08:21 +1100
Subject: [PATCH] refactor: add basic unauthenticated registry passthrough

---
 controllers/v1beta1/build_controller.go | 1 +
 controllers/v1beta1/build_helpers.go    | 2 +-
 main.go                                 | 8 ++++++++
 3 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/controllers/v1beta1/build_controller.go b/controllers/v1beta1/build_controller.go
index 421cdac3..b974cd14 100644
--- a/controllers/v1beta1/build_controller.go
+++ b/controllers/v1beta1/build_controller.go
@@ -75,6 +75,7 @@ type LagoonBuildReconciler struct {
 	LagoonFeatureFlags               map[string]string
 	LagoonAPIConfiguration           helpers.LagoonAPIConfiguration
 	ProxyConfig                      ProxyConfig
+	UnauthenticatedRegistry          string
 }
 
 // BackupConfig holds all the backup configuration settings
diff --git a/controllers/v1beta1/build_helpers.go b/controllers/v1beta1/build_helpers.go
index 81831ab1..bdd7d2fd 100644
--- a/controllers/v1beta1/build_helpers.go
+++ b/controllers/v1beta1/build_helpers.go
@@ -507,7 +507,7 @@ func (r *LagoonBuildReconciler) processBuild(ctx context.Context, opLog logr.Log
 	})
 	podEnvs = append(podEnvs, corev1.EnvVar{
 		Name:  "REGISTRY",
-		Value: lagoonBuild.Spec.Project.Registry,
+		Value: r.UnauthenticatedRegistry,
 	})
 	// this is enabled by default for now
 	// eventually will be disabled by default because support for the generation/modification of this will
diff --git a/main.go b/main.go
index 7238b1ab..60f4463b 100644
--- a/main.go
+++ b/main.go
@@ -175,6 +175,8 @@ func main() {
 	var enablePodProxy bool
 	var podsUseDifferentProxy bool
 
+	var unauthenticatedRegistry string
+
 	flag.StringVar(&metricsAddr, "metrics-addr", ":8080",
 		"The address the metric endpoint binds to.")
 	flag.StringVar(&lagoonTargetName, "lagoon-target-name", "ci-local-control-k8s",
@@ -334,6 +336,9 @@ func main() {
 	flag.StringVar(&harborWebhookEventTypes, "harbor-webhook-eventtypes", "SCANNING_FAILED,SCANNING_COMPLETED",
 		"The event types to use for the Lagoon webhook")
 
+	// this is for legacy reasons, and backwards compatability support due to removal of https://github.com/uselagoon/lagoon/pull/3659
+	flag.StringVar(&unauthenticatedRegistry, "unauthenticated-registry", "registry.lagoon.svc:5000", "An unauthenticated registry URL that could be used for local testing without a harbor")
+
 	// NS cleanup configuration
 	flag.BoolVar(&cleanNamespacesEnabled, "enable-namespace-cleanup", false,
 		"Tells the controller to remove namespaces marked for deletion with labels (lagoon.sh/expiration=<unixtimestamp>).")
@@ -407,6 +412,8 @@ func main() {
 
 	nativeCronPodMinFrequency = helpers.GetEnvInt("NATIVE_CRON_POD_MINIMUM_FREQUENCY", nativeCronPodMinFrequency)
 
+	unauthenticatedRegistry = helpers.GetEnv("UNAUTHENTICATED_REGISTRY", unauthenticatedRegistry)
+
 	// harbor envvars
 	harborURL = helpers.GetEnv("HARBOR_URL", harborURL)
 	harborAPI = helpers.GetEnv("HARBOR_API", harborAPI)
@@ -805,6 +812,7 @@ func main() {
 			HTTPSProxy: httpsProxy,
 			NoProxy:    noProxy,
 		},
+		UnauthenticatedRegistry: unauthenticatedRegistry,
 	}).SetupWithManager(mgr); err != nil {
 		setupLog.Error(err, "unable to create controller", "controller", "LagoonBuild")
 		os.Exit(1)