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

trytond: Add final init hooks [CUSTOM] #374

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JCavallo
Copy link
Contributor

Copy link
Collaborator

@nicoe nicoe left a comment

Choose a reason for hiding this comment

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

N'est-il pas aussi nécessaire de faire un appel à setup_complete quand on fait un upgrade via la ligne de commande (trytond-admin) ?

@JCavallo
Copy link
Contributor Author

@nicoe Pas nécessaire je pense, dans ce cas on va propager un refresh de la pool sur les process existants, qui lui pourra trigger

@nicoe
Copy link
Collaborator

nicoe commented Dec 20, 2024

@nicoe Pas nécessaire je pense, dans ce cas on va propager un refresh de la pool sur les process existants, qui lui pourra trigger

Disons que ça dépend fatalement de ce qu'on fait dans ces hooks mais si ils ont un effet de bord ce ne serait pas safe de le faire au cas où ?

@JCavallo
Copy link
Contributor Author

En relisant, a priori setup_complete est appelé dans tous les cas à la fin de load_module_graph, donc il devrait être trigger lors de l'upgrade en ligne de commande.

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.

2 participants