Skip to content

[TEMP PR] Investigate why Central configuration implementation #497 has tests hanging on Linux#499

Closed
SergeyKleyman wants to merge 67 commits intoelastic:masterfrom
SergeyKleyman:TEMP_Investigate_Central_config_on_Linux
Closed

[TEMP PR] Investigate why Central configuration implementation #497 has tests hanging on Linux#499
SergeyKleyman wants to merge 67 commits intoelastic:masterfrom
SergeyKleyman:TEMP_Investigate_Central_config_on_Linux

Conversation

@SergeyKleyman
Copy link
Copy Markdown
Contributor

@SergeyKleyman SergeyKleyman commented Sep 19, 2019

Investigation results are summarized at #497 (comment)

@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 19, 2019

Codecov Report

Merging #499 into master will increase coverage by 4.25%.
The diff coverage is 72.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #499      +/-   ##
==========================================
+ Coverage   73.09%   77.35%   +4.25%     
==========================================
  Files          99      101       +2     
  Lines        4115     3903     -212     
  Branches      859      702     -157     
==========================================
+ Hits         3008     3019      +11     
+ Misses        871      632     -239     
- Partials      236      252      +16
Impacted Files Coverage Δ
src/Elastic.Apm/Config/ConfigConsts.cs 100% <ø> (ø) ⬆️
src/Elastic.Apm/BackendComm/BackendCommUtils.cs 100% <100%> (+1.88%) ⬆️
src/Elastic.Apm/Helpers/ExceptionUtils.cs 94.87% <100%> (+8.82%) ⬆️
src/Elastic.Apm/Model/Transaction.cs 94.77% <100%> (+3.39%) ⬆️
src/Elastic.Apm/Helpers/IAgentTimer.cs 100% <100%> (+100%) ⬆️
src/Elastic.Apm/Agent.cs 75.86% <100%> (+3.98%) ⬆️
src/Elastic.Apm/Helpers/ToStringBuilder.cs 94.44% <100%> (+1.11%) ⬆️
src/Elastic.Apm/Helpers/ObjectExtensions.cs 100% <100%> (+20%) ⬆️
src/Elastic.Apm/Helpers/AgentSpinLock.cs 100% <100%> (+95.23%) ⬆️
src/Elastic.Apm.AspNetCore/AspNetCoreLogger.cs 29.41% <100%> (+4.41%) ⬆️
... and 52 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4dd1751...08ef2fd. Read the comment docs.

@SergeyKleyman SergeyKleyman changed the title TEMP PR: Investigate why Central configuration implementation #497 has tests hanging on Linux [TEMP PR] Investigate why Central configuration implementation #497 has tests hanging on Linux Sep 19, 2019
@SergeyKleyman
Copy link
Copy Markdown
Contributor Author

Investigation results are summarized at #497 (comment)

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.

2 participants