add enterprise helm to helm repo#1622
Conversation
WalkthroughThis pull request increments the Parseable Helm chart version from 2.6.5 to 2.6.6 across chart metadata, deployment values, and repository index, while removing deprecated operator chart entries and adding new parseable-enterprise chart entries. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
index.yaml (1)
6-1582: Consider preserving historicalcreatedtimestamps to reduce index churn.The bulk rewrite of
createdfields across old releases makes diffs noisy and obscures original release metadata. Consider an incremental index update flow that only adds new entries (plusgenerated) when possible.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@index.yaml` around lines 6 - 1582, The index generation is overwriting historical created timestamps in index.yaml (the repeated "created:" fields) causing noisy diffs; change the index update routine to preserve existing entry.created values and only set created for new chart entries (and update the top-level generated timestamp) — locate the code that writes index.yaml / the index generation routine (the logic that iterates chart entries and assigns "created") and modify it to: 1) load existing index.yaml, 2) merge new releases by key (name+version) leaving existing entry.created intact, and 3) only assign created for truly new entries while still updating the overall generated timestamp.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@index.yaml`:
- Around line 49-52: Mismatch between declared fluent-bit version (0.48.0) and
the locked dependency (0.25.0) causing stale Chart.lock; regenerate it. Update
the chart lock by running helm dependency update in the chart directory to
refresh Chart.lock so the fluent-bit entry matches version 0.48.0, then commit
the updated Chart.lock; verify the Chart.lock fluent-bit version is 0.48.0 (and
that Chart.yaml still declares fluent-bit: 0.48.0) before pushing.
---
Nitpick comments:
In `@index.yaml`:
- Around line 6-1582: The index generation is overwriting historical created
timestamps in index.yaml (the repeated "created:" fields) causing noisy diffs;
change the index update routine to preserve existing entry.created values and
only set created for new chart entries (and update the top-level generated
timestamp) — locate the code that writes index.yaml / the index generation
routine (the logic that iterates chart entries and assigns "created") and modify
it to: 1) load existing index.yaml, 2) merge new releases by key (name+version)
leaving existing entry.created intact, and 3) only assign created for truly new
entries while still updating the overall generated timestamp.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 4f2357e1-f8b2-46a0-8293-d5a194b14638
📒 Files selected for processing (5)
helm-releases/parseable-2.6.6.tgzhelm-releases/parseable-enterprise-2.6.6.tgzhelm/Chart.yamlhelm/values.yamlindex.yaml
Enterprise image needs a license to run
Summary by CodeRabbit
Release Notes