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

Slack token from helm chart is out of sync with code #799

Open
leovalverde-practiv opened this issue Mar 4, 2025 · 0 comments
Open

Slack token from helm chart is out of sync with code #799

leovalverde-practiv opened this issue Mar 4, 2025 · 0 comments

Comments

@leovalverde-practiv
Copy link

Issue description

The latest version of the helm chart (v1.0.5) feeds the SLACK_TOKEN secret (via Kubernetes secrets) into the application container, but the codebase expects it provided as SLACK_BOT_TOKEN.

Temporary fix

There is a workaround, which is to set secret.create="false" on the helm chart, and then provide your own secret containing a value for SLACK_BOT_TOKEN, also setting your custom secret name (e.g. "keel") as secret.name="keel", but that's not good and should be fixed.

Note: Missing info on docs

As a side note, the documentation seems to be quite behind compared to the codebase, requiring developers to explore the codebase in order to customise the application's behaviour or solve issues. Some examples are:

  • Authentication via Google Workload Identity is now implemented, but there are no mentions to it on the documentation.
  • When using slack for notifications, granting the scope chat:write.customize allows us to customise the username for slack messages via the helm parameter slack.botName. The docs don't mention anything about slack scopes.
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

No branches or pull requests

1 participant