feat(it-tools): Change service port to 8080 (#8)

* fix(actions): Add NET_BIND_SERVICE capability

* test(it-tools): Remove capabilities options

* feat(it-tools): Removed liveness & readiness probes

* feat(actions): Update chart testing workflow

* test(actions): Re-add probes & test security context

* feat(it-tools): Changed service port to 8080

* feat(doc): Be more specific in the changelog

.github/cr-config.yaml

@@ -0,0 +1,2 @@
+generate-release-notes: true
+skip-existing: true

.github/ct-install-config.yaml

@@ -0,0 +1,4 @@
+debug: true
+all: true
+chart-repos:
+  - bjw-s=https://bjw-s.github.io/helm-charts

.github/ct-lint-config.yaml

@@ -0,0 +1,5 @@
+debug: true
+all: true
+validate-maintainers: false
+chart-repos:
+  - bjw-s=https://bjw-s.github.io/helm-charts

.github/workflows/lint-test.yaml

@@ -0,0 +1,56 @@
+name: Linting and Testing
+
+on:
+  push:
+    branches: [master]
+    paths:
+      - "charts/**"
+      - ".github/workflows/**"
+  pull_request:
+    branches: [master]
+    paths:
+      - "charts/**"
+  workflow_dispatch:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Set up chart-testing
+        uses: helm/chart-testing-action@v2.6.1
+
+      - name: Run chart-testing (lint)
+        run: ct lint --config .github/ct-lint-config.yaml
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Install Helm
+        uses: azure/setup-helm@v4
+
+      - name: Set up chart-testing
+        uses: helm/chart-testing-action@v2.6.1
+
+      - name: Run chart-testing (list-changed)
+        id: list-changed
+        run: |
+          changed=$(ct list-changed --chart-dirs charts --target-branch ${{ github.event.repository.default_branch }})
+          if [[ -n "$changed" ]]; then
+            echo "changed=true" >> $GITHUB_OUTPUT
+          fi
+
+      - name: Create kind cluster
+        if: steps.list-changed.outputs.changed == 'true'
+        uses: helm/kind-action@v1.10.0
+
+      - name: Run chart-testing (install)
+        if: steps.list-changed.outputs.changed == 'true'
+        run: ct install --config .github/ct-install-config.yaml

.github/workflows/release.yaml

@@ -1,4 +1,4 @@
-name: Release Charts
+name: Release
 
 on:
   workflow_dispatch:
@@ -44,9 +44,7 @@ jobs:
           git config user.email "${{ env.AUTHOR_EMAIL }}"
 
       - name: Add Helm repos
-        run: |
+        run: helm repo add bjw-s https://bjw-s.github.io/helm-charts
-          helm repo add bitnami https://charts.bitnami.com/bitnami
-          helm repo add bjw-s https://bjw-s.github.io/helm-charts
 
       - name: Install chart-releaser
         uses: helm/chart-releaser-action@v1.5.0
@@ -91,6 +89,7 @@ jobs:
         uses: helm/chart-releaser-action@v1.5.0
         if: steps.package_charts.outputs.changed_charts != ''
         with:
+          config: .github/cr-config.yaml
           skip_packaging: true
         env:
           CR_TOKEN: ${{ steps.app-token.outputs.token }}

README.md

@@ -3,9 +3,10 @@
 </p>
 
 <p align="center">
-    <a href="./LICENSE"><img src="https://img.shields.io/github/license/plcnk/charts" /></a>
+    <a href="https://github.com/plcnk/charts/blob/master/LICENSE"><img src="https://img.shields.io/github/license/plcnk/charts" /></a>
     <a href="https://artifacthub.io/packages/search?repo=plcnk"><img src="https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/plcnk" /></a>
     <a href="https://github.com/plcnk/charts/actions/workflows/release.yaml"><img src="https://github.com/plcnk/charts/actions/workflows/release.yaml/badge.svg" /></a>
+    <a href="https://github.com/plcnk/charts/actions/workflows/lint-test.yaml"><img src="https://github.com/plcnk/charts/actions/workflows/lint-test.yaml/badge.svg" /></a>
 </p>
 
 # Charts
@@ -22,7 +23,7 @@ The code in this repository is provided as-is with no warranties.
 
 | Chart | Description |
 | ----- | ----------- |
-| [# it-tools <img src='https://raw.githubusercontent.com/plcnk/charts/master/charts/it-tools/icon.svg' alt='it-tools icon' width='18px' align='right' loading='lazy'>](charts/it-tools/) | Collection of handy online tools for developers, with great UX. |
+| [# it-tools <img src='https://raw.githubusercontent.com/plcnk/charts/master/charts/it-tools/icon.svg' alt='it-tools icon' width='18px' align='right' loading='lazy'>](https://github.com/plcnk/charts/tree/master/charts/it-tools/) | Collection of handy online tools for developers, with great UX. |
 
 ## License
 

charts/it-tools/Chart.yaml

@@ -4,7 +4,7 @@ description: Collection of handy online tools for developers, with great UX.
 home: https://github.com/plcnk/charts/tree/master/charts/it-tools
 icon: https://raw.githubusercontent.com/plcnk/charts/master/charts/it-tools/icon.svg
 type: application
-version: 2.0.0
+version: 2.1.0
 # renovate datasource=docker depName=ghcr.io/corentinth/it-tools
 appVersion: "2024.5.13-a0bc346"
 kubeVersion: ">=1.22.0-0"
@@ -21,7 +21,7 @@ sources:
 annotations:
   artifacthub.io/changes: |-
     - kind: changed
-      description: Major chart overhaul. Now using bjw-s's common library chart.
+      description: Changed service port from 80 to 8080
   artifacthub.io/links: |-
     - name: App Source
       url: https://github.com/CorentinTh/it-tools

charts/it-tools/README.md

@@ -2,7 +2,7 @@
 
 <img src="https://raw.githubusercontent.com/plcnk/charts/master/charts/it-tools/icon.svg" align="right" width="92" alt="it-tools logo">
 
-![Version: 2.0.0](https://img.shields.io/badge/Version-2.0.0-informational?style=flat)
+![Version: 2.1.0](https://img.shields.io/badge/Version-2.1.0-informational?style=flat)
 ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat)
 ![AppVersion: 2024.5.13-a0bc346](https://img.shields.io/badge/AppVersion-2024.5.13--a0bc346-informational?style=flat)
 
@@ -11,7 +11,7 @@ Collection of handy online tools for developers, with great UX.
 **Homepage:** <https://github.com/plcnk/charts/tree/master/charts/it-tools>
 
 **This chart is not maintained by the upstream project and any issues with the chart should be raised
-[here](https://github.com/plcnk/charts/issues/new?assignees=plcnk&labels=bug&template=bug_report.yaml&name=it-tools&version=2.0.0)**
+[here](https://github.com/plcnk/charts/issues/new?assignees=plcnk&labels=bug&template=bug_report.yaml&name=it-tools&version=2.1.0)**
 
 ## Source Code
 

charts/it-tools/templates/common.yaml

@@ -18,7 +18,7 @@ configMaps:
             include /etc/nginx/mime.types;
         
             server {
-                listen 80;
+                listen 8080;
                 server_name _;
                 root /usr/share/nginx/html;
                 index index.html;
@@ -59,7 +59,7 @@ controllers:
     containers:
       app:
         ports:
-          - containerPort: 80
+          - containerPort: 8080
             name: http
             protocol: TCP
         probes:

charts/it-tools/values.yaml

@@ -62,7 +62,7 @@ service:
     controller: main
     ports:
       http:
-        port: 80
+        port: 8080
 
 # -- Configure persistence for the chart under this key.
 # @default -- See [values.yaml](./values.yaml)

hack/gen-chart-summary.sh

@@ -34,6 +34,6 @@ sed -i '' '/^'"$charts_header"'$/,$d' "$charts_summary_file"
       chart_icon="<img src='$chart_icon' alt='$chart_name icon' width='${width}px' align='right' loading='lazy'>"
     fi
     pretty_name="$(head -n1 "$(dirname "$chart_yaml")/README.md" | sed 's/^# //')"
-    echo "| [$pretty_name $chart_icon](charts/$chart_name/) | $chart_description |"
+    echo "| [$pretty_name $chart_icon](https://github.com/plcnk/charts/tree/master/charts/$chart_name/) | $chart_description |"
   done
 } >> "$charts_summary_file"