From 53fc7a990d0967a1db95dfd66d2ec2847768e531 Mon Sep 17 00:00:00 2001
From: Michael Jerger <michael.jerger@meissa-gmbh.de>
Date: Tue, 27 Feb 2024 10:09:47 +0100
Subject: [PATCH] fix service names

---
 .../c4k_website/website/website_internal.cljc |  2 --
 src/main/resources/website/nginx-service.yaml |  4 +--
 .../website/website_internal_test.cljc        | 32 +++++++------------
 3 files changed, 13 insertions(+), 25 deletions(-)

diff --git a/src/main/cljc/dda/c4k_website/website/website_internal.cljc b/src/main/cljc/dda/c4k_website/website/website_internal.cljc
index a28414d..4297b6f 100644
--- a/src/main/cljc/dda/c4k_website/website/website_internal.cljc
+++ b/src/main/cljc/dda/c4k_website/website/website_internal.cljc
@@ -102,7 +102,6 @@
         name (replace-dots-by-minus unique-name)]
     (->
      (yaml/load-as-edn "website/nginx-configmap.yaml")
-     (assoc-in [:metadata :labels :app.kubernetes.part-of] name)
      (assoc-in [:metadata :namespace] name)
      (replace-all-matching-substrings-beginning-with "NAME" name)
      (#(assoc-in %
@@ -117,7 +116,6 @@
         name (replace-dots-by-minus unique-name)]
     (->
      (yaml/load-as-edn "website/nginx-service.yaml")
-     (assoc-in [:metadata :labels :app.kubernetes.part-of] name)
      (assoc-in [:metadata :namespace] name)
      (replace-all-matching-substrings-beginning-with "NAME" name))))
 
diff --git a/src/main/resources/website/nginx-service.yaml b/src/main/resources/website/nginx-service.yaml
index 361b5c6..e72704d 100644
--- a/src/main/resources/website/nginx-service.yaml
+++ b/src/main/resources/website/nginx-service.yaml
@@ -5,10 +5,10 @@ metadata:
   namespace: default
   labels:
     app: NAME
-    app.kubernetes.part-of: NAME
+    app.kubernetes.part-of: NAME-website
 spec:    
   selector:
-    app: NAME
+    app: nginx
   ports:
     - name: nginx-http
       port: 80               
diff --git a/src/test/cljc/dda/c4k_website/website/website_internal_test.cljc b/src/test/cljc/dda/c4k_website/website/website_internal_test.cljc
index 25b1c61..307ec29 100644
--- a/src/test/cljc/dda/c4k_website/website/website_internal_test.cljc
+++ b/src/test/cljc/dda/c4k_website/website/website_internal_test.cljc
@@ -83,7 +83,7 @@
                                                             :fqdns ["test.de" "www.test.de" "test-it.de" "www.test-it.de"]})))))
   (is (= {:apiVersion "v1",
           :kind "ConfigMap",
-          :metadata {:labels {:app.kubernetes.part-of "test-io"},
+          :metadata {:labels {:app.kubernetes.part-of "test-io-website"},
                      :namespace "test-io",
                      :name "etc-nginx"}}
          (dissoc (cut/generate-nginx-configmap {:issuer "staging"
@@ -99,15 +99,15 @@
                                                 :fqdns ["test.de" "www.test.de" "test-it.de" "www.test-it.de"]}) :data))))
 
 (deftest should-generate-nginx-service
-  (is (= {:name-c1 "test-io",
-          :name-c2 "test-org",
-          :app-c1 "test-io",
-          :app-c2 "test-org",
-          :app.kubernetes.part-of-c1 "test-io",
-          :app.kubernetes.part-of-c2 "test-org"
-          :namespace-c1 "test-io", 
-          :namespace-c2 "test-org"}
-         (th/map-diff (cut/generate-nginx-service {:issuer "staging"
+  (is (= {:kind "Service",
+          :apiVersion "v1",
+          :metadata
+          {:name "test-io",
+           :namespace "test-io",
+           :labels {:app "test-io", :app.kubernetes.part-of "test-io-website"}},
+          :spec
+          {:selector {:app "nginx"}, :ports [{:name "nginx-http", :port 80}]}}
+         (cut/generate-nginx-service {:issuer "staging"
                                                    :build-cpu-request "500m"
                                                    :build-cpu-limit "1700m"
                                                    :build-memory-request "256Mi"
@@ -118,17 +118,7 @@
                                                    :forgejo-repo "none",
                                                    :branchname "mablain",
                                                    :fqdns ["test.de" "www.test.de" "test-it.de" "www.test-it.de"]})
-                      (cut/generate-nginx-service {:issuer "staging"
-                                                   :build-cpu-request "500m"
-                                                   :build-cpu-limit "1700m"
-                                                   :build-memory-request "256Mi"
-                                                   :build-memory-limit "512Mi"
-                                                   :volume-size "3"
-                                                   :unique-name "test.org",
-                                                   :forgejo-host "gitea.evilorg",
-                                                   :forgejo-repo "none",
-                                                   :branchname "mablain",
-                                                   :fqdns ["test.de" "www.test.de" "test-it.de" "www.test-it.de"]})))))
+)))
 
 (deftest should-generate-website-build-cron
   (is (= {:apiVersion "batch/v1",