Skip to content
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

change stopping tracim strategy #184

Merged
merged 5 commits into from
Feb 6, 2017
Merged

Conversation

buxx
Copy link
Contributor

@buxx buxx commented Feb 5, 2017

Avec ces changements Tracim se ferme enfin correctement:

  • Le worker RQ se termine également correctement: J'ai procédé à une astuce : on passe l'attribut du worker, dans le daemon,_stop_requested à vrai depuis le main thread. Et un surcharge dans le worker vérifie cet attribut entre chaque "traitement" de job. On fait par la suite faire au worker un job "bidon" (d'ailleurs il y a un job bidon à disposition dans le code de rq, à se demander si ça sert pas à un truc similaire), notre surcharge lève par la suite un StopRequested qui applique la procédure standard d'arrêt du worker.
  • Une fois les démons arrétés on fait un kill sur le processus de tracim. Après avoir essayé de trouver un moyen de stopper "softwarement" le process du serveur WSGI je me suis randu compte que ce n'était peut-être pas vraiment possible: le serevur étant créé/géré à un niveau plus élevé, par uwsgi ou apache2 par exemple. D'ailleurs, en fouillant dans les commande gearbox j'ai vu qu'il faisait des kill lui aussi.
    Tu me dira ce que tu pense de cette stratégie et si ça mérite d'être intégré dans la beta 1.0 : )

Copy link
Contributor

@lebouquetin lebouquetin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you rename "send_callable" in "sendmail_callable" ?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 65.322% when pulling 6bc5d2d on buxx:166/dev/signals into b3a0054 on tracim:master.

1 similar comment
@coveralls
Copy link

coveralls commented Feb 6, 2017

Coverage Status

Coverage decreased (-0.02%) to 65.322% when pulling 6bc5d2d on buxx:166/dev/signals into b3a0054 on tracim:master.

@buxx
Copy link
Contributor Author

buxx commented Feb 6, 2017

Renommage effectué et doc ajouté.

@coveralls
Copy link

coveralls commented Feb 6, 2017

Coverage Status

Coverage decreased (-0.02%) to 65.322% when pulling 17a300d on buxx:166/dev/signals into b3a0054 on tracim:master.

@tracim tracim merged commit 90374bd into tracim:master Feb 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants