docs: restore missing 3.0.0 Helm chart entries in index.yaml#1795
docs: restore missing 3.0.0 Helm chart entries in index.yaml#1795PhilipSchmid wants to merge 1 commit intosplunk:mainfrom
Conversation
Upstream commit 38373ce accidentally dropped the 3.0.0 entries for both splunk-operator and splunk-enterprise from docs/index.yaml while publishing 3.1.0. The 3.0.0 chart tarballs are still available at their direct URLs, so the index should reference them. Restored entries: - splunk-operator 3.0.0 (digest bd318b1f, URL splunk-operator-3.0.0.tgz) - splunk-enterprise 3.0.0 (3 entries with digests ae82f6c8, 9a4133ae, db8d639d, all pointing to splunk-enterprise-3.0.0.tgz) Signed-off-by: Philip Schmid <phisch@cisco.com>
|
CLA Assistant Lite bot All contributors have signed required documents ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
I have read the Code of Conduct and I hereby sign the COC |
vivekr-splunk
left a comment
vivekr-splunk
left a comment
There was a problem hiding this comment.
I found one blocking issue in the restored Helm repo metadata.
| urls: | ||
| - https://splunk.github.io/splunk-operator/splunk-enterprise-3.0.0.tgz | ||
| version: 3.0.0 | ||
| - apiVersion: v2 |
There was a problem hiding this comment.
This adds multiple conflicting splunk-enterprise entries for version 3.0.0 that all point to the same chart URL. In the checked-out branch there are now three 3.0.0 entries under entries.splunk-enterprise; only the first digest (ae82f6...) matches docs/splunk-enterprise-3.0.0.tgz, while the later ones have different digests, and one of them even carries appVersion: 2.8.1. From a Helm repo consumer point of view, that leaves the index ambiguous and partially incorrect, so I don’t think this is safe to merge as-is.
vivekr-splunk
left a comment
vivekr-splunk
left a comment
There was a problem hiding this comment.
This also looks duplicate/superseded by the later Helm 3.0.0 index-restore PRs in the repo. I would consolidate on a single corrected restore PR instead of merging multiple overlapping attempts in this area.
|
Ah, perfect, then let's leave it to the other PR from one of your colleagues to address it. 😄 |
2 participants
Upstream commit 38373ce accidentally dropped the 3.0.0 entries for both splunk-operator and splunk-enterprise from docs/index.yaml while publishing 3.1.0, which breaks existing installation automations that refer to 3.0.0. The 3.0.0 chart tarballs are still available at their direct URLs (e.g., https://splunk.github.io/splunk-operator/splunk-operator-3.0.0.tgz), so the index should reference them.
Restored entries: