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

Feature: automatically set up GOMEMLIMIT #691

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Feature: automatically set up GOMEMLIMIT #691

merged 1 commit into from
Apr 19, 2024

Conversation

mem
Copy link
Contributor

@mem mem commented Apr 19, 2024

Automatically set up GOMEMLIMIT to 90% of the cgroup allowed memory. If
running in an environment where there's no cgroup limit set up, fallback
to 90% of the system's memory. Note that this doesn't mean that the
program will use 90% of the system's memory, but that if it does get to
that point, the garbaga collector will try to be more aggresive.

It is strongly recommended to set a limit via cgroups.

Signed-off-by: Marcelo E. Magallon [email protected]

@mem mem requested a review from a team as a code owner April 19, 2024 01:16
@mem
Copy link
Contributor Author

mem commented Apr 19, 2024

This needs #689

@mem mem force-pushed the setup-auto-memlimit branch from aa7af33 to 06c8d48 Compare April 19, 2024 16:52
@mem mem changed the title Log configuration at start up Feature: automatically set up GOMEMLIMIT Apr 19, 2024
@mem mem force-pushed the setup-auto-memlimit branch from 06c8d48 to a9a038b Compare April 19, 2024 17:10
Automatically set up GOMEMLIMIT to 90% of the cgroup allowed memory. If
running in an environment where there's no cgroup limit set up, fallback
to 90% of the system's memory. Note that this doesn't mean that the
program will use 90% of the system's memory, but that if it does get to
that point, the garbaga collector will try to be more aggresive.

It is strongly recommended to set a limit via cgroups.

Signed-off-by: Marcelo E. Magallon <[email protected]>
@mem mem force-pushed the setup-auto-memlimit branch from a9a038b to 00ecc8a Compare April 19, 2024 17:18
@mem mem merged commit 79b1927 into main Apr 19, 2024
4 checks passed
@mem mem deleted the setup-auto-memlimit branch April 19, 2024 19:55
The-9880 added a commit that referenced this pull request Apr 30, 2024
* Feature: upgrade k6 to v0.50.0 (#681)
* Chore(deps): Bump github.com/prometheus/blackbox_exporter
* skip interpolating multihttp variables into the metric names (#683)
* proto fields for scripted check enablement (#674)
* Switch to using buf to manage protobuf code (#641)
* Release v0.23.4 (#685)
* Fix: TestTickWithOffset sometimes if offset is 0 (#686)
* Chore(deps): Bump the prometheus-go group with 2 updates
* Chore(deps): Bump github.com/miekg/dns from 1.1.58 to 1.1.59
* Log configuration at start up (#689)
* Feature: automatically set up GOMEMLIMIT (#691)
* Terminate agent if capabilities not supported (#684)
* Make k6 capability validation nil-safe (#692)
* Fix: use uniform timeout validation logic (#693)

Signed-off-by: Anant Sharma <[email protected]>
@The-9880 The-9880 mentioned this pull request Apr 30, 2024
The-9880 added a commit that referenced this pull request Apr 30, 2024
* Feature: upgrade k6 to v0.50.0 (#681)
* Chore(deps): Bump github.com/prometheus/blackbox_exporter
* skip interpolating multihttp variables into the metric names (#683)
* proto fields for scripted check enablement (#674)
* Switch to using buf to manage protobuf code (#641)
* Release v0.23.4 (#685)
* Fix: TestTickWithOffset sometimes if offset is 0 (#686)
* Chore(deps): Bump the prometheus-go group with 2 updates
* Chore(deps): Bump github.com/miekg/dns from 1.1.58 to 1.1.59
* Log configuration at start up (#689)
* Feature: automatically set up GOMEMLIMIT (#691)
* Terminate agent if capabilities not supported (#684)
* Make k6 capability validation nil-safe (#692)
* Fix: use uniform timeout validation logic (#693)

Signed-off-by: Anant Sharma <[email protected]>
@The-9880 The-9880 mentioned this pull request Apr 30, 2024
The-9880 added a commit that referenced this pull request Apr 30, 2024
* Feature: upgrade k6 to v0.50.0 (#681)
* Chore(deps): Bump github.com/prometheus/blackbox_exporter
* skip interpolating multihttp variables into the metric names (#683)
* proto fields for scripted check enablement (#674)
* Switch to using buf to manage protobuf code (#641)
* Release v0.23.4 (#685)
* Fix: TestTickWithOffset sometimes if offset is 0 (#686)
* Chore(deps): Bump the prometheus-go group with 2 updates
* Chore(deps): Bump github.com/miekg/dns from 1.1.58 to 1.1.59
* Log configuration at start up (#689)
* Feature: automatically set up GOMEMLIMIT (#691)
* Terminate agent if capabilities not supported (#684)
* Make k6 capability validation nil-safe (#692)
* Fix: use uniform timeout validation logic (#693)
* k6runner: handle errors reported by http runners
* k6runner/test: add test for RunResponse error handling
* k6runner: inspect errors and propagate unexpected ones to the probe

Signed-off-by: Anant Sharma <[email protected]>
@The-9880 The-9880 mentioned this pull request Apr 30, 2024
mem pushed a commit that referenced this pull request Apr 30, 2024
* Feature: upgrade k6 to v0.50.0 (#681)
* Chore(deps): Bump github.com/prometheus/blackbox_exporter
* skip interpolating multihttp variables into the metric names (#683)
* proto fields for scripted check enablement (#674)
* Switch to using buf to manage protobuf code (#641)
* Release v0.23.4 (#685)
* Fix: TestTickWithOffset sometimes if offset is 0 (#686)
* Chore(deps): Bump the prometheus-go group with 2 updates
* Chore(deps): Bump github.com/miekg/dns from 1.1.58 to 1.1.59
* Log configuration at start up (#689)
* Feature: automatically set up GOMEMLIMIT (#691)
* Terminate agent if capabilities not supported (#684)
* Make k6 capability validation nil-safe (#692)
* Fix: use uniform timeout validation logic (#693)
* k6runner: handle errors reported by http runners
* k6runner/test: add test for RunResponse error handling
* k6runner: inspect errors and propagate unexpected ones to the probe

Signed-off-by: Anant Sharma <[email protected]>
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.

2 participants