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

[docs] Update default memory limit #104535

Merged
merged 2 commits into from
Jul 7, 2021
Merged

[docs] Update default memory limit #104535

merged 2 commits into from
Jul 7, 2021

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Jul 6, 2021

Upgrading to Node 12+ removed the 1.4gb heap allocation limit,
and set it to scale with total available memory.

[1] Properly configure default heap limits
This update will configure the JavaScript heap size based on available
memory instead of using defaults that were set by V8 for use with
browsers. In previous releases, unless configured, V8 defaulted to
limiting the max heap size to 700 MB or 1400MB on 32 and 64-bit
platforms respectively. Configuring the heap size based on available
memory ensures that Node.js does not try to use more memory than is
available and terminating when its memory is exhausted.

[1] nodejs.medium.com/introducing-node-js-12-76c41a1b3f3f

This updates our documentation to reflect this change.

Closes #104451

Upgrading to Node 12+ removed the 1.4gb heap allocation limit,
and set it to scale with total available memory.

> [1] Properly configure default heap limits
This update will configure the JavaScript heap size based on available
memory instead of using defaults that were set by V8 for use with
browsers. In previous releases, unless configured, V8 defaulted to
limiting the max heap size to 700 MB or 1400MB on 32 and 64-bit
platforms respectively. Configuring the heap size based on available
memory ensures that Node.js does not try to use more memory than is
available and terminating when its memory is exhausted.

[1] nodejs.medium.com/introducing-node-js-12-76c41a1b3f3f

This updates our documentation to reflect this change.

Closes elastic#104451
@jbudz jbudz added Team:Docs Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.14.0 auto-backport Deprecated - use backport:version if exact versions are needed v7.11.3 v7.12.2 v7.15.0 v7.13.3 labels Jul 6, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
Copy link
Contributor

@gchaps gchaps left a comment

Choose a reason for hiding this comment

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

LGTM

@joshdover joshdover merged commit 02b05c9 into elastic:master Jul 7, 2021
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
7.11 Commit could not be cherrypicked due to conflicts
7.12
7.13
7.14
7.x

Successful backport PRs will be merged automatically after passing CI.

To backport manually run:
node scripts/backport --pr 104535

kibanamachine added a commit that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

Co-authored-by: Jonathan Budzenski <jon@budzenski.me>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

Co-authored-by: Jonathan Budzenski <jon@budzenski.me>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

Co-authored-by: Jonathan Budzenski <jon@budzenski.me>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

Co-authored-by: Jonathan Budzenski <jon@budzenski.me>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
@joshdover
Copy link
Contributor

Working on the 7.11 backport now...

joshdover pushed a commit to joshdover/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
# Conflicts:
#	docs/user/production-considerations/production.asciidoc
joshdover added a commit that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
# Conflicts:
#	docs/user/production-considerations/production.asciidoc

Co-authored-by: Jonathan Budzenski <jon@budzenski.me>
darnautov pushed a commit to darnautov/kibana that referenced this pull request Jul 7, 2021
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Docs Team:Operations Team label for Operations Team v7.11.3 v7.12.2 v7.13.3 v7.14.0 v7.15.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] Update --max-old-space-size documentation to reflect Node.js 12+
5 participants