Skip to content

chore(notediscovery): bump notediscovery to 0.28.1#719

Merged
mberlofa merged 1 commit into
mainfrom
chore/notediscovery-0.28.1
Jul 7, 2026
Merged

chore(notediscovery): bump notediscovery to 0.28.1#719
mberlofa merged 1 commit into
mainfrom
chore/notediscovery-0.28.1

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Closes #700.

Summary

  • Bump NoteDiscovery from 0.27.3 to 0.28.1.
  • Update chart defaults, values schema, unittest expectations, README, and design docs.
  • Document the 0.28.x large-vault indexing/storage path changes and keep the chart storage contract stable through persistence.mountPath.

Upstream Evidence

  • GitHub release: https://github.com/gamosoft/NoteDiscovery/releases/tag/v0.28.1
  • Release status verified with GitHub CLI: not draft, not pre-release, published 2026-07-06T07:59:01Z.
  • Docker image manifest verified: ghcr.io/gamosoft/notediscovery:0.28.1.
  • Manifest platforms: linux/amd64, linux/arm64.
  • Upstream notes: 0.28.0 adds the large-vault in-memory note index, background warmup, faster search/graph/stats paths, and configurable NOTES_DIR/PLUGINS_DIR; 0.28.1 fixes task rendering.

Site Sync

Validation

  • make image-verify IMAGE=ghcr.io/gamosoft/notediscovery:0.28.1
  • make deps-check CHART=notediscovery
  • helm unittest charts/notediscovery passed: 8 suites, 33 tests.
  • make standards-check CHART=notediscovery
  • node scripts/charts/template-standards-check.js --chart notediscovery
  • make validate-chart CHART=notediscovery passed fully: 19 layers, including k3d behavioral validation for default plus all CI values scenarios.
  • make site-sync-check CHART=notediscovery
  • make release-check REPO=charts
  • make attribution-check REPO=charts

Summary by CodeRabbit

  • New Features

    • Updated the bundled NoteDiscovery version to 0.28.1 across the Helm chart and default image settings.
  • Documentation

    • Revised setup and design docs to reference the newer image version.
    • Added upgrade notes with guidance for safer upgrades, including storage consistency and backup recommendations.
  • Tests

    • Updated chart tests to expect the new pinned image version.

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Standards Check (GR-079) — PASS

Every changed chart fully passes standards-check.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 003abbe0-85d0-45a9-8adf-8961818b2dea

📥 Commits

Reviewing files that changed from the base of the PR and between 66a35d5 and bf4a29f.

📒 Files selected for processing (6)
  • charts/notediscovery/Chart.yaml
  • charts/notediscovery/DESIGN.md
  • charts/notediscovery/README.md
  • charts/notediscovery/tests/templates_test.yaml
  • charts/notediscovery/values.schema.json
  • charts/notediscovery/values.yaml

📝 Walkthrough

Walkthrough

This PR bumps the NoteDiscovery Helm chart's image version from 0.27.3 to 0.28.1 across Chart.yaml, values.yaml, values.schema.json, DESIGN.md, README.md, and the unit test. README.md also adds a new "Upgrade Notes" section describing 0.28.1 changes and upgrade guidance.

Changes

NoteDiscovery Image Version Bump

Layer / File(s) Summary
Chart metadata and values update
charts/notediscovery/Chart.yaml, charts/notediscovery/values.yaml, charts/notediscovery/values.schema.json
appVersion, Artifact Hub changelog entry, image tag default, and values tag are bumped from 0.27.3 to 0.28.1.
Documentation updates
charts/notediscovery/DESIGN.md, charts/notediscovery/README.md
Image references updated to 0.28.1; README adds an "Upgrade Notes" section covering vault performance work, task rendering fix, in-memory note index, background index warmup, and PVC backup guidance.
Test assertion update
charts/notediscovery/tests/templates_test.yaml
The pinned image assertion is updated to expect the 0.28.1 tag.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related issues: #700 (upstream image update for notediscovery, 0.27.3 → 0.28.1)

Suggested labels: chore, dependencies, notediscovery

Suggested reviewers: helmforgedev/charts maintainers

🐰 A tiny bump, from 27 to 28,
Notes now index a little more,
Backups tucked away with care,
Docs updated everywhere,
Hop along, the chart's secure!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is concise and accurately summarizes the main change: bumping NoteDiscovery to 0.28.1.
Linked Issues check ✅ Passed The chart update, defaults, tests, and docs all align with issue #700's request to move NoteDiscovery to 0.28.1.
Out of Scope Changes check ✅ Passed The README, DESIGN, schema, and test updates are all relevant support changes for the version bump, with no obvious unrelated scope.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/notediscovery-0.28.1

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

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🟢 Security Scan: notediscovery

Framework Score
MITRE + NSA + SOC2 87.37374%

✅ Security posture acceptable.

@mberlofa mberlofa merged commit 5f5ada7 into main Jul 7, 2026
17 checks passed
@mberlofa mberlofa deleted the chore/notediscovery-0.28.1 branch July 7, 2026 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(notediscovery): upstream image update available (0.27.3 → 0.28.1)

1 participant