From 64c240c7c8743f61d58df86b8d9b37b1a92991ba Mon Sep 17 00:00:00 2001
From: dave-juicelabs <70105130+dave-juicelabs@users.noreply.github.com>
Date: Thu, 27 Jul 2023 19:47:02 -0500
Subject: [PATCH] Add port names and run controller action only when changes
 occur to the controller (#44)

---
 .github/workflows/controller-docker-image.yml | 4 ++++
 charts/controller/Chart.yaml                  | 2 +-
 charts/controller/templates/service.yaml      | 6 ++++--
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/controller-docker-image.yml b/.github/workflows/controller-docker-image.yml
index 75e86da..b4614e5 100644
--- a/.github/workflows/controller-docker-image.yml
+++ b/.github/workflows/controller-docker-image.yml
@@ -4,6 +4,10 @@ on:
   push:
     branches:
     - master
+    paths:
+    - cmd/controller/**
+    - docker/controller/**
+    - pkg/**
 
 jobs:
   docker:
diff --git a/charts/controller/Chart.yaml b/charts/controller/Chart.yaml
index 80c63cb..b9b4c9f 100644
--- a/charts/controller/Chart.yaml
+++ b/charts/controller/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: juice-controller
-version: 1.0.3
+version: 1.0.4
 type: application
 home: https://github.com/Juice-Labs/Juice-Labs
 icon: https://raw.githubusercontent.com/Juice-Labs/Juice-Labs/master/assets/Juice.svg
diff --git a/charts/controller/templates/service.yaml b/charts/controller/templates/service.yaml
index 5f4ccd1..1b82cb0 100644
--- a/charts/controller/templates/service.yaml
+++ b/charts/controller/templates/service.yaml
@@ -12,11 +12,13 @@ spec:
   selector:
     app.juicelabs.co/controller: frontend
   ports:
-    - protocol: TCP
+    - name: controller
+      protocol: TCP
       port: {{ .Values.service.port }}
       targetPort: controller
 {{- if .Values.prometheus }}
-    - protocol: TCP
+    - name: prometheus
+      protocol: TCP
       port: {{ .Values.prometheus.port }}
       targetPort: prometheus
 {{- end }}
-- 
GitLab