Skip to content

Conversation

@rahulsadanandan
Copy link
Collaborator

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • CHANGELOG.md updated
  • Variables and other changes are documented in the README.md
  • Title of the PR starts with chart name (e.g. [artifactory])

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

password: "your-custom-password"
```

## Deployment

Choose a reason for hiding this comment

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

The readme tells the user how to deploy, but not how to verify that Valkey is actually running or that Catalog is connecting to it. After deployment, please add the Verification section towards the end.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Its done

global:
valkey:
password: "your-custom-password"
```

Choose a reason for hiding this comment

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

add a note

For production environments, avoid hardcoding passwords in plain text. Use Kubernetes Secrets or an external secrets manager.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@nagarajuv-jfrog
Currently, we do not support Valkey passwords as secrets. This is a feature request that needs to be taken up.


This example shows how to deploy JFrog Xray with Catalog using Valkey for caching. Valkey is an open-source, in-memory data structure store that acts as a cache to improve performance.

## What is Valkey?
Copy link
Contributor

Choose a reason for hiding this comment

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

You are not describing what Valkey is. Just explaining how it's deployed. I suggest adjusting the title of this section.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Changed now..

Copy link
Member

@oumkale oumkale left a comment

Choose a reason for hiding this comment

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

LGTM!

@rahulsadanandan rahulsadanandan merged commit 34b8bc6 into master Jan 28, 2026
1 check passed
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.

6 participants