dify-helm icon indicating copy to clipboard operation
dify-helm copied to clipboard

[BUG] Error when setting externalAzureBlobStorage to true

Open Onoshun-HWS opened this issue 1 year ago • 0 comments

This is my first time reporting a bug on github, so please forgive the poor quality.

Describe the bug Setting the next value to true will result in an error.

externalAzureBlobStorage:
  enabled: true

Because there is no condition for this. .Values.externalAzureBlobStorage.enabled

dify-helm / charts / dify / templates/ api-deployment

volumeMounts:
        {{- if not .Values.externalS3.enabled }}
        - name: app-data

I believe that these conditions will resolve the error.

volumeMounts:
        {{- if not (or .Values.externalS3.enabled .Values.externalAzureBlobStorage.enabled) }}
        - name: app-data

To Reproduce Steps to reproduce the behavior:

  1. externalAzureBlobStorage: enabled: true
  2. helm upgrade dify dify/dify -f values.yaml
  3. Error: UPGRADE FAILED: cannot patch "dify-api" with kind Deployment: Deployment.apps "dify-api" is invalid: spec.template.spec.containers[0].volumeMounts[0].name: Not found: "app-data" && cannot patch "dify-worker" with kind Deployment: Deployment.apps "dify-worker" is invalid: spec.template.spec.containers[0].volumeMounts[0].name: Not found: "app-data"

Desktop (please complete the following information):

  • OS: [e.g. ubuntu 22.04 on WSL2]
  • Helm version: version.BuildInfo{Version:"v3.14.2", GitCommit:"", GitTreeState:"clean", GoVersion:"go1.21.6"}
  • Dify version:
    • CHART VERSION: 0.20.0
    • APP VERSION: 0.6.3

Onoshun-HWS avatar May 14 '24 12:05 Onoshun-HWS