Use grep for setting fetch in cron job

alexAubin commented 3 years ago

Problem

In multiinstall context, because multiple cron job are triggered at the exact same time, there's a weird lock issue going on resulting in the cron job reporting: Uhoh, somehow the lock /var/run/moulinette_yunohost.lock did not exist ...

Solution

I guess the lock mecanism should be improved in many ways ... But in the meantime, here's a fix using a simple grep | awk instead of yunohost app setting get

PR Status

[ ] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

alexAubin commented 3 years ago

!testme

yunohost-bot commented 3 years ago

:rocket:

YunoHost-Apps / borgserver_ynh

Use grep for setting fetch in cron job #49

Problem

Solution

PR Status

Automatic tests