Upgrade fail with userdel: user matrix-synapse is currently used by process 1621. It's quite bad because the upgrade will fail and then app will be completely removed and so after the restore will fail because some part of the migration was already done. So we must avoid as much as possible that the upgrade fail.
Solution
Use the fix from #449 and also a6c090bd9b5d6116318c511b9b0f890ed88cf297
Fix upgrade crash by bb18bd7fb2cb194e770a8966a69c59151d290939
PR Status
[x] Code finished and ready to be reviewed/tested
[x] 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)
Problem
userdel: user matrix-synapse is currently used by process 1621
. It's quite bad because the upgrade will fail and then app will be completely removed and so after the restore will fail because some part of the migration was already done. So we must avoid as much as possible that the upgrade fail.Solution
PR Status
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)