Skip to content

fix: hotfix for v3.17.0 set maximum of goroutines#296

Closed
cx-rogerio-dalot wants to merge 5 commits intorelease/v3.17.0from
AST-99769-thread-exhaustion-error-in-2-ms
Closed

fix: hotfix for v3.17.0 set maximum of goroutines#296
cx-rogerio-dalot wants to merge 5 commits intorelease/v3.17.0from
AST-99769-thread-exhaustion-error-in-2-ms

Conversation

@cx-rogerio-dalot
Copy link
Copy Markdown
Contributor

Proposed Changes

The binary as is can take the process to resource exhaustion due to going over the limit of 10k goroutines.

This is a quickfix/hotfix and in no case sustainable for the future, we need indeed to refactor our current implementation but it should solve the issue for now.

Checklist

  • I covered my changes with tests.
  • I Updated the documentation that is affected by my changes:
    • Change in the CLI arguments
    • Change in the configuration file

I submit this contribution under the Apache-2.0 license.

@cx-rogerio-dalot cx-rogerio-dalot requested a review from a team as a code owner June 4, 2025 12:14
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2025

Logo
Checkmarx One – Scan Summary & Details21fedb2f-ad0f-4822-a301-8a0f8112a258

Policy Management Violations (1)
Policy Name Rule(s) Break Build
FluentAssertions v8 true

@cx-rogerio-dalot cx-rogerio-dalot force-pushed the AST-99769-thread-exhaustion-error-in-2-ms branch from a71d70c to a95cd85 Compare June 4, 2025 12:20
@cx-rogerio-dalot cx-rogerio-dalot changed the title fix: hotfix for v3.17.0(#274) - set maximum of goroutines fix: hotfix for v3.17.0 set maximum of goroutines Jun 4, 2025
<!--
Thanks for contributing to 2ms by offering a pull request.
-->

Closes #

**Proposed Changes**

<!--
Please describe the big picture of your changes here. If it fixes a bug
or resolves a feature request, be sure to link to that issue.
-->

**Checklist**

- [ ] I covered my changes with tests.
- [ ] I Updated the documentation that is affected by my changes:
  - [ ] Change in the CLI arguments
  - [ ] Change in the configuration file

I submit this contribution under the Apache-2.0 license.

---------

Co-authored-by: cx-leonardo-fontes <204389152+cx-leonardo-fontes@users.noreply.github.com>
@cx-rogerio-dalot cx-rogerio-dalot force-pushed the AST-99769-thread-exhaustion-error-in-2-ms branch 4 times, most recently from 14a1c4f to b1a77a1 Compare June 4, 2025 14:11
@cx-rogerio-dalot cx-rogerio-dalot force-pushed the AST-99769-thread-exhaustion-error-in-2-ms branch from b1a77a1 to da1764d Compare June 4, 2025 14:25
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