Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix missing dependency when postgres and redis is disabled #115

Closed
wants to merge 4 commits into from
Closed

Fix missing dependency when postgres and redis is disabled #115

wants to merge 4 commits into from

Conversation

Placidina
Copy link

@Placidina Placidina commented Oct 26, 2023

Reproduce:

Custom values:

nodeSelector:
  environment: support
postgresql:
  enabled: false
redis:
  enabled: false

Apply

helm upgrade --install  --version 1.1.3 --wait --timeout 300s --debug --dry-run -f values.yaml chatwoot chatwoot/chatwoot

Output:

history.go:56: [debug] getting history for release chatwoot
Release "chatwoot" does not exist. Installing it now.
install.go:214: [debug] Original chart version: "1.1.3"
install.go:231: [debug] CHART PATH: /home/arch/.cache/helm/repository/chatwoot-1.1.3.tgz

Error: template: 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"
helm.go:84: [debug] template: 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"

@vishnu-narayanan
Copy link
Member

@Placidina Thank you for the PR. Could you please create an issue that describes the problem being solved?

@Placidina Placidina mentioned this pull request Oct 27, 2023
@Placidina
Copy link
Author

Placidina commented Oct 27, 2023

@Placidina Thank you for the PR. Could you please create an issue that describes the problem being solved?

@vishnu-narayanan Craeted #116

@Placidina
Copy link
Author

#101

@Placidina Placidina closed this Mar 13, 2024
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