Skip to content

Commit

Permalink
Configure whether languages should be installed on a per chart basis (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
jmgrady authored Sep 7, 2023
1 parent fac3bf4 commit d5b341a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion deploy/scripts/setup_combine.py
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,8 @@ def main() -> None:
]
)

add_language_overrides(this_config, chart=chart, langs=args.langs)
if config["charts"][chart]["install_langs"]:
add_language_overrides(this_config, chart=chart, langs=args.langs)

add_override_values(
this_config,
Expand Down
3 changes: 3 additions & 0 deletions deploy/scripts/setup_files/combine_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@ profiles:
charts:
thecombine:
namespace: thecombine
install_langs: true
secrets:
- config_item: awsAccount
env_var: AWS_ACCOUNT
Expand All @@ -134,6 +135,7 @@ charts:
env_var: COMBINE_SMTP_PASSWORD
create-admin-user:
namespace: thecombine
install_langs: false
secrets:
- config_item: awsAccount
env_var: AWS_ACCOUNT
Expand All @@ -153,6 +155,7 @@ charts:
env_var: COMBINE_ADMIN_EMAIL
cert-proxy-server:
namespace: combine-cert-proxy
install_langs: false
secrets:
- config_item: awsAccount
env_var: AWS_ACCOUNT
Expand Down

0 comments on commit d5b341a

Please sign in to comment.