[pve-devel] applied: [PATCH manager 2/7] postinst: use reload-or-restart instead of reload-or-try-restart

Thomas Lamprecht t.lamprecht at proxmox.com
Thu Oct 7 10:38:46 CEST 2021


we normally use `d/<foo>` for changes to the debian/ folder, iow, packaging changes.

On 07.10.21 10:27, Dominik Csapak wrote:
> the only difference is that reload-or-try-restart does not
> do anything if the service is not started already.
> 
> on upgrade, we explicitely check if the service is enabled, and only

s/explicitely/explicitly/

> then do this action. so it would now start daemons that were
> stopped but not disabled (which is not really valid  state anyway)
> 
> this starts new daemons (like the pvescheduler) automatically on
> a package upgrade
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  debian/postinst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

with a few brushups to the commit message (and I took the freedom to
add a suggested-by me ;): applied, thanks!





More information about the pve-devel mailing list