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.

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

CP-25794: Add new network backend type for network SR-IOV #193

Merged
merged 1 commit into from
Jan 29, 2018

Conversation

minglumlu
Copy link
Member

@minglumlu minglumlu commented Jan 19, 2018

A new network backend type for network SR-IOV VF backed vif is added.
This backend type also introduces a new filed 'vlan' in 'Vif.t'.

Signed-off-by: Ming Lu ming.lu@citrix.com

@coveralls
Copy link

coveralls commented Jan 19, 2018

Coverage Status

Coverage remained the same at 54.169% when pulling f157b15 on minglumlu:private/mingl/CP-25794 into 73025f1 on xapi-project:sr-iov.

@cheng-z cheng-z requested a review from robhoes January 22, 2018 01:09
Copy link
Collaborator

@mseri mseri left a comment

Choose a reason for hiding this comment

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

Looks good to me, but I'd like for @robhoes to have a look as well

@robhoes
Copy link
Member

robhoes commented Jan 23, 2018

Firstly, the commit message needs a signed-off-by line.

@robhoes
Copy link
Member

robhoes commented Jan 23, 2018

Secondly, the commit message talks about adding a backend type, but the commit sneaks in a VLAN field as well... ;)

A new network backend type for network SR-IOV VF backed vif is added.
This backend type also introduces a new filed 'vlan' in 'Vif.t'.

Signed-off-by: Ming Lu ming.lu@citrix.com
@minglumlu
Copy link
Member Author

@robhoes, I've refined the commit message. Thanks:)

@cheng-z
Copy link
Member

cheng-z commented Jan 29, 2018

Merge this PR as reviewed with no more comments.

@cheng-z cheng-z merged commit f563156 into xapi-project:sr-iov Jan 29, 2018
@minglumlu minglumlu deleted the private/mingl/CP-25794 branch March 10, 2018 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants