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

so-import-pcap: only decrease ES heap if less than 9GB RAM #1642

Closed
dougburks opened this issue Sep 15, 2019 · 5 comments
Closed

so-import-pcap: only decrease ES heap if less than 9GB RAM #1642

dougburks opened this issue Sep 15, 2019 · 5 comments
Assignees
Projects

Comments

@dougburks
Copy link
Contributor

@dougburks dougburks commented Sep 15, 2019

No description provided.

@dougburks dougburks self-assigned this Sep 15, 2019
@dougburks dougburks added this to To do in 16.04.6.3 via automation Sep 15, 2019
@dougburks

This comment has been minimized.

dougburks added a commit to Security-Onion-Solutions/securityonion-elastic that referenced this issue Oct 22, 2019
dougburks added a commit to Security-Onion-Solutions/securityonion-elastic that referenced this issue Oct 22, 2019
@dougburks dougburks moved this from To do to In progress in 16.04.6.3 Oct 22, 2019
@dougburks dougburks moved this from In progress to In Testing in 16.04.6.3 Nov 14, 2019
@weslambert

This comment has been minimized.

Copy link
Collaborator

@weslambert weslambert commented Nov 23, 2019

Looks like this is set to look for less than 9GB?

We may need to s/9000/8000 for TOTAL_MEM.

Ref: Security-Onion-Solutions/securityonion-elastic@5569465#diff-44df288646dd8caeedfa1e99760337b1R307

@dougburks dougburks changed the title so-import-pcap: only decrease ES heap if less than 8GB RAM so-import-pcap: only decrease ES heap if less than 9GB RAM Nov 23, 2019
@dougburks

This comment has been minimized.

Copy link
Contributor Author

@dougburks dougburks commented Nov 23, 2019

Good catch! I decided while working on this that we need to keep the lower heap size if RAM is 8GB, so I changed the threshold to 9GB. I've updated the title of this issue to reflect that.

@weslambert

This comment has been minimized.

Copy link
Collaborator

@weslambert weslambert commented Nov 23, 2019

In that case, it is working as intended 👍 .

@weslambert weslambert moved this from In Testing to Tested in 16.04.6.3 Nov 23, 2019
@dougburks

This comment has been minimized.

@dougburks dougburks closed this Nov 25, 2019
16.04.6.3 automation moved this from Tested to Done Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
16.04.6.3
  
Done
2 participants
You can’t perform that action at this time.