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

Add tests for signal closing shutdown #5910

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

bogdandrutu
Copy link
Member

@bogdandrutu bogdandrutu commented Aug 15, 2022

Avoid using directly the signal channel outside of the test and struct itself.

Signed-off-by: Bogdan [email protected]

@bogdandrutu bogdandrutu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 15, 2022
@bogdandrutu bogdandrutu requested review from a team and tigrannajaryan August 15, 2022 16:56
@codecov
Copy link

codecov bot commented Aug 15, 2022

Codecov Report

Merging #5910 (8a4edfa) into main (f64389d) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5910      +/-   ##
==========================================
- Coverage   91.46%   91.44%   -0.03%     
==========================================
  Files         195      195              
  Lines       11915    11933      +18     
==========================================
+ Hits        10898    10912      +14     
- Misses        807      811       +4     
  Partials      210      210              
Impacted Files Coverage Δ
service/collector.go 76.05% <100.00%> (-0.17%) ⬇️
pdata/internal/logs.go 91.07% <0.00%> (-6.08%) ⬇️
pdata/internal/generated_plog.go 96.62% <0.00%> (-0.03%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Avoid using directly the signal channel outside of the test and struct itself.

Signed-off-by: Bogdan <[email protected]>
@bogdandrutu bogdandrutu merged commit e89786d into open-telemetry:main Aug 15, 2022
@bogdandrutu bogdandrutu deleted the signaltest branch August 15, 2022 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants