Skip to content

Update Helm chart database config to include spider database name #1845

@coderabbitai

Description

@coderabbitai

Description

The Helm chart configuration needs to be updated to match the new database config structure introduced in #1828.

Problem

In #1828, the Database config struct was updated to change from name: String to names: ClpDbNames { clp, spider }. However, the Helm chart configuration files were not updated accordingly, causing API server deployments to fail with:

Error: Config file /etc/clp-config.yaml does not exist

Caused by:
    0: `serde_yaml::Error`: database.names: missing field `spider` at line 35 column 5
    1: database.names: missing field `spider` at line 35 column 5

Files to Update

  1. tools/deployment/package-helm/values.yaml - Add spider field under clpConfig.database.names
  2. tools/deployment/package-helm/templates/configmap.yaml - Add spider field in the database config template

Current Configuration

database:
  names:
    clp: "clp-db"

Expected Configuration

database:
  names:
    clp: "clp-db"
    spider: "spider-db"

Related

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions