-
Notifications
You must be signed in to change notification settings - Fork 114
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
Remove bool ptr #625
Remove bool ptr #625
Conversation
We dont really need these fields to be pointers, change to concrete type to simplify code. Signed-off-by: adrianc <[email protected]>
run `make generate` to update deepcopy functions Signed-off-by: adrianc <[email protected]>
Thanks for your PR,
To skip the vendors CIs use one of:
|
Pull Request Test Coverage Report for Build 7816545048
💛 - Coveralls |
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. Thanks for making our code more simple
@SchSeba could you take a look, this is a short one. |
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.
general question this will mean that it will be false by default so if someone create the sriovOperatorConfig without this fields it will be false and in case of a pointer it will not exist no?
yes, but we have no use-case for a "missing" value. |
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.
yep you are right I have no problem moving this one out of a pointer and if in the future this is needed we can revert it. from the CRD point of view there is no change so it's transparent to the user
Change boolean attribute in SriovOperatorConfig from pointer to concrete.
Main motivation is to simplify code a bit