search

hubspotdevops / puppet-nexus

Puppet module for Sonatype Nexus
MIT License
24 stars 93 forks source link

Make sure that the nexus_root directory exist #21

Closed feniix closed 8 years ago

feniix commented 10 years ago

if the nexus_root directory does not exist (e.g. $nexus_root = '/srv/nexus') then the module fails.

if the nexus_root directory is /srv then nexus logs the exceptions below because it cannot write to the ${user.home} (/srv) that is owned by root. (see this: https://issues.sonatype.org/browse/NEXUS-3671 )

so to avoid that I added logic to make sure the module has this new functionality while at the same time keep backwards compatibility.

tmclaugh commented 9 years ago

$nexus_root in our environment is already a managed resource so this breaks for me. I probably should have made ${nexus_root}/${nexus_home_dir} a single parameter and used some functions to split the name into componants. I probably wasn't working with the latest stdlib at the time and did the best I could with the older stdlib module I was using. I'd be open to this sort of change.