Skip to content

fix: Fix front-end startup stuck#96

Merged
max-lvs merged 1 commit intolinuxdeepin:masterfrom
wangrong1069:startup-stuck
Jan 19, 2024
Merged

fix: Fix front-end startup stuck#96
max-lvs merged 1 commit intolinuxdeepin:masterfrom
wangrong1069:startup-stuck

Conversation

@wangrong1069
Copy link
Contributor

When the disk manager starts, it will send an quit message to the background service, then start a new background service, and wait for the new background service to start before displaying the main interface.

In the extreme case, when the old service has not exited and the new service has not been started, the front end detects the existence of the service, then displays the main interface and sends a request to the old service. Since the old service may have exited during the execution of the request, the front end cannot receive the updateDeviceInfo signal and remains in the initialization interface.

The solution is that the front end needs to determine the PID of the service process when waiting for the new background service to start to ensure that the new service is running.

Log: fix front-end startup stuck
Bug: https://pms.uniontech.com/bug-view-239747.html

When the disk manager starts, it will send an quit message to the
background service, then start a new background service, and wait for
the new background service to start before displaying the main
interface.

In the extreme case, when the old service has not exited and the new
service has not been started, the front end detects the existence of
the service, then displays the main interface and sends a request to
the old service. Since the old service may have exited during the
execution of the request, the front end cannot receive the
updateDeviceInfo signal and remains in the initialization interface.

The solution is that the front end needs to determine the PID of the
service process when waiting for the new background service to start
to ensure that the new service is running.

Log: fix front-end startup stuck
Bug: https://pms.uniontech.com/bug-view-239747.html
@github-actions
Copy link

TAG Bot

TAG: 1.3.32
EXISTED: no
DISTRIBUTION: stable

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: max-lvs, wangrong1069

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

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