Bazarr does not restart successfully upon applying update about bazarr HOT 14 CLOSED

Yes but it will be released tonight.

from bazarr.

Looks like the update from beta 22 to beta 23 went fine here. Bazarr process was running after the update.

from bazarr.

Automatic update and restart went fine from beta 23 to beta 24 as well.

from bazarr.

Should be fixed in upcoming release. Let me know if it works properly after being updated to beta.21.

from bazarr.

@morpheus65535 I'll have to update to beta.21 and wait until beta.22 to properly verify this, right?

from bazarr.

I went from beta 21 and am on beta 22 right now, but it was not starting by itself. Got the error message below.
Actually, I have had this problem for quite many beta releases now, where I notice the bazarr service is not working sometimes, and I have to manually restart it by killing the nssm process and starting again. Just recently I am starting to relate it to new releases and the updating process.
2024-04-20 09:08:45|ERROR |root |BAZARR Cannot create restart file: KeyError('RESTARTFILE')|

from bazarr.

@flensburg2 can you check if bazarr.py contain those lines? https://github.com/morpheus65535/bazarr/blob/development/bazarr.py#L122-L125

from bazarr.

I can confirm I find the same lines 122-125 in my bazarr.py file

from bazarr.

Still having the same issue going from .21 to .22.

I'm not too familiar with bazarr source or its upgrade flow, but does a new process get created during upgrades? Would that be why there is no os.environ value for RESTARTFILE?

from bazarr.

Previously we weren't defining environment variables in bazarr.py which is only a daemon process. It start a subprocess (main.py) and this one may not be getting parent process environment variables. My latest fix should address that (I hope...).

from bazarr.

Looks like the update from beta 22 to beta 23 went fine here. Bazarr process was running after the update.

Keep me informed on how goes subsequent upgrades.

from bazarr.

@cheng2wei is it fixed for you?

from bazarr.

Yep, auto update and restart is working. On beta 24 now.

from bazarr.

Thanks!

from bazarr.