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.

Sign up for GitHub

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

Jump to bottom

dd_test:login again to avoid previous session expired #3852

Merged
merged 1 commit into from
Nov 30, 2023
Merged

dd_test:login again to avoid previous session expired #3852

merged 1 commit into from
Nov 30, 2023

Conversation

maxujun
Copy link
Contributor

@maxujun maxujun commented Oct 17, 2023

ID:1422

@maxujun
Copy link
Contributor Author

maxujun commented Oct 19, 2023

Sometimes,dd test job return late on ppc rhel7.9 guest due to performance.it will affect output for next session command.
@PaulYuuu @MiriamDeng @qingwangrh Could you help review?

@MiriamDeng
Copy link
Contributor

MiriamDeng commented Oct 19, 2023 •
edited
Loading

@maxujun is it better for you to provide a test result ? Thank you ！

@maxujun
Copy link
Contributor Author

maxujun commented Oct 19, 2023

(2/2) Host_RHEL.m8.u10.product_rhel.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.7.9.ppc64le.io-github-autotest-qemu.dd_test.multiple.disk2null: PASS (70.40 s)

hereischen
hereischen previously approved these changes Oct 26, 2023
View reviewed changes
PaulYuuu
PaulYuuu previously approved these changes Nov 1, 2023
View reviewed changes
@maxujun maxujun dismissed stale reviews from PaulYuuu and hereischen via c880783 November 20, 2023 07:28
@qingwangrh
Copy link
Contributor

python ConfigTest.py --testcase=dd_test --guestname=RHEL.9.3.0 --firmware=default_bios --clone=no

(1/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readwrite.zero2disk.q35: STARTED
(1/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readwrite.zero2disk.q35: PASS (44.49 s)
(2/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readwrite.disk2null.q35: STARTED
(2/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readwrite.disk2null.q35: PASS (35.72 s)
(3/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readonly.zero2disk.q35: STARTED
(3/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readonly.zero2disk.q35: PASS (39.94 s)
(4/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readonly.disk2null.q35: STARTED
(4/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.readonly.disk2null.q35: PASS (38.97 s)
(5/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.multiple.zero2disk.q35: STARTED
(5/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.multiple.zero2disk.q35: PASS (52.78 s)
(6/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.multiple.disk2null.q35: STARTED
(6/6) Host_RHEL.m9.u3.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.3.0.x86_64.io-github-autotest-qemu.dd_test.multiple.disk2null.q35: PASS (110.45 s)

LGTM

PaulYuuu
PaulYuuu reviewed Nov 30, 2023
View reviewed changes
Copy link
Contributor

@PaulYuuu PaulYuuu left a comment

Choose a reason for hiding this comment

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

Code is LGTM. But can you update the commit and PR title.

@maxujun maxujun changed the title dd_test:use another session to clean partition dd_test:login again to avoid previous session expired. Nov 30, 2023
@maxujun
dd_test:login again to avoid previous session expired
9839991 
Signed-off-by: Xujun Ma <xuma@redhat.com>
@maxujun
Copy link
Contributor Author

maxujun commented Nov 30, 2023

Updated

@PaulYuuu PaulYuuu changed the title dd_test:login again to avoid previous session expired. dd_test:login again to avoid previous session expired Nov 30, 2023
@PaulYuuu PaulYuuu merged commit 9c777ea into autotest:master Nov 30, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qingwangrh qingwangrh qingwangrh left review comments

@PaulYuuu PaulYuuu PaulYuuu approved these changes

@hereischen hereischen hereischen left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@maxujun @MiriamDeng @qingwangrh @hereischen @PaulYuuu