1Panel icon indicating copy to clipboard operation
1Panel copied to clipboard

WIP: refactor(service): Refactor OpenRC service manager

Open gcsong023 opened this issue 7 months ago • 3 comments
trafficstars

  • 更新 IsEnabled 和 IsActive 检查逻辑,使用更可靠的命令
  • 修复 ServiceExists 检查,直接使用文件路径判断
  • 优化 FindServices 函数,扫描 /etc/init.d 目录
  • 调整 BuildCommand 函数,支持 OpenRC 特定操作
  • 修改 ParseStatus 函数,使用更新后的正则表达式

What this PR does / why we need it?

Summary of your change

Please indicate you've done the following:

  • [x] Made sure tests are passing and test coverage is added if needed.
  • [ ] Made sure commit message follow the rule of Conventional Commits specification.
  • [ ] Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

gcsong023 avatar Apr 17 '25 14:04 gcsong023

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

f2c-ci-robot[bot] avatar Apr 17 '25 14:04 f2c-ci-robot[bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign wanghe-fit2cloud for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

f2c-ci-robot[bot] avatar Apr 17 '25 14:04 f2c-ci-robot[bot]

感谢支持 👍

wanghe-fit2cloud avatar Apr 17 '25 14:04 wanghe-fit2cloud