docs: add Artifact Hub badge for CLOMonitor#5371
Conversation
Add README badge linking to the official Helm chart on Artifact Hub (helm/volcano-sh/volcano) for CLOMonitor artifacthub_badge (issue 5366). Signed-off-by: Himanshu <144804569+kubeboiii@users.noreply.github.com>
volcano-sh-bot
added
the
kind/documentation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
volcano-sh-bot
added
the
size/XS
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds an Artifact Hub badge to the README to improve discoverability of the Volcano Helm chart on Artifact Hub.
Changes:
- Add Artifact Hub shield/badge linking to the Volcano Helm chart listing.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds an Artifact Hub badge to the README.md file. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
/lgtm /cc @JesseStutler |
|
@yashisrani: changing LGTM is restricted to collaborators DetailsIn response to this:
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/test-infra repository. |
| [](https://github.com/volcano-sh/volcano/blob/master/LICENSE) | ||
| [](https://bestpractices.coreinfrastructure.org/projects/3012) | ||
| [](https://scorecard.dev/viewer/?uri=github.com/volcano-sh/volcano) | ||
| [](https://artifacthub.io/packages/helm/volcano-sh/volcano) |
There was a problem hiding this comment.
https://artifacthub.io/packages/helm/volcano-sh/volcano
Looks like it indeed it's volcano latest helm chart, but who is vshulkin? It's a private owner? Even the page doesn't contain a README
There was a problem hiding this comment.
The biggest risk for a private user is that they might upload a helm chart with a virus. Maybe you should work on other issues first, but if you found an officially certified artifact, that's okay too. However, the artifact hub isn't a key item for CLOMonitor scores
There was a problem hiding this comment.
We use https://github.com/volcano-sh/helm-charts to host helm charts, why do we must link artifacthub, which i think we did not maintain?
What type of PR is this?
/kind documentation
What this PR does / why we need it:
This PR is part of the work for #5366 (improve CLOMonitor score). It addresses Artifact Hub badge on the issue.
Adds a README badge linking to
https://artifacthub.io/packages/helm/volcano-sh/volcano. That listing uses the official Helm repository URL (https://volcano-sh.github.io/helm-charts) already documented in the install section.Does not overlap with #5367 (Security Insights + SBOM), #5369 (contributing guide and workflow permissions), or #5370 (license scanning metadata). README-only change.
Which issue(s) this PR fixes:
Part of #5366
Special notes for your reviewer:
make verifypassed locally (macOS).helm/volcano-sh/volcanois the correct Artifact Hub package for the official chart (listing is not marked verified publisher on Artifact Hub).artifacthub_badgeon clomonitor.io may take up to about an hour to refresh.Does this PR introduce a user-facing change?