elixir-cloud-aai / cwl-WES

Trigger CWL workflows via GA4GH WES and TES
Apache License 2.0
16 stars 18 forks source link

Add option to specify app-config contents in values file #239

Closed zv0n closed 2 years ago

zv0n commented 2 years ago

Details

I have added option to specify app-config.yaml contents in the values file for helm. This PR addresses issue #238 I have also kept the option to use a job to populate the config map contents in case it is necessary for certain workflows.

Documentation

appConfig value contains the contents of the app-config.yaml file

configWithJob value says whether the config map should be populated with a job (as it was until now), this option is set to false by default

lvarin commented 2 years ago

On a first glance the PR looks fine. I will do a quick test and let you know. I hope to have time this week.

uniqueg commented 2 years ago

Thanks a lot @zv0n