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

Memfacts #9839

Merged
merged 1 commit into from
Jan 28, 2015
Merged

Memfacts #9839

merged 1 commit into from
Jan 28, 2015

Conversation

juliedavila
Copy link
Contributor

As opened originally in #9766 this is the same thing just in it's own branch (as it should have been from the start) to avoid pr conflicts. Original text below:

This fixes problem originally discussed here: #8035

As discussed in the issue tracker the output now looks like the following:

        "ansible_memory": {
            "nocache": {
                "free": 395,
                "used": 197
            },
            "real": {
                "free": 37,
                "total": 592,
                "used": 555
            },
            "swap": {
                "cached": 0,
                "free": 2047,
                "total": 2047,
                "used": 0
            }
        }

And it includes the old top-level facts to preserve backwards compatibility.

@bcoca bcoca added P3 labels Dec 17, 2014
@mscherer
Copy link
Contributor

Could you rewrite git history to merge the fix and various clean up ?

@juliedavila
Copy link
Contributor Author

@mscherer yeah, good idea. Done.

bcoca added a commit that referenced this pull request Jan 28, 2015
@bcoca bcoca merged commit 11005e7 into ansible:devel Jan 28, 2015
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 4, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature This issue/PR relates to a feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants