Skip to content

chore: rename testkube-ai-service.db to postgres to align with other services#511

Open
de-wim wants to merge 1 commit intomainfrom
update-postgres-config-ai
Open

chore: rename testkube-ai-service.db to postgres to align with other services#511
de-wim wants to merge 1 commit intomainfrom
update-postgres-config-ai

Conversation

@de-wim
Copy link
Copy Markdown

@de-wim de-wim commented Mar 25, 2026

Updated testkube-ai-service documentation to accompany helm changes of https://github.com/kubeshop/testkube-cloud-api/pull/3238

@de-wim de-wim requested a review from a team as a code owner March 25, 2026 15:49
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
testkube-docs Ready Ready Preview, Comment Mar 25, 2026 3:50pm

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 25, 2026

Greptile Summary

This PR updates the AI enabling documentation to reflect the Helm chart rename of the testkube-ai-service database key from db to postgres, aligning it with changes in testkube-cloud-api#3238. It also fixes a missing trailing newline at the end of the file.

Key changes:

  • Renames db:postgres: in the YAML Helm configuration example (docs/articles/ai-enabling.mdx, line 32).
  • Adds a missing trailing newline at end of file.

Issue found:

  • The Default Values Reference table on line 76 still refers to testkube-ai-service.db.* — this was not updated alongside the YAML config block, leaving the documentation inconsistent.

Confidence Score: 4/5

  • Safe to merge after fixing the stale db.* reference in the Default Values table.
  • The change is straightforward and correct in the YAML example block, but a leftover testkube-ai-service.db.* in the reference table creates documentation inconsistency. Once that single line is fixed, the PR is clean.
  • docs/articles/ai-enabling.mdx line 76 — the Default Values Reference table still uses the old db.* key name.

Important Files Changed

Filename Overview
docs/articles/ai-enabling.mdx Updated YAML config example to use postgres: instead of db:, and added missing trailing newline. However, the Default Values Reference table on line 76 still references the old testkube-ai-service.db.* key and was not updated to match.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Helm Chart: testkube-ai-service] --> B[enabled: true]
    A --> C["postgres: (renamed from db:)"]
    A --> D[llmApi]

    C --> C1[secretRef.name]
    C --> C2[secretRef.endpointKey]
    C --> C3[secretRef.usernameKey]
    C --> C4[secretRef.passwordKey]
    C --> C5[name: testkube_ai]

    D --> D1[secretRef]
    D --> D2[secretRefKey]

    E[testkube-cloud-api] --> E1[aiApiSubdomain]
    E --> E2[restApiSubdomain]

    F[testkube-cloud-ui] --> F1["ai.enabled: true"]

    style C fill:#90EE90
    style A fill:#lightblue
Loading

Comments Outside Diff (1)

  1. docs/articles/ai-enabling.mdx, line 76 (link)

    P2 Stale db.* reference in Default Values table

    The YAML config block was updated from db: to postgres: (line 32), but the Default Values Reference table on this line still uses the old key name testkube-ai-service.db.*. This leaves the documentation internally inconsistent — the table no longer matches the Helm values structure described above it.

Reviews (1): Last reviewed commit: "chore: rename testkube-ai-service.db to ..." | Re-trigger Greptile

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