Skip to content

Commit

Permalink
Bug Fix & Minor Update (BorisPolonsky#46)
Browse files Browse the repository at this point in the history
* Add Release chart action & bump chart version  (BorisPolonsky#3)

* Add helm chart release logic
* Bump chart version

* Add installation guide (BorisPolonsky#4)

* Fix env clash (BorisPolonsky#9)

* Fix ENV clashing for some corner case

* Bump chart version to 0.14.1

* Fix label in worker deployment (BorisPolonsky#12)

* Bump chart version to 0.15.0

* 0.3.8 compatibility for core components (BorisPolonsky#14)

- Fix compatibility issue with URL related ENV introduced in `0.3.8` 
  - langgenius/dify#648
  - BorisPolonsky#11

* Bump chart version to 0.16.0

* Fix postgres port and database config (BorisPolonsky#18)

- Fix port configuration when external postgres exposes port other than 5432
- Fix database conofiguration when utilizing external postgres

* Update Chart.yaml

Bump version to 0.16.1

* Update README.md

Fix grammar

* Create LICENSE

* Update README.md

Add download badge

* Update README.md

* Dify 0.4.9 support (BorisPolonsky#28)

* Add new environment variables to templates

* Update nginx routing

* Remove redundant environment vairables

* Update milvus config

* Update URL setting

* Quote environment variables

* Update README.md

* Bump chart version to 0.17.0

* Update mail config

* Remove redundant extraEnv

* Bump appVersion to 0.4.9

* Update README.md (BorisPolonsky#31)

* Revert "Update README.md" (BorisPolonsky#32)

* Display contributors (BorisPolonsky#33)

* Fix README.md (BorisPolonsky#26)

* Add Release chart action & bump chart version  (BorisPolonsky#3)

* Add helm chart release logic
* Bump chart version

* Add installation guide (BorisPolonsky#4)

* Fix env clash (BorisPolonsky#9)

* Fix ENV clashing for some corner case

* Bump chart version to 0.14.1

* Fix label in worker deployment (BorisPolonsky#12)

* Bump chart version to 0.15.0

* 0.3.8 compatibility for core components (BorisPolonsky#14)

- Fix compatibility issue with URL related ENV introduced in `0.3.8` 
  - langgenius/dify#648
  - BorisPolonsky#11

* Bump chart version to 0.16.0

* Fix postgres port and database config (BorisPolonsky#18)

- Fix port configuration when external postgres exposes port other than 5432
- Fix database conofiguration when utilizing external postgres

* Update Chart.yaml

Bump version to 0.16.1

* Update README.md

Fix grammar

* Create LICENSE

* Update README.md

Add download badge

* Update README.md

* Dify 0.4.9 Compatibility (BorisPolonsky#30)

* Add Release chart action & bump chart version  (BorisPolonsky#3)

* Add helm chart release logic
* Bump chart version

* Add installation guide (BorisPolonsky#4)

* Fix env clash (BorisPolonsky#9)

* Fix ENV clashing for some corner case

* Bump chart version to 0.14.1

* Fix label in worker deployment (BorisPolonsky#12)

* Bump chart version to 0.15.0

* 0.3.8 compatibility for core components (BorisPolonsky#14)

- Fix compatibility issue with URL related ENV introduced in `0.3.8` 
  - langgenius/dify#648
  - BorisPolonsky#11

* Bump chart version to 0.16.0

* Fix postgres port and database config (BorisPolonsky#18)

- Fix port configuration when external postgres exposes port other than 5432
- Fix database conofiguration when utilizing external postgres

* Update Chart.yaml

Bump version to 0.16.1

* Update README.md

Fix grammar

* Create LICENSE

* Update README.md

Add download badge

* Update README.md

* Dify 0.4.9 support (BorisPolonsky#28)

* Add new environment variables to templates

* Update nginx routing

* Remove redundant environment vairables

* Update milvus config

* Update URL setting

* Quote environment variables

* Update README.md

* Bump chart version to 0.17.0

* Update mail config

* Remove redundant extraEnv

* Bump appVersion to 0.4.9

* feat: add files path to nginx proxy config (BorisPolonsky#29)

Resolves the issue of files not being available in the frontend, e.g. DALL-E generated images.

Co-authored-by: Boris Polonsky <BorisPolonsky@users.noreply.github.com>

* Update README.md

---------

Co-authored-by: Pascal M <11357019+perzeuss@users.noreply.github.com>

* Update release condition

* Update values.yaml

Update comments on vector db in values.yaml

* Update values.yaml

Grammar fix

* Fix ConfigMap error when `.Values.externalMilvus.enable=true`

* Fix nginx duplicate location: "/files"

* Bump Chart version to 0.17.1

* Optimize extraEnv configuration

* Bump Chart Version to 0.17.2

* Change chart version to 0.17.2-rc1

* Bump chart version to 0.17.2

* add quote to QDRANT_CLIENT_TIMEOUT/consoleApi/appApi BorisPolonsky#45

lack of quote cause creating configmap error

* Rename port in service as name: `<protocol>[-<suffix>]` format and make them compatible with explicit protocol selection in `istio`

---------

Co-authored-by: Pascal M <11357019+perzeuss@users.noreply.github.com>
Co-authored-by: Ryan <haoinit@gmail.com>
  • Loading branch information
3 people authored Mar 22, 2024
1 parent e4a8d93 commit 3b36e0e
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion charts/dify/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ type: application
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)

version: 0.17.2
version: 0.18.0

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
2 changes: 1 addition & 1 deletion charts/dify/templates/api-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ spec:
clusterIP: {{ .Values.api.service.clusterIP }}
{{- end }}
ports:
- name: api
- name: http-api
port: {{ .Values.api.service.port }}
protocol: TCP
targetPort: api
Expand Down
6 changes: 3 additions & 3 deletions charts/dify/templates/config.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,11 @@ SECRET_KEY: {{ .Values.api.secretKey }}
# The base URL of console application api server, refers to the Console base URL of WEB service if console domain is
# different from api or web app domain.
# example: http://cloud.dify.ai
CONSOLE_API_URL: {{ .Values.api.url.consoleApi }}
CONSOLE_API_URL: {{ .Values.api.url.consoleApi | quote }}
# The URL for Web APP api server, refers to the Web App base URL of WEB service if web app domain is different from
# console or api domain.
# example: http://udify.app
APP_API_URL: {{ .Values.api.url.appApi }}
APP_API_URL: {{ .Values.api.url.appApi | quote }}
# The DSN for Sentry
{{- end }}

Expand Down Expand Up @@ -205,7 +205,7 @@ QDRANT_URL: {{ .Values.externalQdrant.endpoint }}
# The Qdrant API key.
QDRANT_API_KEY: {{ .Values.externalQdrant.apiKey }}
# The Qdrant clinet timeout setting.
QDRANT_CLIENT_TIMEOUT: 20
QDRANT_CLIENT_TIMEOUT: "20"
# The DSN for Sentry error reporting. If not set, Sentry error reporting will be disabled.
{{- else if .Values.externalMilvus.enabled}}
# Milvus configuration Only available when VECTOR_STORE is `milvus`.
Expand Down
2 changes: 1 addition & 1 deletion charts/dify/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ spec:
type: {{ .Values.service.type }}
{{- end }}
ports:
- name: dify
- name: http-dify
port: {{ .Values.service.port }}
protocol: TCP
targetPort: dify
Expand Down
2 changes: 1 addition & 1 deletion charts/dify/templates/web-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ spec:
clusterIP: {{ .Values.web.service.clusterIP }}
{{- end }}
ports:
- name: web
- name: http-web
port: {{ .Values.web.service.port }}
protocol: TCP
targetPort: web
Expand Down

0 comments on commit 3b36e0e

Please sign in to comment.