Skip to content

Conversation

@cx-rogerio-dalot
Copy link
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

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 <[email protected]>
@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