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

Helm - Twilio fails validation unless all auth fields are provided #2654

Closed
njohnstone2 opened this issue Jul 27, 2023 · 0 comments · Fixed by #2674
Closed

Helm - Twilio fails validation unless all auth fields are provided #2654

njohnstone2 opened this issue Jul 27, 2023 · 0 comments · Fixed by #2674
Labels
bug Something isn't working part:helm/kubernetes/docker

Comments

@njohnstone2
Copy link
Contributor

What went wrong?

What happened:
the helm chart validation for Twilio requires all 3 auth fields to be provided, resulting in a validation error if any are not specified.

What did you expect to happen:

Either (TWILIO_API_KEY_SID + TWILIO_API_KEY_SECRET) or TWILIO_AUTH_TOKEN is required.

Oncall Env Variables defines two options for configuring Twilio Auth:

  1. TWILIO_API_KEY_SID + TWILIO_API_KEY_SECRET
  2. TWILIO_AUTH_TOKEN

It would be good if these fields were made optional so that Oncall does not require config that is never used

How do we reproduce it?

values.yaml

oncall:
  twilio:
    accountSid: "test-sid"
    existingSecret: "my-secret"
    authTokenKey: "auth-token-key"
    phoneNumberKey: "phone-key"
    verifySidKey: "verify-key"
    # apiKeySidKey: "api-sid-key"
    # apiKeySecretKey: "api-secret-key"

Create the values file above then attempt to do a helm install with only one of the twilio auth methods

helm install oncall grafana/oncall -f values.yaml

Grafana OnCall Version

v1.3.17

Product Area

Helm

Grafana OnCall Platform?

Kubernetes

User's Browser?

n/a

Anything else to add?

No response

@njohnstone2 njohnstone2 added the bug Something isn't working label Jul 27, 2023
@joeyorlando joeyorlando linked a pull request Jul 31, 2023 that will close this issue
3 tasks
github-merge-queue bot pushed a commit that referenced this issue Jul 31, 2023
# What this PR does
When configuring twilio auth only the provided values are templated. 

## Which issue(s) this PR fixes
#2654

## Checklist

- [X] Unit, integration, and e2e (if applicable) tests updated
- [ ] Documentation added (or `pr:no public docs` PR label added if not
required)
- [ ] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)

---------

Co-authored-by: Joey Orlando <[email protected]>
Co-authored-by: Joey Orlando <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working part:helm/kubernetes/docker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant