puppetlabs / puppetserver-helm-chart

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

Fix permission issue and template issues #216

Closed johannagnarsson closed 6 months ago

johannagnarsson commented 6 months ago

First two changes will fix the puppetdb-preinit container from failing with a mkdir: cannot create directory ‘/opt/puppetlabs/server/data/puppetdb/certs’: Permission denied when puppetmasters.preGeneratedCertsJob.enabled is set to true. as it tries to run as non-root by default even if it's disabled. Second change fixes a broken reference to the puppetmaster-preinstall configmap which prevents the server from starting up correctly when puppetmasters.preGeneratedCertsJob.enabled is set to true.

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.

johannagnarsson commented 6 months ago

Changes made as requested! @Xtigyro

johannagnarsson commented 6 months ago

Result below, I do see the same fails on the upstream repo as well.

Chart [ puppetserver ] .

FAIL test jmx serviceMonitor creation tests/jmx-servicemonitor_test.yaml

Charts: 1 failed, 0 passed, 1 total Test Suites: 6 failed, 8 passed, 14 total Tests: 6 failed, 65 passed, 71 total Snapshot: 12 passed, 12 total Time: 334.573499ms

Error: plugin "unittest" exited with error

Xtigyro commented 6 months ago

@johannagnarsson That's fine. Thank you once more!