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

[full-ci] Improve ini backend, support redis sentinel caches #5737

Merged
merged 10 commits into from
Mar 6, 2023

Conversation

aduffeck
Copy link
Contributor

@aduffeck aduffeck commented Mar 6, 2023

This PR pulls in the latest reva edge which improves the ini backend for file metadata:

  • Improve performance
  • Optionally use a reva cache for storing the metadata, which helps tremendously when using distributed file systems, for example
  • Allow for using different metadata backends for different storages
  • We also switched the s3ng integration tests to the ini backend so we cover both the xattrs and the ini backend at the same time.

Finally it adds support for redis sentinel caches.

Fixes #5645

@update-docs
Copy link

update-docs bot commented Mar 6, 2023

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

aduffeck and others added 4 commits March 6, 2023 12:07
Co-authored-by: Martin <github@diemattels.at>
Co-authored-by: Martin <github@diemattels.at>
Co-authored-by: Jörn Friedrich Dreyer <jfd@butonic.de>
Co-authored-by: Martin <github@diemattels.at>
@sonarcloud
Copy link

sonarcloud bot commented Mar 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@butonic butonic merged commit 8b68830 into owncloud:master Mar 6, 2023
@butonic butonic deleted the filemetadatacache branch March 6, 2023 14:04
ownclouders pushed a commit that referenced this pull request Mar 6, 2023
Author: Andre Duffeck <aduffeck@users.noreply.github.com>
Date:   Mon Mar 6 15:04:19 2023 +0100

    [full-ci] Improve ini backend, support redis sentinel caches (#5737)

    * Configure the file metadata cache for storage-users

    * Also configure the cache for the system storage

    * Update services/storage-system/pkg/config/config.go

    Co-authored-by: Martin <github@diemattels.at>

    * Improve documentation

    * Bump reva to pull in the latest ini backend changes

    * Fix missing comment

    * Update services/storage-users/README.md

    Co-authored-by: Martin <github@diemattels.at>

    * Update services/storage-system/README.md

    Co-authored-by: Martin <github@diemattels.at>

    * Tweak docs

    * Apply suggestions from code review

    Co-authored-by: Jörn Friedrich Dreyer <jfd@butonic.de>
    Co-authored-by: Martin <github@diemattels.at>

    ---------

    Co-authored-by: Martin <github@diemattels.at>
    Co-authored-by: Jörn Friedrich Dreyer <jfd@butonic.de>
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.

support redis sentinel
3 participants