new role for "limesurvey"

debops / debops-playbooks

Ansible playbooks used by DebOps project

GNU General Public License v3.0

489 stars 88 forks source link

Open stefangweichinger opened 7 years ago

stefangweichinger commented 7 years ago

Tried to adapt/write a role for installing limesurvey application. Current state of draft: https://github.com/stefangweichinger/ansible-limesurvey

missing: it's mariadb-only for now please review and advise

ganto commented 7 years ago

Hi, I had a quick look, here a few things that I found:

naming conventions: ok 👍
for password management DebOps rules usually use debops.secret via hard-dependency
any way to validate the download archive? See Software Source Policy
apache restart handler is redundant
where do the SQL script/templates come from?
handle limesurvey version updates
docs are missing, see e.g. debops.owncloud: docs for a good example

stefangweichinger commented 7 years ago

thanks for the initial check and the feedback. Sounds as if there is still much work to do ;-)

password: you mean setting the default pw in the DB ?
validation of archive: I will try to check the hash etc
SQL script/templates come from the repo mentioned in the top README : https://github.com/ICTO/ansible-limesurvey
version updates: how to approach that?
docs: sure :-P

stefangweichinger commented 7 years ago