[Bug]: set -e added to 30-set-permissions.sh causes script failure when using mounted configmap configuration files

[johnwarburton]

Is this a critical security issue?

• This is not a security issue.

Describe the Bug

Hi

I'd like to re-open the issue previously at voxpupuli/container-puppetserver#126

I am attempting to use a customised auth.conf using the configmap feature introduced in puppetlabs/puppetserver-helm-chart#125

When I add my custom auth.conf, I get an error

openvoxserver-puppetserver Running /docker-entrypoint.d/30-set-permissions.sh
. . .
openvoxserver-puppetserver chown: changing ownership of '/etc/puppetlabs/puppetserver/conf.d/auth.conf': Read-only file system

I will propose in the related PR to remove the set -e

Thanks

John

Expected Behavior

/docker-entrypoint.d/30-set-permissions.sh should run and not fail

Steps to Reproduce

add to `values.yaml

puppetserver:
  customconfigs:
    enabled: true
    configmaps:
      auth.conf: |-
        authorization: {
          version: 1
          rules: [
            {
                   # Allow everything from everyone
                   match-request: {
                       path: "/"
                       type: path
                   }
                   allow-unauthenticated: true
                   sort-order: 100
                   name: "allow all"
            },
          ]
        }

Environment

Helm Chart https://github.com/OpenVoxProject/openvox-helm-chart/releases/tag/v10.0.0

puppetserver:
  name: puppetserver
  image: ghcr.io/openvoxproject/openvoxserver
  tag: 8.8.0-main
  pullPolicy: IfNotPresent

Additional Context

No response

Relevant log output