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

Patch Milvus & Nginx Configuration Error in 0.17.0 #39

Merged
merged 32 commits into from
Mar 4, 2024
Merged

Conversation

BorisPolonsky
Copy link
Owner

  • Fix ConfigMap error when .Values.externalMilvus.enable=true
  • Fix a bug where nginx fails to boot up (when .Values.proxy.enable=true) due to configuration error that caused by incorrect merge request operations

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 <[email protected]>

* Update README.md

---------

Co-authored-by: Pascal M <[email protected]>
Update comments on vector db in values.yaml
Grammar fix
@BorisPolonsky BorisPolonsky merged commit 52cb27c into master Mar 4, 2024
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.

1 participant