-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add operations for FreezeUnit and ThawUnit #489
Conversation
hi @irishair First of all, many thanks for this PR and your interest in the project! I just wanted to flag that we're currently in the process of changing the license of the project from GPLv2+ to LGPLv2.1+: (cf #462) as well as contributing it to the Eclipse Foundation for it to become part of the Eclipse SDV umbrella. Thanks in advance! :) |
Need fixed for bluechi-ctl as well. |
as in, could you also document these features in https://github.com/containers/bluechi/blob/main/doc/man/bluechictl.1.md ? |
21b09fb
to
ba07513
Compare
I updated bluechictl.1.md. |
Yes, we are fine with license change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, great work!
@irishair Could you fetch the latest changes from main? Then the 2 failing integration tests should pass as well and we could merge.
It would be great to have integration tests for these new features, but that can be done in a separate PR - I'll create an issue for it.
Update: Created #495
- Add FreezeUnit/ThawUnit operations to agent and manager - Add freeze/thaw commands to client - Add FreezeUnit/ThawUnit interfaces to DBus-API Signed-off-by: minsoo.kim <[email protected]>
ba07513
to
a4eafad
Compare
fetched latest changes, and I'll follow up on the issue for integration test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM @irishair should we consider a RFE for
|
I'll apply this to |
Good idea @dougsland!
|
We are trying to use bluechi for multi-node container ochestration in our automotive HPC.
In our system, we have additional requirements for suspend and resume of units.
From systemd 246, it supports "systemctl freeze/thaw" and D-Bus APIs for those based on cgroup v2.
So we added a new feature that supports freeze and thaw units on multi-node to bluechi, and we would like to submit this patch.
This PR contains following changes.
Example test results