Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new role for "limesurvey" #373

Open
stefangweichinger opened this issue Jun 12, 2017 · 3 comments
Open

new role for "limesurvey" #373

stefangweichinger opened this issue Jun 12, 2017 · 3 comments

Comments

@stefangweichinger
Copy link

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
Copy link
Contributor

ganto commented Jun 13, 2017

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
Copy link
Author

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
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants