Skip to content

Commit

Permalink
#2183: Use vtWarn instead of vtAssert for LDMS related functions
Browse files Browse the repository at this point in the history
  • Loading branch information
JacobDomagala committed Oct 6, 2023
1 parent df42c33 commit 628bd73
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/vt/vrt/collection/balance/node_lb_data.cc
Original file line number Diff line number Diff line change
Expand Up @@ -157,10 +157,10 @@ void NodeLBData::initialize() {

#if vt_check_enabled(ldms)
ldms_ = ldms_xprt_new_with_auth("sock", "none", NULL);
vtAssert(ldms_, "ldms_xprt_new_with_auth failed!");
vtWarnIf(ldms_, "ldms_xprt_new_with_auth failed!");

const auto returnCode = ldms_xprt_connect_by_name(ldms_, "localhost", "10444", NULL, NULL);
vtAssert(returnCode == 0, fmt::format("ldms_xprt_connect_by_name failed with code {} \n", returnCode));
vtWarnIf(returnCode == 0, fmt::format("ldms_xprt_connect_by_name failed with code {} \n", returnCode));
#endif
}

Expand Down Expand Up @@ -307,7 +307,7 @@ void NodeLBData::outputLBDataForPhase(PhaseType phase) {
const auto returnVal = ldmsd_stream_publish(
ldms_, "LB_data", LDMSD_STREAM_JSON, jsonStr.c_str(), jsonStr.length() + 1
);
vtAssert(returnVal == 0, fmt::format("ldmsd_stream_publish returned {}!\n", returnVal));
vtWarnIf(returnVal == 0, fmt::format("ldmsd_stream_publish returned {}!\n", returnVal));
#endif
}

Expand Down

0 comments on commit 628bd73

Please sign in to comment.