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

Show molecule ram usage #401

Merged
merged 1 commit into from Aug 21, 2020
Merged

Conversation

mikedep333
Copy link
Member

So we know if the CI env running 4 containers (w / Pulp running) +
4 ansible processes is low on RAM.

PR is based on #395

[noissue]

@pulpbot
Copy link
Member

pulpbot commented Aug 20, 2020

WARNING!!! This PR is not attached to an issue. In most cases this is not advisable. Please see our PR docs for more information about how to attach this PR to an issue.

Comment on lines +86 to +105
- name: Check RAM usage
command: free -m
changed_when: false
register: result

- name: Show RAM usage, so we know if our CI env is low on RAM
debug: var=result.stdout_lines
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! But I wonder if molecule has something for it

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the amount of RAM, yes. For currently free, it doesn't seem like it. I researched it for a bit.

@mikedep333
Copy link
Member Author

mikedep333 commented Aug 20, 2020

Per our convo, the verification tests are not running at all on CI.

I'm looking into this.

So we know if the CI env running 4 containers (w / Pulp running) +
4 ansible processes is low on RAM.

[noissue]
@mikedep333
Copy link
Member Author

mikedep333 commented Aug 21, 2020

On source-dynamic (4 distros/containers running), we seem to have plenty of RAM free. About 3.5 GB out of 7GB.

Any swap usage (387 MB) is concerning for performance, but that probably reflects 1 or 2 tasks that needed swap temporarily.

ok: [centos-8] => {
    "result.stdout_lines": [
        "              total        used        free      shared  buff/cache   available",
        "Mem:           6927        2812         447         125        3667        3711",
        "Swap:          4095         387        3708"
    ]
}

@mikedep333 mikedep333 merged commit addc342 into pulp:master Aug 21, 2020
@mikedep333 mikedep333 deleted the show_molecule_ram_usage branch August 21, 2020 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants