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

feat: add aliyun oss storage #3690

Merged
merged 3 commits into from
Apr 25, 2024
Merged

feat: add aliyun oss storage #3690

merged 3 commits into from
Apr 25, 2024

Conversation

henrybit
Copy link
Contributor

Description

Use Aliyun oss to store files, need oss2>=2.15.0 library.

Type of Change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  1. use my Aliyun ram account's token(AK/SK), create a private read/write bucket, grant full control permission to my ram account;
  2. start API & Web server locally, use Milvus/Redis/Postgresql in My Environment;
  3. create a knowledge repository, and upload some PDFs.

results: Upload success/Content vector successfully obtained/Chat success in RAG.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. 📚 feat:datasource Data sources like web, Notion, Logseq, Lark, Docs labels Apr 23, 2024
@takatost takatost requested a review from JohnJyong April 23, 2024 05:04
@langgenius langgenius deleted a comment from JohnJyong Apr 24, 2024
@takatost
Copy link
Collaborator

There is a lint issue that did not pass, please fix it.

@JohnJyong
Copy link
Contributor

@henrybit

@henrybit
Copy link
Contributor Author

There is a lint issue that did not pass, please fix it.

image

fix this lint issue.

@henrybit
Copy link
Contributor Author

@henrybit

finished.

@JohnJyong
Copy link
Contributor

thanks for your contribution~

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 25, 2024
@JohnJyong JohnJyong merged commit 80b9507 into langgenius:main Apr 25, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 feat:datasource Data sources like web, Notion, Logseq, Lark, Docs lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants