Fix: Allow using boolean and integer as a value in with MM_* config helm parameters#447
Fix: Allow using boolean and integer as a value in with MM_* config helm parameters#447mike-code wants to merge 1 commit intomattermost:masterfrom
Conversation
|
Hello @mike-code, Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here. Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project. Please help complete the Mattermost contribution license agreement? This is a standard procedure for many open source projects. Please let us know if you have any questions. We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team. |
|
This PR has been automatically labelled "stale" because it hasn't had recent activity. |
|
/check-cla |
Fixes #238
Summary
This fix allows setting integers and booleans as values for helm parameters.
Description
Without this fix, using boolean as a value in any
config.MM_*helm parameter would result in the following errorA workaround would be to wrap the quoted boolean in single quotes ie.
however this environment variable would be rejected by the mattermost server and thus the user API deletion would still be disabled.