search

supabase-community / supabase-kubernetes

Helm 3 charts to deploy a Supabase on Kubernetes
Apache License 2.0
452 stars 119 forks source link

Fix kong permission and execution errors in wrapper.sh #45

Open pieveee opened 1 year ago

pieveee commented 1 year ago

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Currently, kong fails to start running with the following error:

/scripts/wrapper.sh: 3: set: Illegal option -o pipefail

And:

/scripts/wrapper.sh: line 8: /home/kong/kong.yml: Permission denied

What is the new behavior?

  1. The shebang was adjusted from /bin/sh to /bin/bash (See https://www.baeldung.com/linux/illegal-option-o-pipefail)
  2. kong.yml declarative configuration file was moved from /home/kong/kong.yml to /usr/local/kong/kong.yml, as well as /home/kong/template.yml to /usr/local/template.yml in order to run wrapper.sh script without permission issues
pieveee commented 1 year ago

@milanvanschaik Can you check the merge request?

ngladbach commented 1 year ago

@milanvanschaik @bigbitbus @icrotz @cpdeethree @devudopw @kiwicopple could you have a look?

icrotz commented 1 year ago

I don't have the rights to merge on this repository