search

fluent / fluent-package-builder

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

rpm: take over enabled state of systemd service from v4 #613

Closed kenhys closed 4 months ago

kenhys commented 5 months ago

In the previous versions, the enabled service status of td-agent was not migrated.

This commit try to take over existing td-agent service status.

Note that if systemctl enable fluentd is executed in %post section, it was reverted in td-agent's %preun scriptlet. So it is enabled in %posttrans explicitly.

Closes: #610

kenhys commented 5 months ago

enable state will be kept like this:

  Running scriptlet: td-agent-4.5.2-1.el8.x86_64                                                                                        2/2 
  Running scriptlet: fluent-package-5.0.2-2.el8.x86_64                                                                                  2/2 
Provides /usr/sbin/td-agent symlink for backward compatibility
Provides /usr/sbin/td-agent-gem symlink for backward compatibility
Provides /etc/td-agent symlink for backward compatibility
Provides /var/log/td-agent symlink for backward compatibility
td-agent was enabled. Take over enabled service status ...
Created symlink /etc/systemd/system/td-agent.service → /usr/lib/systemd/system/fluentd.service.
Created symlink /etc/systemd/system/multi-user.target.wants/fluentd.service → /usr/lib/systemd/system/fluentd.service.

  Running scriptlet: td-agent-4.5.2-1.el8.x86_64                                                                                        2/2 
  Verifying        : fluent-package-5.0.2-2.el8.x86_64                                                                                  1/2 
  Verifying        : td-agent-4.5.2-1.el8.x86_64                                                                                        2/2 

Installed:
  fluent-package-5.0.2-2.el8.x86_64                                                                                                         

Complete!
[vagrant@almalinux8 ~]$ sudo systemctl is-enabled fluentd
enabled
kenhys commented 5 months ago

614 should be fixed in advance.

kenhys commented 4 months ago

NOTE: https://chocolatey.org/ is under maintenance.

so https://chocolatey.org/install.ps1 is not accessible for a while. (windows CI fails)

daipom commented 4 months ago

Thanks!