-
Notifications
You must be signed in to change notification settings - Fork 281
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
xe pif-list
default params (#5263)
#5264
Conversation
Run |
@@ -935,7 +935,8 @@ let gen_cmds rpc session_id = | |||
; Client.PIF.( | |||
mk get_all_records_where get_by_uuid pif_record "pif" [] | |||
[ | |||
"uuid"; "device"; "VLAN"; "mac"; "network-uuid"; "currently-attached" | |||
"uuid"; "device"; "VLAN"; "MAC"; "network-uuid"; "currently-attached"; | |||
"host-id" |
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.
It's host-uuid
... and please make sure you test it ;)
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.
For some reason, xe pif-list
still does not output either MAC or host-id here, after integrating this patch into the RPM build chain to be sure not to miss anything.
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.
It works for me.
Signed-off-by: Yann Dirson <yann.dirson@vates.fr>
The wrong case causes the MAC param not to be part of the default display. Signed-off-by: Yann Dirson <yann.dirson@vates.fr>
e191254
to
4825a17
Compare
Adds
host-id
topif-list
params printed by default, and fix the "MAC" case discrepancy noticed by @robhoes