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.

Sign up for GitHub

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

Jump to bottom

improve rdma logging & clean up resources in destructor; issue-1020 #1920

Merged
merged 2 commits into from
Sep 2, 2024
Merged

improve rdma logging & clean up resources in destructor; issue-1020 #1920

merged 2 commits into from
Sep 2, 2024

Conversation

tpashkin
Copy link
Collaborator

  • add more debug points to differentiate between states
  • move non-essential messages to debug
  • use RDMA_ logging macros in server
  • resolve client's hostname
  • clean up resources in destructor if endpoint wasn't stopped

@tpashkin
improve rdma logging; issue-1020 (#1912)
1d9cd6a 
* add more debug points to differentiate between states
* move non-essential messages to debug
* use RDMA_ logging macros in server
* resolve client's hostname
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 1d9cd6a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1551 1551 0 0 0 0

@tpashkin tpashkin added the blockstore Add this label to run only cloud/blockstore build and tests on PR label Aug 30, 2024
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 1d9cd6a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1551 1551 0 0 0 0

@tpashkin tpashkin added asan Launch builds with address sanitizer along with regular build ubsan Launch builds with undefined behaviour sanitizer along with regular build msan Launch builds with memory sanitizer along with regular build labels Sep 2, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 2, 2024

Note

This is an automated comment that will be appended during run.

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit c273c34.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1551 1551 0 0 0 0

@tpashkin tpashkin merged commit 63d2ea4 into stable-23-3 Sep 2, 2024
9 of 12 checks passed
@tpashkin tpashkin deleted the users/tpashkin/1020/stable-23-3/improve-rdma-logging branch September 2, 2024 11:58
@dvrazumov dvrazumov mentioned this pull request Nov 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drbasic drbasic drbasic approved these changes

@qkrorlqr qkrorlqr qkrorlqr approved these changes

@sharpeye sharpeye Awaiting requested review from sharpeye

Assignees
No one assigned
Labels
asan Launch builds with address sanitizer along with regular build blockstore Add this label to run only cloud/blockstore build and tests on PR msan Launch builds with memory sanitizer along with regular build ubsan Launch builds with undefined behaviour sanitizer along with regular build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tpashkin @qkrorlqr @drbasic