Skip to content

Fix broken charts/ link on GitHub Pages landing page Fixes #233 - #237

Open
Ravi041 wants to merge 2 commits into
valkey-io:gh-pagesfrom
Ravi041:fix/gh-pages-index
Open

Fix broken charts/ link on GitHub Pages landing page Fixes #233#237
Ravi041 wants to merge 2 commits into
valkey-io:gh-pagesfrom
Ravi041:fix/gh-pages-index

Conversation

@Ravi041

@Ravi041 Ravi041 commented Aug 4, 2026

Copy link
Copy Markdown

Replace the dead charts/ directory link with GitHub Releases and list all three available charts with install examples.

Fixes #233

@greptile-apps

greptile-apps Bot commented Aug 4, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the landing page with installation examples for the standalone Valkey chart, the Valkey operator, and an operator-managed ValkeyCluster. It also replaces the dead packaged-charts directory link with GitHub Releases.

The documented Helm installation path was checked against the live repository: all three chart references resolved successfully, and the GitHub Releases link returned HTTP 200. The potential documentation failure was disproved by this executed path.

Confidence Score: 5/5

The PR is safe to merge; no blocking failure remains.

The exact repository-add, update, and chart-resolution flow succeeds for every documented chart, and the replacement releases destination is reachable.

T-Rex T-Rex Logs

What T-Rex did

  • I prepared an isolated Helm workspace and parsed the landing page to identify chart references before running commands.
  • I executed helm repo add valkey https://valkey.io/valkey-helm/, helm repo update, and helm show chart for valkey/valkey, valkey/valkey-operator, and valkey-resources in the isolated environment.
  • I confirmed the Helm fetch returned metadata for valkey 0.11.0, valkey-operator 0.4.1, and valkey-resources 0.1.2.
  • I followed the GitHub Releases URL and verified HTTP 200, and confirmed the same resolution path on the parent page, showing PR changes do not affect chart resolution.
  • I collected and organized artifacts from pre- and post-change validations, plus the focused validation script to support review.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (4): Last reviewed commit: "Updated ValkeyCluster Chart installation..." | Re-trigger Greptile

Comment thread index.html Outdated
Replace the dead charts/ directory link with GitHub Releases and list
all three available charts with install examples.

Fixes valkey-io#233

Signed-off-by: Ravindra Kumar <ravindrak041@gmail.com>
@Ravi041
Ravi041 force-pushed the fix/gh-pages-index branch from d338b31 to 416b34d Compare August 4, 2026 04:52
Signed-off-by: Ravindra Kumar <ravindrak041@gmail.com>
@Ravi041
Ravi041 force-pushed the fix/gh-pages-index branch from a7617ae to 54214ca Compare August 4, 2026 05:16
@Ravi041 Ravi041 changed the title Fix broken charts/ link on GitHub Pages landing page Fix broken charts/ link on GitHub Pages landing page Fixes #233 Aug 4, 2026
@Bloodraven21

Copy link
Copy Markdown
Collaborator

hey @Ravi041 can you help me with how you tested the code before raising the pr

@Ravi041

Ravi041 commented Aug 4, 2026

Copy link
Copy Markdown
Author

hey @Ravi041 can you help me with how you tested the code before raising the pr

I tested the index.html file locally and I cannot see 404 error anymore.

python3 -m http.server 8000

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