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
Support address and rom for hostdev #3362
Support address and rom for hostdev #3362
Conversation
Add 'rom' and 'address' for Hostdev class. Signed-off-by: Yingshun Cui <yicui@redhat.com>
|
||
|
||
class Hostdev(base.TypedDeviceBase): | ||
|
||
__slots__ = ('type', 'mode', 'managed', 'sgio', 'rawio', | ||
'source', 'boot_order', 'readonly', 'shareable', | ||
'alias', 'model', 'teaming') | ||
'alias', 'model', 'teaming', 'rom', 'address') |
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.
perhaps add simple hostdev xml under class Hostdev(base.TypedDeviceBase):
to make people understand what one typical hostdev looks like.
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.
A reference link has been added to the top of the code, I don't think we need to 'copy' them.
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.
Currently already there are some devices xml following up that practice. But it doesn't block approve
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
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
Add 'rom' and 'address' for Hostdev class.
Signed-off-by: Yingshun Cui yicui@redhat.com
Test results:
autotest/tp-libvirt#4073 (comment)