-
Notifications
You must be signed in to change notification settings - Fork 275
Conversation
Systemd is a hard requirement for roles in cloudalchemy. Plan is to use even more systemd features and this would conflict with this change. More in #161 (comment) |
I understand where you are coming from but systemd and other service managers can coexist in the role quite easily, the systemd specific tasks aren't that many in the role and any future systemd specific ones could simply have a This pull request already addresses the points you make in #161 (comment)
Addressed in: 0c6e877
Addressed in: 7d60fe7
When that happens we can simply add a
I don't see how the CLI flags are a systemd specific option, the templates I created for upstart/sysvinit use the CLI flags, just as the systemd service does. |
Don't get me wrong, I don't have anything against other init systems. My concern is only about the future of the role and the maintenance toll other init systems bring into this. Which in this case is substantial and can cause confusion in the long run. For example, I am trying in my free time to revive #141 which would bring support for textfile collectors into the role, but it is heavily dependant on systemd timers as those offer a much greater degree of control than simple crontab. Such a feature simply isn't possible in sysvinit or upstart. Hence I am willing to merge this, but under 2 conditions:
Additionally to above, CI tests for this feature either need to be switched to a supported system (ubuntu 14.04 is EOL since April 30th 2019) or disabled. I have no intention of supporting an OS just for CI tests :) |
Like @paulfantom said, 14.04 is EoL. Also CentOS 6 is going EoL in less than a month. Adding support for EoL systems is a non-goal of this project. |
I understand that it can add some maintenance as different init systems support different things (otherwise we wouldn't have multiple init systems in the first place right? :) ). Currently the systemd/upstart/sysvinit templates all provide a pretty close functionality but if at some point it will move so far apart that it wouldn't make sense to maintain them then this should perhaps be re-evaluated, but let's just cross that bridge when we come to it. (At that point systemd might be even more wide spread than it is now). Stating that support for non-systemd devices is experimental/without warranty/at own risk in the readme is acceptable to me, a debug message containing the same warning is fine too, and if you want to take it one step further we could even have a bool variable ( I debated with myself if I should include a dockerfile with centos8/debian10 with sysvinit/upstart or if using a legacy version that came with sysvinit/upstart would make more sense, or even if I should introduce a new distro that ships with sysvinit/upstart that's not already present in the tests. The reason why I chose CentOS 6 even though it is close to EOL is that rhel6 is still getting extended support from RedHat and will do so until mid year 2024, but I don't think we can run the tests on a rhel6 container without license so running the tests against CentOS 6 would be the next best thing. I don't think running tests on centos 8 with sysvinit will guarantee that the role works on rhel 6. (Believe me I wish I would have the luxury of only maintaining the latest distro versions :) but with this at least I'm not stuck with a legacy monitoring system) |
If we're going to support another process supervisor, supporting Podman/Docker would be preferred over sysvinit or upstart. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
incorporate upstream changes
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This role has been deprecated in favor of a the prometheus-community/ansible collection. |
Adds support for devices with sysvinit and upstart service managers, made the service templates to be as close to the systemd service as possible.
Been using this internally for months with good success.
I know sysvinit/upstart might be considered legacy by many but I believe this could benefit others with adopting prometheus in their environments.