by Luc Saffre · in Developers · assigned to khchine5

Summary

Children

Comments

• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Add more test applications to the test suite
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Going to move the socket generated file to /tmp/ for permissions issues.
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Check existence of /usr/bin/mysql_config before adding the symbolic link
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Fix typo with apt_packages of DbEngine. Release 20.4.5 version to PyPI.
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Getting error: No module named 'lino_local.settings
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : certbot-auto should be run with sudo.
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Not need anymore to get certificate with certob with www version. We use just the domain without the www
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : I'd say that the healthcheck.sh script needs to get optimized so that it reports any "anormal" output of "supervisorctl status".
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : No, adding "user = {usergroup}" to LIBREOFFICE_SUPERVISOR_CONF doesn't work, so I didn't push this. I guess that libreoffice must run as root, and the supervisor warning is simply too paranoid.
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : Oops, I saw that healthcheck.sh already does sudo. Seems that I had an old version installed on my machine. But what happens under Docker when a non-root user runs sudo? Maybe the problem is there. Our trick of using awk to analyze the output is definitively poor because it hides possible error mess...
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : "sudo service supervisor status" gives a warning "supervisord[7307]: CRIT Supervisor running as root (no user in config file)", so I added a line "user = {usergroup}" to LIBREOFFICE_SUPERVISOR_CONF. Not sure whether this will work.
  
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : I think you don't need to wait 10 seconds. The explanation is much easier: healthcheck.sh works only when invoked by root. Look at what the script does. When you aren't root, supervisorctl gives some error message and our awk command is not designed to recognize this.
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : I need to fix these points (after testing with avanti new demo server): mysql or mariadb for debian 10
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Need to restart supervisor after adding libreoffice config
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Need to wait 10 sec until the supervisor finish restarting
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : Ideas : 1) I can reproduce the problem on my machine: $ sudo service supervisor restart ; healthcheck.sh Checking supervisor status: ERROR: unix:///var/run/supervisor.sock is not running $ healthcheck.sh Checking supervisor status: ERROR: error: is not running Looks as if the restart command termi...
• 6 years, 6 months ago by khchine5 about #3300 (Increase getlino test coverage) : Can't find a way to force supervisor to restart.
• 6 years, 6 months ago by Luc Saffre about #3300 (Increase getlino test coverage) : Note that i.must_restart("supervisor") doesn't actually restart it. We splitted i.finish() into i.run_apt_install() and i.restart_services().
• 6 years, 7 months ago by khchine5 about #3300 (Increase getlino test coverage) : The pull.sh script doesn't work correctly . The paths of the repositories are wrong
• 6 years, 7 months ago by khchine5 about #3300 (Increase getlino test coverage) : Use buster-backports to install monit for debian from deb ftp.de.debian.org/debian buster-backports main