Skip to content

Fix process resource detector triggers warning if running with unknown uid#840

Merged
brettmc merged 1 commit into
open-telemetry:mainfrom
Nevay:fix/process-owner-triggers-warning-if-no-user
Oct 9, 2022
Merged

Fix process resource detector triggers warning if running with unknown uid#840
brettmc merged 1 commit into
open-telemetry:mainfrom
Nevay:fix/process-owner-triggers-warning-if-no-user

Conversation

@Nevay

@Nevay Nevay commented Oct 9, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@Nevay
Nevay requested a review from a team October 9, 2022 20:23
@codecov

codecov Bot commented Oct 9, 2022

Copy link
Copy Markdown

Codecov Report

Merging #840 (023e1bb) into main (a3a47a6) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #840   +/-   ##
=========================================
  Coverage     82.49%   82.49%           
- Complexity     1883     1884    +1     
=========================================
  Files           234      234           
  Lines          4901     4901           
=========================================
  Hits           4043     4043           
  Misses          858      858           
Flag Coverage Δ
7.4 82.50% <100.00%> (ø)
8.0 82.52% <100.00%> (ø)
8.1 82.52% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/SDK/Resource/Detectors/Process.php 100.00% <100.00%> (ø)

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 a3a47a6...023e1bb. Read the comment docs.

@brettmc
brettmc merged commit 7940773 into open-telemetry:main Oct 9, 2022
@Nevay
Nevay deleted the fix/process-owner-triggers-warning-if-no-user branch April 19, 2023 19:11
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