Skip to content

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

@johnwarburton

Description

@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

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions