Skip to content
Draft
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 40 additions & 0 deletions .github/workflows/helm-lint-test.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
name: Lint and Test Helm Chart

on:
pull_request:
branches:
- main
push:
branches:
- '**'

jobs:
lint-test:
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Jan 6, 2026

Choose a reason for hiding this comment

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

P2: Missing explicit permissions specification. Following security best practices, add a permissions block with minimal required access (contents: read) at the job or workflow level.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .github/workflows/helm-lint-test.yaml, line 12:

<comment>Missing explicit `permissions` specification. Following security best practices, add a permissions block with minimal required access (`contents: read`) at the job or workflow level.</comment>

<file context>
@@ -0,0 +1,40 @@
+      - &#39;**&#39;
+
+jobs:
+  lint-test:
+    runs-on: ubuntu-latest
+    steps:
</file context>

✅ Addressed in 76cc880

runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Set up Helm
uses: azure/setup-helm@v4.3.0
with:
version: v3.12.0

- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: 3.12

- name: Set up chart-testing
uses: helm/chart-testing-action@v2.7.0

- name: Run chart-testing (lint)
run: ct lint --config helm/ct.yaml

- name: Create kind cluster
uses: helm/kind-action@v1.8.0

- name: Run chart-testing (install)
run: ct install --config helm/ct.yaml
58 changes: 58 additions & 0 deletions .github/workflows/helm-release-oci.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
name: Release Helm Chart

on:
push:
branches:
- main
workflow_dispatch:

permissions:
contents: write
packages: write

jobs:
release:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Configure Git
run: |
git config user.name "$GITHUB_ACTOR"
git config user.email "$GITHUB_ACTOR@users.noreply.github.com"

- name: Set up Helm
uses: azure/setup-helm@v4.3.0
with:
version: v3.12.0

- name: Add dependency chart repos
run: |
helm repo add bitnami https://charts.bitnami.com/bitnami

- name: Run chart-releaser
uses: helm/chart-releaser-action@v1.5.0
with:
charts_dir: helm
config: helm/cr.yaml
env:
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
CR_SKIP_EXISTING: "true"

- name: Login to GHCR
uses: docker/login-action@v3.6.0
with:
registry: ghcr.io
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}

- name: Push charts to GHCR
run: |
for pkg in .cr-release-packages/*.tgz; do
if [ -f "${pkg}" ]; then
helm push "${pkg}" "oci://ghcr.io/${GITHUB_REPOSITORY_OWNER}/helm-charts"
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Jan 6, 2026

Choose a reason for hiding this comment

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

P2: GHCR requires lowercase repository paths, but GITHUB_REPOSITORY_OWNER preserves original casing. Convert to lowercase to prevent failures for users with mixed-case GitHub usernames/organizations.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .github/workflows/helm-release-oci.yaml, line 56:

<comment>GHCR requires lowercase repository paths, but `GITHUB_REPOSITORY_OWNER` preserves original casing. Convert to lowercase to prevent failures for users with mixed-case GitHub usernames/organizations.</comment>

<file context>
@@ -0,0 +1,58 @@
+        run: |
+          for pkg in .cr-release-packages/*.tgz; do
+            if [ -f &quot;${pkg}&quot; ]; then
+              helm push &quot;${pkg}&quot; &quot;oci://ghcr.io/${GITHUB_REPOSITORY_OWNER}/helm-charts&quot;
+            fi
+          done
</file context>

✅ Addressed in 76cc880

fi
done
27 changes: 27 additions & 0 deletions helm/Chart.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
apiVersion: v2
name: open-notebook
description: An Open Source implementation of Notebook LM with more flexibility and features
type: application
version: 1.0.0
appVersion: "0.0.1"
keywords:
- notebook
- notebooklm
- knowledge-management
- surrealdb
home: https://github.com/lfnovo/open-notebook
sources:
- https://github.com/lfnovo/open-notebook
maintainers:
- name: lfnovo
url: https://github.com/lfnovo
- name: thuanpham582002
url: https://github.com/thuanpham582002
annotations:
category: Analytics
licenses: MIT

dependencies:
- name: common
repository: oci://registry-1.docker.io/bitnamicharts
version: 2.x.x
Loading