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

update_iface_with_unchangable: no rom on aarch64 #6111

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

update_iface_with_unchangable: no rom on aarch64 #6111

wants to merge 1 commit into from

Conversation

dzhengfy
Copy link
Contributor

@dzhengfy dzhengfy commented Jan 6, 2025 •
edited
Loading

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

@dzhengfy
update_iface_with_unchangable: no rom on aarch64
d764e31 
Signed-off-by: Dan Zheng <dzheng@redhat.com>
@dzhengfy
Copy link
Contributor Author

dzhengfy commented Jan 6, 2025

Before the fix:
[rhel.virtual_network.update_device.with_unchangable]
VM 'avocado-vt-vm1' failed to start: error: Failed to start domain 'avocado-vt-vm1' error: internal error: QEMU ...}: failed to find romfile "/usr/share/ipxe/1af41000.rom"(exit status: 1)

After the fix:
(1/1) type_specific.io-github-autotest-libvirt.virtual_network.update_device.with_unchangable: PASS (38.91 s)

@dzhengfy
Copy link
Contributor Author

dzhengfy commented Jan 6, 2025

@yafu-1 please help review

Yingshun
Yingshun reviewed Jan 6, 2025
View reviewed changes
Copy link
Contributor

@Yingshun Yingshun left a comment

Choose a reason for hiding this comment

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

1af41000.rom is installed from ipxe-roms and ipxe-roms-qemu, do you need to install these two packages if not?

@yafu-1
Copy link
Contributor

yafu-1 commented Jan 6, 2025

1af41000.rom is installed from ipxe-roms and ipxe-roms-qemu, do you need to install these two packages if not?
The case can pass after installing ipxe-roms-qemu pkg.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yingshun Yingshun Yingshun left review comments

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

Successfully merging this pull request may close these issues.
3 participants
@dzhengfy @yafu-1 @Yingshun