Skip to content

fix(libvirt-guests): redirect service output to journal to suppress s…#28

Merged
Zeno-sole merged 1 commit into
deepin-community:masterfrom
wxphaha:fix-log
Jun 22, 2026
Merged

fix(libvirt-guests): redirect service output to journal to suppress s…#28
Zeno-sole merged 1 commit into
deepin-community:masterfrom
wxphaha:fix-log

Conversation

@wxphaha

@wxphaha wxphaha commented Jun 22, 2026

Copy link
Copy Markdown

…hutdown

error messages

The libvirt-guests service prints "Can't connect to default. Skipping." to the console when shutting down if libvirtd is already stopped. This error message appears on the screen during system shutdown, disturbing the user experience.

Change StandardOutput and StandardError from "journal+console" to "journal" to suppress console output while still keeping logs available in the system journal for debugging purposes.

This ensures a clean shutdown experience without unnecessary error messages appearing on the console.

修复(libvirt-guests):将服务输出重定向到 journal,抑制关机时的错误打印

当 libvirtd 在关机时已提前停止,libvirt-guests 服务会在控制台打印
"Can't connect to default. Skipping." 错误信息,影响关机体验。

将 StandardOutput 和 StandardError 从 "journal+console" 改为 "journal", 在保留日志供调试的同时,屏蔽控制台输出,确保关机过程干净无干扰。
Bug: 362087

…hutdown

error messages

The libvirt-guests service prints "Can't connect to default. Skipping."
to the console when shutting down if libvirtd is already stopped. This
error message appears on the screen during system shutdown, disturbing
the user experience.

Change StandardOutput and StandardError from "journal+console" to
"journal" to suppress console output while still keeping logs available
in the system journal for debugging purposes.

This ensures a clean shutdown experience without unnecessary error
messages appearing on the console.

修复(libvirt-guests):将服务输出重定向到 journal,抑制关机时的错误打印

当 libvirtd 在关机时已提前停止,libvirt-guests 服务会在控制台打印
"Can't connect to default. Skipping." 错误信息,影响关机体验。

将 StandardOutput 和 StandardError 从 "journal+console" 改为 "journal",
在保留日志供调试的同时,屏蔽控制台输出,确保关机过程干净无干扰。
Bug: 362087
@deepin-ci-robot

Copy link
Copy Markdown
Contributor

Hi @wxphaha. Thanks for your PR. 😃

@deepin-ci-robot deepin-ci-robot requested review from BLumia and myml June 22, 2026 07:24
@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 10.7.0+really9.10.0-1deepin21
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[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 tsic404 for approval. For more information see the Code Review Process.

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

Details 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

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

Hi @wxphaha. Thanks for your PR.

I'm waiting for a deepin-community member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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.

@Zeno-sole Zeno-sole merged commit dfb948c into deepin-community:master Jun 22, 2026
2 of 3 checks passed
@Zeno-sole

Copy link
Copy Markdown
Contributor

/integrate

@github-actions

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants