◄ ► | Bearbeiten | Startseite | Admin | en de fr et | Anmelden

#3300 (Increase getlino test coverage)

by Luc Saffre · in Developers · assigned to khchine5

Kurzbeschreibung

Kinder

Kommentare

  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Add more test applications to the test suite
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Going to move the socket generated file to /tmp/ for permissions issues.
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Check existence of /usr/bin/mysql_config before adding the symbolic link
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Fix typo with apt_packages of DbEngine. Release 20.4.5 version to PyPI.
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Getting error: No module named 'lino_local.settings
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : certbot-auto should be run with sudo.
  • 6 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Need to restart supervisor after adding libreoffice config
  • 6 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Need to wait 10 sec until the supervisor finish restarting
  • 6 Jahre, 6 Monate ago von 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 Jahre, 6 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : Can't find a way to force supervisor to restart.
  • 6 Jahre, 6 Monate ago von 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 Jahre, 7 Monate ago von khchine5 about #3300 (Increase getlino test coverage) : The pull.sh script doesn't work correctly . The paths of the repositories are wrong
  • 6 Jahre, 7 Monate ago von 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