Installation fail: synapse-ynh-deps doesn't install correctly about synapse_ynh HOT 12 CLOSED

Hello,
I it look like a problem with the statoverride file. What did you do ? Have you tried to install the old synapse package and after did you try to install the new version without the upgrade script ?

If it is the case I know this issue and it's the reason of these line here (in upgrade script) : https://github.com/YunoHost-Apps/synapse_ynh/blob/master/scripts/upgrade#L47-L50

I try just to understand why do you have this error.

I all case mybe you need to put these command manually to solve this issue : https://github.com/YunoHost-Apps/synapse_ynh/blob/master/scripts/upgrade#L47-L50

from synapse_ynh.

I removed synapse, and rebooted, before trying to install again (I'm changing the domain and so on).

I just did a yunohost app install synapse (and also tried with this repo url, same result)

from synapse_ynh.

In this case it is exactly what a thought. You need to type this command :

sudo sed --in-place ':a;N;$!ba;s@matrix-synapse nogroup 755 /var/lib/matrix-synapse\n@@g' /var/lib/dpkg/statoverride
sudo sed --in-place ':a;N;$!ba;s@matrix-synapse nogroup 755 /var/log/matrix-synapse\n@@g' /var/lib/dpkg/statoverride
sudo sed --in-place ':a;N;$!ba;s@matrix-synapse nogroup 755 /etc/matrix-synapse\n@@g' /var/lib/dpkg/statoverride

Or just remove every line in relation with matrix in : /var/lib/dpkg/statoverride

And after install the app by the URL (without URL the old version will be installed).

from synapse_ynh.

It worked, thanks a lot :)

And after install the app by the URL (without URL the old version will be installed).

Why ? The reference in the community app list is not the right one ?
(in that case, we should make a PR to change it, don't you think so ?)

from synapse_ynh.

Because, the migration script (from old version) is not really really stable (I see some Issue about that !!). So while it is not really stable I won't update the link (in community list). I think that if I update it a lot of people will update app in the same time and if I break anything on a lot of instance I wont be good. So I prefer so solve any issue and after I will update it.

But mybe it's not the best idea, so if you have any other idea, say me.

from synapse_ynh.

Because, the migration script (from old version) is not really really stable (I see some Issue about that !!).

Ok, I didn't know that. Then it looks very reasonable.
Maybe you can change the update script to create a backup before upgrading the app ?
(this might be very useful for some people in case of problem)

As my issue is solved, I'm closing. Thanks :)

from synapse_ynh.

Yes, I will implement this, but I don't know I it's enough to solve everything

from synapse_ynh.

At least it will avoid big troubles in case of trouble.

from synapse_ynh.

The problem is that after if it fail, the restoration also fail.

from synapse_ynh.

With the new version only (because of the database change) or in general ?

from synapse_ynh.

The upgrade script change something in debian package (remove synapse package and reinstall coturn as dependence), but I saw sometime that the install of dependence fail but I don't really understand why. After sometime apt fail if we don't debug it. And in this case you can't just say I reinstall the backup. But in all case I did really understand what does it happen.

But in all case saw something like the helper ynh_install_app_dependencies return 0 (the command don't fail) but the dependance is not installed. But I think that it's more in the yunohost side. What do you think ?

from synapse_ynh.

I have no clue, sorry. Maybe you can get in touch with Yunohost's team to get more details regarding this helper's behavior ?

from synapse_ynh.