Skip to content

Add bitnami common lib as a dependency#183

Merged
vishnu-narayanan merged 7 commits intochatwoot:mainfrom
jklemm:main
Feb 19, 2026
Merged

Add bitnami common lib as a dependency#183
vishnu-narayanan merged 7 commits intochatwoot:mainfrom
jklemm:main

Conversation

@jklemm
Copy link
Copy Markdown
Contributor

@jklemm jklemm commented Jan 20, 2026

Details

Starting with PR #45, a dependency on the Bitnami "common" library was added.

There are two issues addressing this, #101 and #116, and a solution has not yet been applied.

This PR adds the common library as a dependency of chart, thus avoiding the error below:

│ Error: malformed chart or values:
│ 	templates/:
            template:
                  chatwoot/charts/chatwoot/templates/tests/test-connection.yaml:19:20:
                  executing "chatwoot/charts/chatwoot/templates/tests/test-connection.yaml" at <include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $)>:
                  error calling include: template: no template "common.tplvalues.render" associated with template "gotpl"

Steps to reproduce

  1. Create a values.yaml at repo root.
nodeSelector:
  chatwoot: "true"

tolerations: []

postgresql:
  enabled: false
  auth:
    username: username
    postgresPassword: password
    database: chatwoot
  postgresqlHost: chatwoot
  postgresqlPort: 5432

redis:
  enabled: false
  host: host
  port: 6379
  1. Run command helm template chatwoot charts/chatwoot -f values.yaml > rendered.yaml
  2. Results the error:
Error: chatwoot/templates/tests/test-connection.yaml:19:20
  executing "chatwoot/templates/tests/test-connection.yaml" at <include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $)>:
    error calling include:
template: no template "common.tplvalues.render" associated with template "gotpl"

Use --debug flag to render out invalid YAML

@jklemm
Copy link
Copy Markdown
Contributor Author

jklemm commented Jan 20, 2026

@Sharygin-Aleksei @jdbranham @Placidina help me with your comments about this issue.

Thanks.

Copy link
Copy Markdown
Member

@vishnu-narayanan vishnu-narayanan left a comment

Choose a reason for hiding this comment

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

@jklemm Thank you for the PR. Could you please increment the chart version as well?

Comment thread charts/chatwoot/Chart.yaml Outdated
@vishnu-narayanan vishnu-narayanan merged commit 7f49f82 into chatwoot:main Feb 19, 2026
1 check passed
vishnu-narayanan added a commit that referenced this pull request Feb 23, 2026
Reverts #183. The OCI registry dependency breaks chart-releaser
and helm dependency resolution due to Docker Hub auth requirements.
vishnu-narayanan added a commit that referenced this pull request Feb 23, 2026
Helm 3.4.0 needs HELM_EXPERIMENTAL_OCI=1 to resolve
OCI registry dependencies added in #183.
vishnu-narayanan added a commit that referenced this pull request Feb 23, 2026
Helm 3.4.0 needs HELM_EXPERIMENTAL_OCI=1 to resolve
OCI registry dependencies added in #183.
@jklemm
Copy link
Copy Markdown
Contributor Author

jklemm commented Feb 23, 2026

Thanks @vishnu-narayanan !

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