-
Notifications
You must be signed in to change notification settings - Fork 19
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
Adding ballooning support #207
Conversation
646429e
to
adf38ca
Compare
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.
can we implement any tests for this ?
done |
memoryPolicy := vm.MemoryPolicy() | ||
ballooning := memoryPolicy.Ballooning() | ||
if ballooning != testCase.expected { | ||
t.Fatalf("Incorrect ballooning 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.
Minor: If we make this Fatalf
more expressive, it'll be easier to figure out what is wrong. E.g.
t.Fatalf("Expected balooning value to be %t, but got %t", testCase.expected, balooning)
Please describe the change you are making
Adding support for memory ballooning. It also changes the return of the
MemoryPolicy()
function as I discovered that it is always present.Are you the owner of the code you are sending in, or do you have permission of the owner?
Yes
The code will be published under the BSD 3 clause license. Have you read and understood this license?
Yes