Merge pull request #6 from adamrushuk/add-letsencrypt

Removed storageClassName
adamrushuk · Sep 7, 2020 · 0c0410d · 0c0410d
2 parents 81a8919 + ec5083b
commit 0c0410d
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 5 deletions.
diff --git a/.github/ct.yaml b/.github/ct.yaml
@@ -1,5 +1,5 @@
 # See https://github.com/helm/chart-testing#configuration
-debug: true
+debug: false
 remote: origin
 target-branch: main
 chart-dirs:

diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -7,7 +7,7 @@ on:
     inputs: {}
 
 jobs:
-  lint-test:
+  test:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout

diff --git a/charts/sonatype-nexus/Chart.yaml b/charts/sonatype-nexus/Chart.yaml
@@ -3,7 +3,7 @@ name: sonatype-nexus
 description: Sonatype Nexus is an open source repository manager
 type: application
 # This is the chart version
-version: 0.2.1
+version: 0.2.2
 # This is the version number of the application being deployed
 appVersion: 3.27.0
 keywords:

diff --git a/charts/sonatype-nexus/templates/statefulset.yaml b/charts/sonatype-nexus/templates/statefulset.yaml
@@ -77,7 +77,8 @@ spec:
       name: {{ template "sonatype-nexus.fullname" . }}-data
     spec:
       accessModes: [ "ReadWriteOnce" ]
-      storageClassName: ""
+      # GitHub Action helm/kind-action uses: storageClassName: "standard"
+      # storageClassName: ""
       resources:
         requests:
           # minimum is 4Gi, but 8Gi recommended during testing

diff --git a/charts/sonatype-nexus/values.yaml b/charts/sonatype-nexus/values.yaml
@@ -6,7 +6,7 @@ nexus:
   baseDomain: domain.com
   # certEmail is a valid email address used by Let's Encrypt. It does not have to be at the baseDomain.
   certEmail: certadmin@domain.com
-  storageSize: 10Gi
+  storageSize: 8Gi
   # The ports should only be changed if the nexus image uses a different port
   dockerPort: 5000
   nexusPort: 8081