Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

channelz: fix race accessing channelMap without lock #7079

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

dfawley
Copy link
Member

@dfawley dfawley commented Apr 3, 2024

This will need a backport.

RELEASE NOTES: none

@dfawley dfawley added this to the 1.63 Release milestone Apr 3, 2024
@dfawley dfawley requested a review from arvindbr8 April 3, 2024 16:21
Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Merging #7079 (5f93246) into master (4ec8307) will decrease coverage by 1.45%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7079      +/-   ##
==========================================
- Coverage   82.54%   81.10%   -1.45%     
==========================================
  Files         305      345      +40     
  Lines       31393    33925    +2532     
==========================================
+ Hits        25913    27514    +1601     
- Misses       4425     5228     +803     
- Partials     1055     1183     +128     
Files Coverage Δ
internal/channelz/funcs.go 100.00% <100.00%> (ø)

... and 58 files with indirect coverage changes

@dfawley dfawley merged commit b7346ae into grpc:master Apr 3, 2024
14 checks passed
@dfawley dfawley deleted the czrace branch April 3, 2024 16:27
dfawley added a commit to dfawley/grpc-go that referenced this pull request Apr 3, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 1, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants