Skip to content

docs(helm): replace unavailable chart repositories with source installation - #43

Open
ANAMASGARD wants to merge 1 commit into
kubevirt:mainfrom
ANAMASGARD:fix/helm-chart-publication-21
Open

ANAMASGARD wants to merge 1 commit into
kubevirt:mainfrom
ANAMASGARD:fix/helm-chart-publication-21

Conversation

@ANAMASGARD

Copy link
Copy Markdown

Summary

The documented GitHub Pages chart repository returns 404, and the documented Quay OCI chart endpoint is unavailable to anonymous users. This leaves the Helm quickstart without a usable chart source.

Update the main README, chart guide, and OpenShift walkthrough to install the controller from the chart included in a source checkout:

  • Document cloning the repository, selecting an optional Git revision, exporting and editing values, and installing from ./helm.
  • Use the local chart for upgrades, preserving the release namespace and custom values.
  • Correct the image override to global.image.tag and explain that selecting a Git revision does not automatically select a matching container image.
  • Remove the broken chart badge, hosted-chart command examples, and obsolete version and release claims. Retain the personal repository only for the walkthrough's separate VM chart dependency.

This is a documentation-only workaround for #21. It does not restore public chart hosting or change the application, chart metadata, defaults, or publishing configuration.

Fixes #21

Validation

  • helm show values ./helm
  • helm lint ./helm
  • helm template with defaults, exported values, and the OpenShift walkthrough's custom values
  • Image-tag override checks for both custom value sets, confirming the rendered deployment uses the requested global.image.tag
  • git diff --check

All checks passed. No live Kubernetes or OpenShift deployment was tested.

AI assistance

An AI assistant was used to prepare and review the documentation changes, run validation, and raise this PR.

Signed-off-by: Gaurav Chaudhary <chaudharygaurav2004@gmail.com>
@kubevirt-prow kubevirt-prow Bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Nov 12, 2026
@kubevirt-prow
kubevirt-prow Bot requested review from MarSik and yanirq November 12, 2026 18:48
@kubevirt-prow

kubevirt-prow Bot commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign marsik for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-prow

kubevirt-prow Bot commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Hi @ANAMASGARD. Thanks for your PR.

PRs from untrusted users cannot be marked as trusted with /ok-to-test in this repo meaning untrusted PR authors can never trigger tests themselves. Collaborators can still trigger tests on the PR using /test all.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubevirt-prow kubevirt-prow Bot added the size/L label Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Both helm repos appear to be missing.

1 participant