-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Death of the task_manager process #681
Comments
Looks like the task manager should check if the list still exists and discard the task if the list is gone. |
Hello,
That's what I did. But there should be no stopping of the task_manager process in this context. Do not treat a task for a list that no longer exists, in my opinion, directly in the code.
Cordially
…---
Benoît MARCHAL
Sous direction Infrastructure & Services
Direction du Numérique
Université de Lorraine
Tél : 03.72.74.11.39
mél : Benoit.Marchal@univ-lorraine.fr
----- Le 3 Juil 19, à 8:31, Stefan Hornburg (Racke) <notifications@github.com> a écrit :
Looks like the task manager should check if the list still exists and discard
the task if the list is gone.
—
You are receiving this because you authored the thread.
Reply to this email directly, [
#681?email_source=notifications&email_token=AINKMBOWQZW7YUG2KCUHBI3P5RBTHA5CNFSM4H5CKDSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZDNOYQ#issuecomment-507959138
| view it on GitHub ] , or [
https://github.com/notifications/unsubscribe-auth/AINKMBKMPDATZ2CXSUXUTM3P5RBTHANCNFSM4H5CKDSA
| mute the thread ] .
|
Hi @bmarchal54, |
Hello,
Thank you for the patch proposal. I just put it in place. I will follow in the day if the problem comes back and I'll let you know.
Have a good day
…---
Benoît MARCHAL
Sous direction Infrastructure & Services
Direction du Numérique
Université de Lorraine
Tél : 03.72.74.11.39
mél : Benoit.Marchal@univ-lorraine.fr
----- Le 7 Juil 19, à 13:45, IKEDA Soji <notifications@github.com> a écrit :
Hi [ https://github.com/bmarchal54 | @bmarchal54 ] ,
Could you please apply [
https://github.com/sympa-community/sympa/commit/245428aa3fcc5588361f27370e1df6917e7f3f63.patch
| this patch ] and check if the problem will be solved? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, [
#681?email_source=notifications&email_token=AINKMBJNZMXGVRQI5FTHQ3DP6HJPPA5CNFSM4H5CKDSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLJ4IY#issuecomment-508993059
| view it on GitHub ] , or [
https://github.com/notifications/unsubscribe-auth/AINKMBK5HRQXPT6RJWILYZ3P6HJPPANCNFSM4H5CKDSA
| mute the thread ] .
|
Hello,
Since the passage of the patch proposed last Sunday, I did not stop the task_manager process.
Thank you so much.
cordially
…---
Benoît MARCHAL
Sous direction Infrastructure & Services
Direction du Numérique
Université de Lorraine
Tél : 03.72.74.11.39
mél : Benoit.Marchal@univ-lorraine.fr
----- Le 7 Juil 19, à 13:45, IKEDA Soji <notifications@github.com> a écrit :
Hi [ https://github.com/bmarchal54 | @bmarchal54 ] ,
Could you please apply [
https://github.com/sympa-community/sympa/commit/245428aa3fcc5588361f27370e1df6917e7f3f63.patch
| this patch ] and check if the problem will be solved? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, [
#681?email_source=notifications&email_token=AINKMBJNZMXGVRQI5FTHQ3DP6HJPPA5CNFSM4H5CKDSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLJ4IY#issuecomment-508993059
| view it on GitHub ] , or [
https://github.com/notifications/unsubscribe-auth/AINKMBK5HRQXPT6RJWILYZ3P6HJPPANCNFSM4H5CKDSA
| mute the thread ] .
|
ikedas
added a commit
that referenced
this issue
Jul 11, 2019
task_manager.pl dies when the list no longer exist (#681)
I merged fix, and it will be included in the next release. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Since switching to version 6.2.42, I have regularly stopped the task_manager process.
When I look at which list that might correspond, I suspect that it corresponds to a list that has been deleted between the definition of the task and the launch of it.
Version
6.2.42
Installation method
yum package
Error message in sympa.log
List of task to execute
But in the directory /var/lib/sympa/list_data/etu.univ-lorraine.fr :
The list all-lettres-l-3-ncy-et-culturelles-fi-1920 t does not exist anymore
Thanks in advance
The text was updated successfully, but these errors were encountered: