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

set kerberosvault to true (thumbnails) #23

Merged
merged 1 commit into from
Dec 2, 2024
Merged

Conversation

cedricve
Copy link
Member

@cedricve cedricve commented Nov 29, 2024

Description

Pull Request Description: Set kerberosvault to True (Thumbnails)

Motivation

The primary motivation behind this change is to enable the use of Kerberos Vault for storing thumbnails. By setting kerberosvault.enabled to true, we can leverage the secure and efficient storage capabilities of Kerberos Vault, which will improve the overall management and retrieval of thumbnails.

Why It Improves the Project

  1. Enhanced Security: Kerberos Vault provides a secure storage solution, ensuring that thumbnails are stored in a protected environment.
  2. Improved Efficiency: By using Kerberos Vault, the project benefits from optimized storage and retrieval processes, leading to better performance.
  3. Centralized Management: Storing thumbnails in Kerberos Vault allows for centralized management, making it easier to maintain and scale the storage system as the project grows.
  4. Future-Proofing: Enabling Kerberos Vault prepares the project for future enhancements and integrations that may rely on secure and efficient storage solutions.

Changes Made

  • Updated charts/hub/values.yaml to set kerberosvault.enabled from false to true.
  kerberosvault:
-    enabled: false # If you want to use Kerberos Vault to store the thumbnails
+    enabled: true # If you want to use Kerberos Vault to store the thumbnails

This change is repeated in multiple sections of the charts/hub/values.yaml file to ensure that Kerberos Vault is enabled consistently across different configurations.

By implementing this change, we enhance the project's security, efficiency, and scalability, aligning with our long-term goals for robust and reliable storage solutions.

@cedricve cedricve merged commit 121ea68 into main Dec 2, 2024
9 checks passed
@cedricve cedricve deleted the cedricve-patch-1 branch December 2, 2024 10:22
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