Copilot commented on code in PR #17946:
URL: https://github.com/apache/pinot/pull/17946#discussion_r2978798006
##########
helm/index.yaml:
##########
@@ -277,13 +300,13 @@ entries:
name: pinot-dev
name: pinot
sources:
- - https://github.com/apache/pinot/tree/master/helm
+ - https://github.com/apache/pinot/tree/master/kubernetes/helm
urls:
Review Comment:
Some older chart entries in `helm/index.yaml` have their `sources` URLs
rewritten (e.g., to `.../tree/master/kubernetes/helm` or
`https://github.com/apache/pinot`). This is unrelated to publishing `1.0.0` and
risks making past releases’ metadata inaccurate. Please keep existing `sources`
values as-is and only add the new version entry.
##########
helm/index.yaml:
##########
@@ -3,35 +3,58 @@ entries:
pinot:
- apiVersion: v1
appVersion: 1.0.0
- created: "2025-10-08T22:20:54.54165-07:00"
+ created: "2026-03-23T20:24:09.802587-07:00"
+ description: Apache Pinot is an open-source distributed OLAP database for
user-facing
+ and agent-facing real-time analytics. From interactive dashboards to
LLM-powered
+ decision engines, Pinot delivers sub-second queries on fresh data at
petabyte
+ scale.
+ digest: 2d262e0d298cdc3969f2cb3aa82de1d932740b47f62e539922af98b76c717656
+ home: https://pinot.apache.org/
+ keywords:
+ - olap
+ - analytics
+ - database
+ - pinot
+ maintainers:
+ - email: [email protected]
+ name: pinot-dev
+ name: pinot
+ sources:
+ - https://github.com/apache/pinot/tree/master/helm
+ urls:
+ - pinot-1.0.0.tgz
+ version: 1.0.0
+ - apiVersion: v1
+ appVersion: 1.0.0
+ created: "2026-03-23T20:24:09.800885-07:00"
dependencies:
- - condition: pinot.zookeeper.enabled,zookeeper.enabled
- name: zookeeper
- repository: https://charts.bitnami.com/bitnami
- version: 13.x.x
+ - condition: pinot.zookeeper.enabled,zookeeper.enabled
+ name: zookeeper
+ repository: https://charts.bitnami.com/bitnami
+ version: 13.x.x
Review Comment:
`helm/index.yaml` looks like it was regenerated in a way that rewrites
historical entries (e.g., updating the `created` timestamps for older chart
versions). This makes the index noisy and can break expectations for
reproducible chart metadata. Prefer updating the index by only appending the
new `pinot-1.0.0.tgz` entry and leaving existing entries untouched (e.g., use a
merge-based index generation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
