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

Wait more time for zypper process in background to finish #13526

Conversation

lemon-suse
Copy link
Contributor

@lemon-suse lemon-suse commented Oct 25, 2021

sometimes the zypper processes already in quit process but can't pgrep the process, we need wait several seconds for them to finish deactivate.

@lemon-suse lemon-suse requested a review from juadk October 25, 2021 05:25
lib/zypper.pm Outdated Show resolved Hide resolved
lib/zypper.pm Outdated Show resolved Hide resolved
@lemon-suse lemon-suse force-pushed the wait-more-time-for-backend-zypper-process-finish branch from e83e86f to 953908b Compare October 25, 2021 07:03
lib/zypper.pm Outdated Show resolved Hide resolved
@lemon-suse lemon-suse force-pushed the wait-more-time-for-backend-zypper-process-finish branch 3 times, most recently from 1186b4d to 4afa5b0 Compare October 25, 2021 09:17
@lemon-suse lemon-suse force-pushed the wait-more-time-for-backend-zypper-process-finish branch from 4afa5b0 to 0230b47 Compare October 26, 2021 01:15
Copy link
Contributor

@rfan1 rfan1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@tinawang123 tinawang123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lemon-suse lemon-suse merged commit fe2e530 into os-autoinst:master Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants