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

Handle a max memory value of 'max' #77289

Merged

Conversation

pugnascotia
Copy link
Contributor

Closes #77280.

@pugnascotia pugnascotia added >bug :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts v8.0.0 labels Sep 5, 2021
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Sep 5, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@elasticsearchmachine
Copy link
Collaborator

Hi @pugnascotia, I've created a changelog YAML for you.

@pugnascotia pugnascotia added >test Issues or PRs that are addressing/adding tests and removed >bug labels Sep 6, 2021
@pugnascotia pugnascotia added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 6, 2021
@elasticsearchmachine elasticsearchmachine merged commit ebf5a92 into elastic:master Sep 6, 2021
pugnascotia added a commit to pugnascotia/elasticsearch that referenced this pull request Sep 6, 2021
* Handle a max memory value of 'max'

* Update docs/changelog/77289.yaml

* Delete 77289.yaml
elasticsearchmachine pushed a commit that referenced this pull request Sep 6, 2021
* Handle cgroups v2 in `OsProbe` (#77128)

Closes #76812. Closes #77126.

OsProbe was only capable of handle cgroup data in the v1 format.
However, Debian 11 uses cgroups v2 by default, and Elasticsearch isn't
capable of reporting any cgroup information. Therefore, add support for
the v2 layout.

Note that we have to open access to all of /sys/fs/cgroup because with
cgroups v2, the files we need are in an unpredictably location.

* Handle a max memory value of 'max' (#77289)

* Handle a max memory value of 'max'

* Update docs/changelog/77289.yaml

* Delete 77289.yaml

* Fixes to backport

* Fix
@pugnascotia pugnascotia deleted the 77280-cgroups-v2-more-fix branch September 14, 2021 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OsProbeTests.testOsStats failures on Debian 11
4 participants