search

fluent / fluent-package-builder

td-agent (Fluentd) Building and Packaging System
Apache License 2.0
22 stars 26 forks source link

deb: ensure to reload unit file before restarting #512

Closed kenhys closed 1 year ago

kenhys commented 1 year ago

It fixes the following case:

  Failed to restart fluentd.service: Unit fluentd.service not found.
  dpkg: error processing package fluent-package (--configure):
   installed fluent-package package post-installation script subprocess returned error exit status 5
kenhys commented 1 year ago

still this failure is not reproduced on local VM yet.

daipom commented 1 year ago

Hmm, still failing? I will check this problem too.

daipom commented 1 year ago

On ubuntu-focal, test -e /etc/systemd/system/td-agent.service fails. This means there is no td-agent.service file after removing.

https://github.com/fluent/fluent-package-builder/actions/runs/5560646924/jobs/10158772068?pr=512

daipom commented 1 year ago

test -e /etc/systemd/system/td-agent.service fails. This means there is no td-agent.service file after removing.

https://github.com/fluent/fluent-package-builder/actions/runs/5560646924/jobs/10158772068?pr=512

It is because the file is linked to a missing file and is judged as not existing by -e. I have fixed to use -h.