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

Option to restart applications if method known #52

Closed
tsujamin opened this issue Jan 29, 2016 · 7 comments
Closed

Option to restart applications if method known #52

tsujamin opened this issue Jan 29, 2016 · 7 comments
Labels

Comments

@tsujamin
Copy link

For machines that run automatic security updates, it would be useful if tracer were configurable to execute service restarts for daemons it knows about. This could be enabled by a command line switch, or a configuration file for the dnf-plugin (with white listing of particular packages).

Not sure if this is a function you see tracer fulfilling, but I might hack on it a bit if I find some time.

@FrostyX
Copy link
Owner

FrostyX commented Jan 29, 2016

I personally haven't needed this, but I agree that it can be nice in some scenarios. Feel free to send PR if you want to code it by yourself :-). Or I will do it, but it can take some while.

@FrostyX FrostyX changed the title Feature: Option to restart applications if method known Option to restart applications if method known Jan 29, 2016
@tsujamin
Copy link
Author

I've started on an implementation (https://github.com/tsujamin/tracer) which adds a new Controller/View for restarting services and the commandline flags "--restart-daemons" and "--restart-services"

I've tried to mock up some unit tests for the controller, which appear to pass, but I haven't really been able to test it in practice as I don't have any daemons that need restarting ;) How should I go about testing this properly?

I'll probably look into integrating this with the dnf-plugin, but it appears that's in a separate project at the moment. If it were to go that far, being able to enable/disable the functionality and blacklist certain packages from automatic restarts would be something I'd look to implement (probably with ConfigParser or something similar), but I figured I'd run this past you before going any further :)

@gsauthof
Copy link
Contributor

FWIW, while waiting on this feature I implemented an alternative stale-process-checker that supports automatic restarts via --restart: https://github.com/gsauthof/utility/#oldprocs

@tsujamin
Copy link
Author

tsujamin commented Dec 31, 2021 via email

@FrostyX
Copy link
Owner

FrostyX commented Jan 3, 2022

I've started on an implementation
(https://github.com/tsujamin/tracer) which adds a new
Controller/View for restarting services and the commandline flags
"--restart-daemons" and "--restart-services"

Ah, I am sorry, I somehow missed the comment. I guess it's not
relevant anymore.

FWIW, while waiting on this feature I implemented an alternative
stale-process-checker that supports automatic restarts via --restart:
https://github.com/gsauthof/utility/#oldprocs

Perhaps I should add a section with known alternatives to the README
section and mention your tool?

@tsujamin tsujamin closed this 4 days ago

Also, if you don't mind, I would keep this issue open. It's a valid
RFE.

@FrostyX FrostyX reopened this Jan 3, 2022
@gsauthof
Copy link
Contributor

Perhaps I should add a section with known alternatives to the README
section and mention your tool?

Sure, adding a section that puts a project into perspective like that usually is a good idea.

@FrostyX
Copy link
Owner

FrostyX commented Feb 1, 2022

@gsauthof I added a Similar software section to the README in b390877, feel free to edit it or list other alternatives.

@tsujamin tsujamin closed this as completed Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants