Patching minor bug in `configmap-env.yaml`

mastodon / chart

Helm chart for Mastodon deployment in Kubernetes

GNU Affero General Public License v3.0

154 stars 89 forks source link

Closed wrmilling closed 1 year ago

wrmilling commented 1 year ago

There is a minor bug in configmap-env.yaml which prevents deployment when using an S3 Alias Host, this patch addresses that.

Also included is a minor change to use an if statement when not using anything but the truthy value of the configuration item.

Please let me know if you have any questions and if you would rather I not submit with the chart patch version bump.

iameli commented 1 year ago

@wrmilling @deepy Can this please be merged? Just ran into this today as well.

deepy commented 1 year ago

@iameli I don't have access to commit or merge so unfortunately you'll have to ping someone else

iameli commented 1 year ago

I see. Ping @Gargron

renchap commented 1 year ago

This has been fixed by #35

Feel free to re-open a PR for your minor if change.