search

puppetlabs / puppetserver-helm-chart

The Helm Chart for Puppet Server
Apache License 2.0
52 stars 56 forks source link

runAsNonRoot #161

Closed antwacky closed 1 year ago

antwacky commented 1 year ago

Describe the Bug

The runAsNonRoot option does not appear to work. When enabling this option, the following error is seen on the pre-install hook batch/Job/puppetserver-puppet-preinstall:

Warning  FailedMount             61s (x8 over 2m4s)  kubelet                  MountVolume.SetUp failed for volume "r10k-code-volume" : configmap "puppetserver-r10k-code-config" not found

I assume this is because the configmap is not created until the preinstall hook has completed.

Expected Behavior

The preinstall hook should complete successfully.

Steps to Reproduce

Add the runAsNonRoot value to deploy the chart:

global:
  runAsNonRoot: true

Environment

Additional Context

Nada

davidphay commented 1 year ago

Yes I am working on it, for now if you want continue, you can deploy r10k as container with r10k.asSidecar: false it should work.

davidphay commented 1 year ago

@pagey101 can you test the latest release please and tell me if the issue is fixed ? https://github.com/puppetlabs/puppetserver-helm-chart/pull/162

davidphay commented 1 year ago

I close the issue, feel free to open it again if it not work as expected

antwacky commented 1 year ago

Apologies, I haven't got round to this yet! Thanks for the fix though.