[bitnami/apisix] Fixes to support running Apisix in standalone mode

[james-mchugh]

Description of the change

Updates the _helpers.tpl file to disable the addition of control-plane environment variables to the apisix.renderConfInitContainer and apisix.prepareApisixInitContainer templates when the control plane is disabled. As these environment variables are sourced from secrets, they cause the init containers to fail if the secrets do not exist.

Updates the defaultConfig in the Values.yaml so the etcd configs are removed if etcd is disabled and no external etcd servers are set.

Updates the standalone instructions in the README.md to include the disabling of the dashboard, adding the #END line to the end of the example config, and updating the config path to be mounted at the path Apisix expects the config file to be at (/usr/local/apisix/conf/apisix.yaml).

Benefits

Apisix standalone mode should now work when following the instructions in the README.

Possible drawbacks

Unexpected impacts to other deployment modes. The changes are pretty minor though, and should have minimal risk of impacting other modes.

Applicable issues

• fixes [bitnami/apisix] Standalone Mode Not Working #26881

Additional information

Checklist

• Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
• Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
• Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
• All commits signed off and in agreement of Developer Certificate of Origin (DCO)

[jotamartos]

Changes LGTM but please undo the change I mentioned in my comment

[james-mchugh]

Hello @jotamartos. I fixed the merge conflicts and responded to your request. Please let me know if there is anything additional needed.

[jotamartos]

Sorry for the delay here. I added more information about why the symlink is lost. Would you mind updating the logic and the PR so everyone can benefit from the change?

Thanks

[jotamartos]

As a final note, the symlink is lost because we are mounting an empty volume to follow best security practices. There are many references to /usr/local/apisix in the code so we will keep the suggested changes and revisit the symlink configuration in the future.

[james-mchugh]

@jotamartos Thank you for reviewing and approving my PR!

[vivekhegadems]

Please provide the timeline on when the next release of apisix will happen from bitnami with this standalone deployment fix?

[carrodher]

The new version was released right after the PR was merged, see https://hub.docker.com/layers/bitnamicharts/apisix/3.2.5/images/sha256-b52d44638bc1cc71ff9656a4073fe3878d955ef5f2c087e57a41c439157e1e55?context=explore