From c98983c3cc0de2524ee4ac8e4772be3f012fbc0f Mon Sep 17 00:00:00 2001 From: Alec Thomas <112640918+a-thomas-22@users.noreply.github.com> Date: Thu, 7 Nov 2024 12:23:44 -0600 Subject: [PATCH 1/4] Add das updatestrategy --- charts/das/Chart.yaml | 2 +- charts/das/templates/statefulset.yaml | 6 ++++++ charts/das/values.yaml | 4 ++++ 3 files changed, 11 insertions(+), 1 deletion(-) diff --git a/charts/das/Chart.yaml b/charts/das/Chart.yaml index 0a4c5c7..53d7156 100644 --- a/charts/das/Chart.yaml +++ b/charts/das/Chart.yaml @@ -7,6 +7,6 @@ maintainers: type: application -version: 0.5.11 +version: 0.5.12 appVersion: "v3.2.1-d81324d" diff --git a/charts/das/templates/statefulset.yaml b/charts/das/templates/statefulset.yaml index 34053a7..4291250 100644 --- a/charts/das/templates/statefulset.yaml +++ b/charts/das/templates/statefulset.yaml @@ -11,6 +11,12 @@ spec: matchLabels: {{- include "das.selectorLabels" . | nindent 6 }} podManagementPolicy: Parallel + {{- if .Values.updateStrategy }} + {{- with .Values.updateStrategy }} + updateStrategy: + {{- toYaml . | nindent 4 }} + {{- end }} + {{- end }} template: metadata: annotations: diff --git a/charts/das/values.yaml b/charts/das/values.yaml index dbb3dde..6e05cc7 100644 --- a/charts/das/values.yaml +++ b/charts/das/values.yaml @@ -65,6 +65,10 @@ readinessProbe: startupProbe: enabled: false +## @param updateStrategy.type Update strategy type +updateStrategy: + type: RollingUpdate + persistence: ## @extra persistence.localdbstorage This will only be created if local db storage is enabled in the configmap ## @param persistence.localdbstorage.size Size of the persistent volume claim From 6e5dddded5e3054491f2adb5bb17df45c5398794 Mon Sep 17 00:00:00 2001 From: a-thomas-22 Date: Thu, 7 Nov 2024 18:25:41 +0000 Subject: [PATCH 2/4] Update README and values.schema.json for modified charts --- charts/das/README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/charts/das/README.md b/charts/das/README.md index 5b380fb..0ad9ccb 100644 --- a/charts/das/README.md +++ b/charts/das/README.md @@ -185,6 +185,7 @@ extraEnv: | `readinessProbe.successThreshold` | Success threshold for the readiness probe | `1` | | `startupProbe` | startupProbe | | | `startupProbe.enabled` | Enable startup probe | `false` | +| `updateStrategy.type` | Update strategy type | `RollingUpdate` | | `persistence.localdbstorage` | This will only be created if local db storage is enabled in the configmap | | | `persistence.localdbstorage.size` | Size of the persistent volume claim | `100Gi` | | `persistence.localdbstorage.storageClassName` | Storage class of the persistent volume claim | `nil` | From 40bf49e7c627fd9f99e955e2ad4efb17f093e00d Mon Sep 17 00:00:00 2001 From: Alec Thomas <112640918+a-thomas-22@users.noreply.github.com> Date: Thu, 7 Nov 2024 12:30:45 -0600 Subject: [PATCH 3/4] update runner name --- .github/workflows/generate-chart-readme.yaml | 2 +- .github/workflows/lint.yaml | 2 +- .github/workflows/push-chart-dev.yaml | 2 +- .github/workflows/push-chart.yaml | 4 ++-- .github/workflows/test-install.yaml | 2 +- .github/workflows/test-upgrade.yaml | 2 +- 6 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.github/workflows/generate-chart-readme.yaml b/.github/workflows/generate-chart-readme.yaml index dcd5d3b..8cbf3e4 100644 --- a/.github/workflows/generate-chart-readme.yaml +++ b/.github/workflows/generate-chart-readme.yaml @@ -11,7 +11,7 @@ permissions: {} # Remove all permissions by default jobs: update-readme-metadata: - runs-on: ubuntu-latest + runs-on: ubuntu-8 permissions: contents: write steps: diff --git a/.github/workflows/lint.yaml b/.github/workflows/lint.yaml index 5f17d16..8c099d1 100644 --- a/.github/workflows/lint.yaml +++ b/.github/workflows/lint.yaml @@ -7,7 +7,7 @@ on: jobs: lint: - runs-on: ubuntu-latest + runs-on: ubuntu-8 steps: - name: Checkout uses: actions/checkout@v4 diff --git a/.github/workflows/push-chart-dev.yaml b/.github/workflows/push-chart-dev.yaml index dd47216..b39d822 100644 --- a/.github/workflows/push-chart-dev.yaml +++ b/.github/workflows/push-chart-dev.yaml @@ -16,7 +16,7 @@ on: jobs: push-charts: - runs-on: ubuntu-latest + runs-on: ubuntu-8 steps: - name: Checkout code uses: actions/checkout@master diff --git a/.github/workflows/push-chart.yaml b/.github/workflows/push-chart.yaml index 4bffda1..298780e 100644 --- a/.github/workflows/push-chart.yaml +++ b/.github/workflows/push-chart.yaml @@ -8,7 +8,7 @@ on: jobs: get-modified-charts: - runs-on: ubuntu-latest + runs-on: ubuntu-8 outputs: paths: ${{ steps.get-modified-charts.outputs.paths }} steps: @@ -24,7 +24,7 @@ jobs: push-charts: needs: get-modified-charts - runs-on: ubuntu-latest + runs-on: ubuntu-8 if: ${{ needs.get-modified-charts.outputs.paths != '' }} steps: - uses: actions/checkout@master diff --git a/.github/workflows/test-install.yaml b/.github/workflows/test-install.yaml index c155a65..2838c8b 100644 --- a/.github/workflows/test-install.yaml +++ b/.github/workflows/test-install.yaml @@ -7,7 +7,7 @@ on: jobs: test-install: - runs-on: ubuntu-latest + runs-on: ubuntu-8 steps: - name: Checkout diff --git a/.github/workflows/test-upgrade.yaml b/.github/workflows/test-upgrade.yaml index 1a11f72..8115360 100644 --- a/.github/workflows/test-upgrade.yaml +++ b/.github/workflows/test-upgrade.yaml @@ -7,7 +7,7 @@ on: jobs: test-upgrade: - runs-on: ubuntu-latest + runs-on: ubuntu-8 steps: - name: Checkout From 8197a218492b76bc62caa5fefeaaad574d67dad2 Mon Sep 17 00:00:00 2001 From: Alec Thomas <112640918+a-thomas-22@users.noreply.github.com> Date: Thu, 7 Nov 2024 12:31:46 -0600 Subject: [PATCH 4/4] u --- .github/workflows/generate-chart-readme.yaml | 2 +- .github/workflows/lint.yaml | 2 +- .github/workflows/push-chart-dev.yaml | 2 +- .github/workflows/push-chart.yaml | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/generate-chart-readme.yaml b/.github/workflows/generate-chart-readme.yaml index 8cbf3e4..13617a4 100644 --- a/.github/workflows/generate-chart-readme.yaml +++ b/.github/workflows/generate-chart-readme.yaml @@ -11,7 +11,7 @@ permissions: {} # Remove all permissions by default jobs: update-readme-metadata: - runs-on: ubuntu-8 + runs-on: ubuntu-4 permissions: contents: write steps: diff --git a/.github/workflows/lint.yaml b/.github/workflows/lint.yaml index 8c099d1..f4aa6c5 100644 --- a/.github/workflows/lint.yaml +++ b/.github/workflows/lint.yaml @@ -7,7 +7,7 @@ on: jobs: lint: - runs-on: ubuntu-8 + runs-on: ubuntu-4 steps: - name: Checkout uses: actions/checkout@v4 diff --git a/.github/workflows/push-chart-dev.yaml b/.github/workflows/push-chart-dev.yaml index b39d822..7d3f600 100644 --- a/.github/workflows/push-chart-dev.yaml +++ b/.github/workflows/push-chart-dev.yaml @@ -16,7 +16,7 @@ on: jobs: push-charts: - runs-on: ubuntu-8 + runs-on: ubuntu-4 steps: - name: Checkout code uses: actions/checkout@master diff --git a/.github/workflows/push-chart.yaml b/.github/workflows/push-chart.yaml index 298780e..bec33af 100644 --- a/.github/workflows/push-chart.yaml +++ b/.github/workflows/push-chart.yaml @@ -8,7 +8,7 @@ on: jobs: get-modified-charts: - runs-on: ubuntu-8 + runs-on: ubuntu-4 outputs: paths: ${{ steps.get-modified-charts.outputs.paths }} steps: @@ -24,7 +24,7 @@ jobs: push-charts: needs: get-modified-charts - runs-on: ubuntu-8 + runs-on: ubuntu-4 if: ${{ needs.get-modified-charts.outputs.paths != '' }} steps: - uses: actions/checkout@master