-
Notifications
You must be signed in to change notification settings - Fork 495
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
Add FreeBSD memory facts #2020
Add FreeBSD memory facts #2020
Conversation
Can one of the admins verify this patch? |
dded107
to
23b2c8e
Compare
23b2c8e
to
733e5ca
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.
🙋🏻♀️
CLA signed by all contributors. |
@IrimieBogdan I added a commit that fix the issues you listed, thank you for catching them, I mostly copied the Solaris facts which mix old and new style specs. Should these be updated too (I mean in this PR, or do we keep the focus on FreeBSD memory facts here)? |
let's clean up solaris tests in a next round |
eef6301
to
40e14ab
Compare
Collect system and swap memory information on FreeBSD
🎁 @igalic
The memory facts where not provided by Facter 4. With this PR, Facter returns the same facts as Facter 3 does.