-
Notifications
You must be signed in to change notification settings - Fork 25
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
Adds blog post on Bitnami's Valkey helm chart #91
Conversation
Signed-off-by: Rafael Rios Saavedra <rrios@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like his post!
There are a couple of small technical issues that should be easy to fix marked in the review.
The preference is to use one sentence-per-line, it makes it easier to review.
Otherwise, I'd like to see a little more distinction between Bitnami and Tanzu's app catalog: both provide value but the reader should fully know they're different and which one is open source.
Signed-off-by: Rafael Rios Saavedra <rrios@vmware.com>
Co-authored-by: Carlos Rodríguez Hernández <carlosrh@vmware.com> Signed-off-by: Rafael Ríos Saavedra <rafael.rios.saavedra@gmail.com>
Co-authored-by: Carlos Rodríguez Hernández <carlosrh@vmware.com> Signed-off-by: Rafael Ríos Saavedra <rafael.rios.saavedra@gmail.com>
@stockholmux Did you have the chance to take a look to my changes/comments ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see four little issues, should be easy fixes.
I really like the way this is going on it. I want to get another set of eyes on it after the fixes are on.
Signed-off-by: Rafael Rios Saavedra <rrios@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@rafariossaa I think I'm good with the post. Aim for early next week to merge and publish. |
Thanks ! |
Description
Adds a blog entry about using Bitnami's Vakey helm chart.
Issues Resolved
Check List
--signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.