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.

Sign up for GitHub

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

Jump to bottom

Add a new service plugin #51

Merged
merged 2 commits into from
Feb 21, 2025
Merged

Add a new service plugin #51

merged 2 commits into from
Feb 21, 2025

Conversation

Wescoeur
Copy link
Member

@Wescoeur Wescoeur commented Feb 19, 2025 •
edited
Loading

Also remove useless sys path modification in many plugins.

@Wescoeur Wescoeur requested review from gthvn1 and Nambrok February 19, 2025 13:14
@gthvn1
Copy link
Contributor

gthvn1 commented Feb 19, 2025

Will it be used by XO to query services for example?

@Wescoeur
Copy link
Member Author

I will be used to deal with XOSTOR in the RPU context.
We just planned to execute start/stop actions. If we have to designate a query command it will be a little more complex than what I propose here. In any case we don't need it for the desired use case.

@Wescoeur
Add a new service plugin
82f380c 
Signed-off-by: Ronan Abhamon <[email protected]>
@Wescoeur Wescoeur merged commit 7b661ff into master Feb 21, 2025
4 checks passed
@Wescoeur Wescoeur deleted the ran-systemctl branch February 21, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gthvn1 gthvn1 gthvn1 approved these changes

@Nambrok Nambrok Nambrok approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Wescoeur @gthvn1 @Nambrok