Skip to content

feat: add local storage driver#10

Merged
nuclearcat merged 1 commit intokernelci:mainfrom
nuclearcat:add-local-storage
Sep 1, 2025
Merged

feat: add local storage driver#10
nuclearcat merged 1 commit intokernelci:mainfrom
nuclearcat:add-local-storage

Conversation

@nuclearcat
Copy link
Copy Markdown
Member

This commit introduces a new local storage driver to the application. This allows users to store files and metadata on the local filesystem instead of relying on cloud-based storage solutions like Azure.

The local storage driver provides the following features:

  • File storage: Files are stored in a configurable local directory.
  • Metadata storage: File metadata, including content type and user-defined tags, is stored in a separate .metadata directory.
  • Directory listing: The driver can list files and directories within the storage path.
  • Configuration: The local storage path can be configured in the config.toml file.

This change improves the flexibility and versatility of the application, making it easier for users to deploy and test it in different environments, especially self-hosted setup.

This commit introduces a new local storage driver to the application.
This allows users to store files and metadata on the local filesystem
instead of relying on cloud-based storage solutions like Azure.

The local storage driver provides the following features:

- File storage: Files are stored in a configurable local directory.
- Metadata storage: File metadata, including content type and user-defined tags, is stored in a separate .metadata directory.
- Directory listing: The driver can list files and directories within the storage path.
- Configuration: The local storage path can be configured in the config.toml file.

This change improves the flexibility and versatility of the application,
making it easier for users to deploy and test it in different environments,
especially self-hosted setup.

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
Copy link
Copy Markdown

@tales-aparecida tales-aparecida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but I'm not rusty enough 😅
Don't know much about this repo. Pls check if you need to update the README whatsoever

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