-
Notifications
You must be signed in to change notification settings - Fork 111
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
Fix root device overlapping #671
Fix root device overlapping #671
Conversation
Thanks for your PR,
To skip the vendors CIs use one of:
|
Pull Request Test Coverage Report for Build 8646930999Details
💛 - Coveralls |
api/v1/helper.go
Outdated
@@ -547,6 +547,19 @@ func ParsePFName(name string) (ifName string, rngSt, rngEnd int, err error) { | |||
return | |||
} | |||
|
|||
// ParseRootDevice: partse the rootDevice with VF range |
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.
nit: partse => parse
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.
done thanks!
8572034
to
2b8c7c9
Compare
Thanks for your PR,
To skip the vendors CIs use one of:
|
@@ -481,7 +481,7 @@ func (p *SriovNetworkNodePolicy) generateVfGroup(iface *InterfaceExt) (*VfGroup, | |||
var rngStart, rngEnd int | |||
found := false | |||
for _, selector := range p.Spec.NicSelector.PfNames { |
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.
Ranges are still defined on PfNames
field only.
You can verify it by adding unit tests like a92b764 .
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.
you are right!
I think supporting rage in the root device will be much harder than I was expecting so I will first fix the webhook issue and than we can take to the community to talk about the range support for rootDevices.
let me update the PR
2b8c7c9
to
9e5c802
Compare
Thanks for your PR,
To skip the vendors CIs use one of:
|
9e5c802
to
04dbdf5
Compare
Thanks for your PR,
To skip the vendors CIs use one of:
|
Signed-off-by: Sebastian Sch <sebassch@gmail.com>
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
Hi @e0ne @Eoghan1232 when you have time please take a look on this small PR |
Thanks for the review folks! merging this one |
No description provided.