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

chore: set systemd services stop timeout to 30s #46

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zsien
Copy link
Member

@zsien zsien commented May 14, 2024

设置 systemd service 退出超时为 30s

Issues: linuxdeepin/developer-center#8554

@zsien zsien requested a review from a team May 14, 2024 05:31
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zsien
Once this PR has been reviewed and has the lgtm label, please assign hudeng-go for approval. For more information see the 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

  * chore: set systemd services stop timeout to 30s (linuxdeepin/developer-cente
r#8554)
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • debian/changelog中添加了版本号和维护者信息,确认是否有遗漏必要的版本日志信息。
  • 检查是否在etc.d/systemd/system.conf.d/timeout-stop-sec.confetc.d/systemd/user.conf.d/timeout-stop-sec.conf中设置了正确的权限和用户owner。

是否建议立即修改:

  • 是,确保版本日志的完整性和文件权限的正确设置是重要的,应立即进行修改。

@zsien
Copy link
Member Author

zsien commented May 14, 2024

/integrate

Copy link

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#1426
PrNumber: 1426
PrBranch: auto-integration-9074295427

@UTsweetyfish
Copy link
Member

/-1

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Sep 2, 2024

TAG Bot

New tag: 2024.08.13
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants