Skip to content

transfer_file_over_ipv6: fix color char#4293

Closed
dzhengfy wants to merge 1 commit intoautotest:masterfrom
dzhengfy:fix_color_bigfile_transfer_ipv6
Closed

transfer_file_over_ipv6: fix color char#4293
dzhengfy wants to merge 1 commit intoautotest:masterfrom
dzhengfy:fix_color_bigfile_transfer_ipv6

Conversation

@dzhengfy
Copy link
Copy Markdown

@dzhengfy dzhengfy commented Mar 25, 2025

The command might output ip address with color characters which will break the script, like below.
Get ipv6 address of vm1: �[34m2620:52:11:1601:982f:7bff:fe94:1415�[0m

This is to disable the color characters in the outputs.

Signed-off-by: Dan Zheng dzheng@redhat.com

Signed-off-by: Dan Zheng <dzheng@redhat.com>
@dzhengfy
Copy link
Copy Markdown
Author

Before the fix:
(1/1) Host_RHEL.m9.u6.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.aarch64.page_64k.io-github-autotest-qemu.bigfile_transfer_over_ipv6.remote_addr.arm64-pci: ERROR: SCP transfer failed (status: 255, output: 'Executing: program /usr/bin/ssh host \033[34m2620:52:11:1601:9824:20ff:fe54:9207\033[0m, user root, command sftp\nhostname contains invalid characters\nConnection closed\n') (112.42 s)

After the fix:
(1/1) Host_RHEL.m9.u6.qcow2.virtio_scsi.up.virtio_net.Guest.RHEL.9.6.0.aarch64.page_64k.io-github-autotest-qemu.bigfile_transfer_over_ipv6.remote_addr.arm64-pci: PASS (132.35 s)

@dzhengfy dzhengfy added the dependencies Pull requests that update a dependency file label Mar 25, 2025
@dzhengfy
Copy link
Copy Markdown
Author

@dzhengfy dzhengfy requested a review from YvanY0 March 25, 2025 04:48
Copy link
Copy Markdown
Contributor

@YvanY0 YvanY0 left a comment

Choose a reason for hiding this comment

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

This issue should be fixed for the entire guest's lifecycle, not at the test case level. We have a fixed for el10, can also apply to el9.

@dzhengfy
Copy link
Copy Markdown
Author

based on above comment, I close this one.

@dzhengfy dzhengfy closed this Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants