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

WIP: add mochi-plumber support for automatic network card selection #298

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

carns
Copy link
Member

@carns carns commented Dec 17, 2024

Adds optional support for the mochi-plumber library and json parameters to configure it.

Copy link

codecov bot commented Dec 17, 2024

Codecov Report

Attention: Patch coverage is 59.09091% with 9 lines in your changes missing coverage. Please review.

Project coverage is 57.88%. Comparing base (c61e8e2) to head (f34e571).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
src/margo-init.c 53.84% 4 Missing and 2 partials ⚠️
src/margo-core.c 0.00% 2 Missing ⚠️
src/margo-hg-config.c 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #298      +/-   ##
==========================================
- Coverage   57.89%   57.88%   -0.01%     
==========================================
  Files          70       70              
  Lines       10198    10216      +18     
  Branches     1341     1347       +6     
==========================================
+ Hits         5904     5914      +10     
- Misses       3458     3465       +7     
- Partials      836      837       +1     

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

@@ -22,7 +22,9 @@
if (options & MARGO_CONFIG_PRETTY_JSON) {
json_to_string_flags |= JSON_C_TO_STRING_PRETTY;
}
struct json_object* root = json_object_new_object();
struct json_object* root = json_object_new_object();
struct json_object* _plumber = NULL;

Check notice

Code scanning / CodeQL

Unused local variable Note

Variable _plumber is not used.
@@ -180,6 +180,9 @@
mid->monitor->finalize(mid->monitor->uargs);
free(mid->monitor);

if (mid->plumber_bucket_policy) free(mid->plumber_bucket_policy);

Check notice

Code scanning / CodeQL

Guarded Free Note

unnecessary NULL check before call to
free
@@ -180,6 +180,9 @@
mid->monitor->finalize(mid->monitor->uargs);
free(mid->monitor);

if (mid->plumber_bucket_policy) free(mid->plumber_bucket_policy);
if (mid->plumber_nic_policy) free(mid->plumber_nic_policy);

Check notice

Code scanning / CodeQL

Guarded Free Note

unnecessary NULL check before call to
free
@@ -387,6 +410,8 @@
ABT_cond_free(&mid->finalize_cond);
ABT_mutex_free(&mid->pending_operations_mtx);
if (mid->current_rpc_id_key) ABT_key_free(&(mid->current_rpc_id_key));
if (mid->plumber_bucket_policy) free(mid->plumber_bucket_policy);

Check notice

Code scanning / CodeQL

Guarded Free Note

unnecessary NULL check before call to
free
@@ -387,6 +410,8 @@
ABT_cond_free(&mid->finalize_cond);
ABT_mutex_free(&mid->pending_operations_mtx);
if (mid->current_rpc_id_key) ABT_key_free(&(mid->current_rpc_id_key));
if (mid->plumber_bucket_policy) free(mid->plumber_bucket_policy);
if (mid->plumber_nic_policy) free(mid->plumber_nic_policy);

Check notice

Code scanning / CodeQL

Guarded Free Note

unnecessary NULL check before call to
free
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.

1 participant