-
Notifications
You must be signed in to change notification settings - Fork 15
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
Removing units #25
Comments
With 05b0393 there is some support for removing units. It seems to work kinda. After reloading and restarting the test units there seems to be quite a lot of issues though. It seems more related to handling panicing services though. For now it would be best not to use this feature. |
In general this and this issue about reloading units can be solved by implementing a few (but a bit complicated) operations on unit sets.
This should also work for adding new units. How to handle changed units is still not entirely clear to me. Some of this already exists but I plan on making this more in style of a transaction on the set of units. |
I think this issue is fixed. To refactor/redesign the internal handling should be tracked in another issue. |
There should be a call to the control interface like this:
{"method": "disable", "params": ["test.service", "test.socket"]}
. This should remove all units that reference these units too.After that a simple
{"method": "reload"}
should be enough to reload the (possibly changed) unit files. Then the units need to be restarted. For a first start manual reactivation should be fine, a command like{"method": "activate-all-new"}
would be nice to have though.The text was updated successfully, but these errors were encountered: