Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug Fix & Minor Update #46

Merged
merged 42 commits into from
Mar 22, 2024
Merged

Bug Fix & Minor Update #46

merged 42 commits into from
Mar 22, 2024

Conversation

BorisPolonsky
Copy link
Owner

@BorisPolonsky BorisPolonsky commented Mar 22, 2024

  • Fix qdrant configuration bug @kaktos
  • Rename ports to make them istio-compatible

BorisPolonsky and others added 30 commits June 17, 2023 11:21
* Add helm chart release logic
* Bump chart version
* Fix ENV clashing for some corner case
- Fix compatibility issue with URL related ENV introduced in `0.3.8` 
  - langgenius/dify#648
  - #11
- Fix port configuration when external postgres exposes port other than 5432
- Fix database conofiguration when utilizing external postgres
Bump version to 0.16.1
Fix grammar
Add download badge
* 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
* Fix README.md (#26)

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

* Add helm chart release logic
* Bump chart version

* Add installation guide (#4)

* Fix env clash (#9)

* Fix ENV clashing for some corner case

* Bump chart version to 0.14.1

* Fix label in worker deployment (#12)

* Bump chart version to 0.15.0

* 0.3.8 compatibility for core components (#14)

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

* Bump chart version to 0.16.0

* Fix postgres port and database config (#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 (#30)

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

* Add helm chart release logic
* Bump chart version

* Add installation guide (#4)

* Fix env clash (#9)

* Fix ENV clashing for some corner case

* Bump chart version to 0.14.1

* Fix label in worker deployment (#12)

* Bump chart version to 0.15.0

* 0.3.8 compatibility for core components (#14)

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

* Bump chart version to 0.16.0

* Fix postgres port and database config (#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 (#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 (#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 comments on vector db in values.yaml
Grammar fix
@BorisPolonsky BorisPolonsky merged commit 3778d88 into master Mar 22, 2024
regnull pushed a commit to regnull/dify-helm that referenced this pull request Dec 30, 2024
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants