voxpupuli / puppet-confluence

A puppet module to install confluence
https://forge.puppet.com/puppet/confluence
MIT License
21 stars 76 forks source link

$data_dir should be documented in ReadMe #129

Closed emoon3 closed 7 years ago

emoon3 commented 7 years ago

When trying to install Confluence 6.1.1, I kept getting "spring application context has not been set" errors when starting confluence.

It turns out that the $data_dir variable is set to undef unless you set it with a variable. As the module stands now, it seems like this MUST be set in order for it to work.

But there is no mention of it in the documentation.

Tokynet commented 7 years ago

I believe what you are seeing is the error caused by the typo mentioned here

For me, if I apply the basic class w/o defining anything else, the $data_dir does get populated (but i'm pulling master branch of this repo instead of the tag, since the fix is yet to make it to a tag release)

emoon3 commented 7 years ago

Ahh, ok. Thanks.

juniorsysadmin commented 7 years ago

Fixed by #128