new role for "limesurvey"
stefangweichinger opened this issue · 3 comments
stefangweichinger commented
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
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
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