Skip to content

update pai helm chart with targetDataset CRD#1597

Merged
nitisht merged 1 commit intoparseablehq:mainfrom
AdheipSingh:main
Mar 24, 2026
Merged

update pai helm chart with targetDataset CRD#1597
nitisht merged 1 commit intoparseablehq:mainfrom
AdheipSingh:main

Conversation

@AdheipSingh
Copy link
Contributor

@AdheipSingh AdheipSingh commented Mar 24, 2026

Fixes #XXXX.

Description


This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

Summary by CodeRabbit

  • Chores
    • Updated Helm chart repository metadata including timestamps and checksums for improved repository integrity and tracking.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 2026

Walkthrough

Updated Helm repository index metadata in index.yaml with new timestamps for pai and parseable chart entries, changed the pai chart digest, and refreshed the top-level generated timestamp. All changes are pure metadata updates with no functional code modifications.

Changes

Cohort / File(s) Summary
Helm Index Metadata
index.yaml
Updated generated timestamp at top level; changed pai chart entry digest and created timestamp; refreshed created timestamps across all 50+ parseable chart versions (ranging from 0.0.1 to 2.5.13).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • nitisht

Poem

🐰 Timestamps tick and digests shift,
Charts aligned with every gift,
Helm's index now gleaming bright,
A thousand versions set just right! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions updating the PAI Helm chart with targetDataset CRD, but the actual changes only involve updating timestamps and a digest in index.yaml—metadata updates unrelated to adding a CRD. Update the title to accurately reflect the changes (e.g., 'Update index.yaml Helm chart metadata') or clarify if this PR is incomplete regarding the targetDataset CRD implementation.
Description check ⚠️ Warning The description contains only the template with all sections unfilled and no meaningful content, providing zero context about the actual changes or their purpose. Fill in the Description section with details about what changes were made, why they were made, and what issue (if any) this PR addresses.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
index.yaml (1)

39-52: ⚠️ Potential issue | 🟠 Major

Bump pai chart version when digest changes.

Line 39 changes the pai package digest, but lines 50-52 still publish it as pai-0.1.0.tgz / version: 0.1.0. This mutates an existing chart version and can cause non-reproducible installs and cache/provenance drift. Publish this as a new chart version (for example 0.1.1) instead of replacing 0.1.0.

💡 Suggested index update pattern
-    - https://charts.parseable.com/helm-releases/pai-0.1.0.tgz
-    version: 0.1.0
+    - https://charts.parseable.com/helm-releases/pai-0.1.1.tgz
+    version: 0.1.1
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@index.yaml` around lines 39 - 52, The index entry for the chart "pai" has a
changed digest but still uses the old published artifact name and version
("pai-0.1.0.tgz" and version: 0.1.0), which mutates an existing chart version;
update the chart artifact name under urls and the version field to a new semver
(e.g., change "pai-0.1.0.tgz" to "pai-0.1.1.tgz" and set version: 0.1.1) and
ensure the digest value remains the new digest so the entry publishes as a new
chart version rather than replacing the existing one.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@index.yaml`:
- Around line 39-52: The index entry for the chart "pai" has a changed digest
but still uses the old published artifact name and version ("pai-0.1.0.tgz" and
version: 0.1.0), which mutates an existing chart version; update the chart
artifact name under urls and the version field to a new semver (e.g., change
"pai-0.1.0.tgz" to "pai-0.1.1.tgz" and set version: 0.1.1) and ensure the digest
value remains the new digest so the entry publishes as a new chart version
rather than replacing the existing one.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ba929c68-0389-4ac2-9eab-fd4567569653

📥 Commits

Reviewing files that changed from the base of the PR and between 3729c70 and e78a793.

📒 Files selected for processing (2)
  • helm-releases/pai-0.1.0.tgz
  • index.yaml

@nitisht
Copy link
Member

nitisht commented Mar 24, 2026

Skipping tests because this is not a code change

@nitisht nitisht merged commit d1b608f into parseablehq:main Mar 24, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants