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

Lev micro stops when server stops #114

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

levb
Copy link
Owner

@levb levb commented Oct 31, 2024

No description provided.

* provide cnats-config-version.cmake to be able to define required version in find_package

* change from requiring same major version to same minor version

---------

Co-authored-by: Rolf Lussi <[email protected]>
@levb levb force-pushed the lev-micro-stops-when-server-stops branch from 557994f to a973019 Compare October 31, 2024 01:11
Copy link

codecov bot commented Oct 31, 2024

Codecov Report

Attention: Patch coverage is 53.55450% with 98 lines in your changes missing coverage. Please review.

Project coverage is 70.48%. Comparing base (1553d4a) to head (9bc5e7f).
Report is 21 commits behind head on main.

Files with missing lines Patch % Lines
src/micro.c 54.77% 41 Missing and 49 partials ⚠️
src/micro_endpoint.c 14.28% 3 Missing and 3 partials ⚠️
src/conn.c 75.00% 0 Missing and 1 partial ⚠️
src/micro_monitoring.c 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   68.69%   70.48%   +1.78%     
==========================================
  Files          39       47       +8     
  Lines       15207    15366     +159     
  Branches     3143     3177      +34     
==========================================
+ Hits        10447    10831     +384     
+ Misses       1702     1464     -238     
- Partials     3058     3071      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- Removed the global hash of services, it was the most immediate source of the deadlock.
- Added the list of microservices to `natsConnection`. **NOTE** increased memory allocation for all connections, need a better way.
- Fixed the flow of replacing an existing endpoint in a service.
- Got rid of ep->name since it was synonimous with ep->cfg->Name
- Adjusted the tests, some minor fixes.
@levb levb force-pushed the lev-micro-stops-when-server-stops branch from be3e578 to bc40ea7 Compare November 3, 2024 19:44
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