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
sriov: Add 3 cases #4073
sriov: Add 3 cases #4073
Conversation
f132621
to
cb7eadb
Compare
cb7eadb
to
a31bba7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Others LGTM
:raise: TestFail if comparison fails | ||
""" | ||
vm_hostdev = vm_xml.VMXML.new_from_dumpxml(vm.name)\ | ||
.devices.by_device_tag(device_type)[0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am just wondering if [0] must be what you want? Shall we also consider multiple devices existing? If not, could we add prerequisite in docstring for special usage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the docstring, seems we don't need multiple hostdev devices for now.
hostdev_dict = {'type': 'pci', 'address': {'attrs':{'type': 'unassigned'}}, 'mode': 'subsystem', 'managed': 'yes', 'source': {'untyped_address': %s}} | ||
- hostdev_interface: | ||
hostdev_iface_dict = {'managed': 'yes', 'hostdev_address': {'attrs': %s}, 'address': {'attrs':{'type': 'unassigned'}}} | ||
- deplicate_cust_alias: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicate_cust_alias?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops! Thank you for pointing it out. Updated.
if p_iface: | ||
test.fail("There should be no interface, but got %s." % p_iface) | ||
|
||
def test_deplicate_cust_alias(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo. test_duplicate_cust_alias
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated.
This PR adds below cases: 1. RHEL-288445: Hotplug hostdev interface with 'unassigned' address type 2. RHEL-288444: Start vm with hostdev interface with 'unassigned' address type 3. RHEL-134143: Hotplug hostdev interface with duplicate custom alias Signed-off-by: Yingshun Cui <yicui@redhat.com>
a31bba7
to
de3b7f9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds below cases:
RHEL-288445: Hotplug hostdev interface with 'unassigned'
address type
RHEL-288444: Start vm with hostdev interface with 'unassigned'
address type
RHEL-134143: Hotplug hostdev interface with duplicate custom alias
Signed-off-by: Yingshun Cui yicui@redhat.com
depends on avocado-framework/avocado-vt#3362
Test results: 2 cases are failed due to a known BZ.