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

Add softfailure for bug: 1221812 #19707

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

RoyCai7
Copy link
Contributor

@RoyCai7 RoyCai7 commented Jul 10, 2024

Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@RoyCai7 RoyCai7 force-pushed the migration branch 10 times, most recently from f33762e to 87eac11 Compare July 11, 2024 07:17
@RoyCai7
Copy link
Contributor Author

RoyCai7 commented Jul 11, 2024

Complete development, and pending final VR test.

@guoxuguang
Copy link
Contributor

LGTM

@alice-suse
Copy link
Contributor

I guess it is ready for merge? @RoyCai7 please reply if there is other ongoing code rework or verification.

@RoyCai7
Copy link
Contributor Author

RoyCai7 commented Jul 16, 2024

I guess it is ready for merge? @RoyCai7 please reply if there is other ongoing code rework or verification.

Completed development, and completed VR tests.
Verification run:
sles15sp6
http://openqa.qam.suse.cz/tests/77864#step/parallel_guest_migration_source/238
other version:
http://openqa.qam.suse.cz/tests/77870

@alice-suse
Copy link
Contributor

Thank you, then let's merge it.

@alice-suse alice-suse merged commit 47f707f into os-autoinst:master Jul 16, 2024
10 checks passed
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.

4 participants