Skip to content

Conversation

@lukeina2z
Copy link
Contributor

Changes

Please add Luke Zhang @lukeina2z as the owner of the AWS components.

@lukeina2z lukeina2z requested a review from a team as a code owner October 16, 2025 17:09
@github-actions github-actions bot added the infra Infra work - CI/CD, code coverage, linters label Oct 16, 2025
@codecov
Copy link

codecov bot commented Oct 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.64%. Comparing base (c9c069d) to head (2a62f69).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3242      +/-   ##
==========================================
+ Coverage   66.37%   70.64%   +4.26%     
==========================================
  Files         428      440      +12     
  Lines       16094    17213    +1119     
==========================================
+ Hits        10683    12160    +1477     
+ Misses       5411     5053     -358     
Flag Coverage Δ
unittests-Contrib.Shared.Tests 86.25% <ø> (-0.07%) ⬇️
unittests-Exporter.Geneva 53.33% <ø> (+8.33%) ⬆️
unittests-Exporter.InfluxDB 95.14% <ø> (+1.21%) ⬆️
unittests-Exporter.Instana 74.86% <ø> (ø)
unittests-Exporter.OneCollector 94.61% <ø> (-0.06%) ⬇️
unittests-Extensions 91.12% <ø> (ø)
unittests-Extensions.Enrichment 100.00% <ø> (ø)
unittests-Extensions.Enrichment.AspNetCore 86.27% <ø> (ø)
unittests-Extensions.Enrichment.Http 94.33% <ø> (+0.22%) ⬆️
unittests-Instrumentation.AWS 83.80% <ø> (-1.45%) ⬇️
unittests-Instrumentation.AspNet 78.14% <ø> (+0.22%) ⬆️
unittests-Instrumentation.AspNetCore 70.76% <ø> (ø)
unittests-Instrumentation.Cassandra 23.52% <ø> (ø)
unittests-Instrumentation.ConfluentKafka 14.10% <ø> (-0.14%) ⬇️
unittests-Instrumentation.ElasticsearchClient 80.12% <ø> (ø)
unittests-Instrumentation.EntityFrameworkCore 80.80% <ø> (ø)
unittests-Instrumentation.EventCounters 77.27% <ø> (ø)
unittests-Instrumentation.GrpcCore 91.42% <ø> (ø)
unittests-Instrumentation.GrpcNetClient 79.61% <ø> (ø)
unittests-Instrumentation.Hangfire 84.61% <ø> (ø)
unittests-Instrumentation.Http 74.18% <ø> (+12.76%) ⬆️
unittests-Instrumentation.Owin 88.62% <ø> (ø)
unittests-Instrumentation.Process 100.00% <ø> (ø)
unittests-Instrumentation.Quartz 78.76% <ø> (ø)
unittests-Instrumentation.Runtime 100.00% <ø> (ø)
unittests-Instrumentation.ServiceFabricRemoting 34.54% <ø> (ø)
unittests-Instrumentation.SqlClient 87.29% <ø> (-2.23%) ⬇️
unittests-Instrumentation.StackExchangeRedis 71.80% <ø> (+2.62%) ⬆️
unittests-Instrumentation.Wcf 79.68% <ø> (+30.03%) ⬆️
unittests-OpAmp.Client 71.55% <ø> (+0.12%) ⬆️
unittests-PersistentStorage 74.91% <ø> (+1.67%) ⬆️
unittests-Resources.AWS 74.42% <ø> (+2.03%) ⬆️
unittests-Resources.Azure 85.31% <ø> (ø)
unittests-Resources.Container 67.34% <ø> (ø)
unittests-Resources.Gcp 71.42% <ø> (ø)
unittests-Resources.Host 71.85% <ø> (ø)
unittests-Resources.OperatingSystem 76.98% <ø> (+1.57%) ⬆️
unittests-Resources.Process 100.00% <ø> (ø)
unittests-Resources.ProcessRuntime 79.59% <ø> (-20.41%) ⬇️
unittests-Sampler.AWS 93.23% <ø> (-1.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 89 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Kielek
Copy link
Member

Kielek commented Oct 17, 2025

@srprash, @normj, @rypdal, @Oberon00, @muhammad-othman

Could you please review and approve this PR? @lukeina2z would like to help you with maintaining these packages as an owner.

Copy link
Contributor

@normj normj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would appreciate @lukeina2z help maintaining the AWS components.

Copy link
Member

@Kielek Kielek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lukeina2z, could you please also update all related README files?

Unfortunately, it is still manual task. If you think that it is worth to create some automation script, we can also accept it as separate PR.

| Status | |
| ----------- | --------- |
| Stability | [Stable](../../README.md#stable) |
| Code Owners | [@srprash](https://github.com/srprash), [@normj](https://github.com/normj) |

@github-actions github-actions bot added comp:extensions.aws Things related to OpenTelemetry.Extensions.AWS documentation Improvements or additions to documentation comp:instrumentation.aws Things related to OpenTelemetry.Instrumentation.AWS comp:instrumentation.awslambda Things related to OpenTelemetry.Instrumentation.AWSLambda comp:resources.aws Things related to OpenTelemetry.Resources.AWS comp:sampler.aws Things related to OpenTelemetry.Samplers.AWS labels Oct 24, 2025
@lukeina2z
Copy link
Contributor Author

@lukeina2z, could you please also update all related README files?

Unfortunately, it is still manual task. If you think that it is worth to create some automation script, we can also accept it as separate PR.

| Status | |
| ----------- | --------- |
| Stability | [Stable](../../README.md#stable) |
| Code Owners | [@srprash](https://github.com/srprash), [@normj](https://github.com/normj) |

ReadMe files updated.

@Kielek Kielek merged commit 2ee3971 into open-telemetry:main Oct 24, 2025
233 checks passed
@lukeina2z lukeina2z deleted the prowner-a branch October 24, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:extensions.aws Things related to OpenTelemetry.Extensions.AWS comp:instrumentation.aws Things related to OpenTelemetry.Instrumentation.AWS comp:instrumentation.awslambda Things related to OpenTelemetry.Instrumentation.AWSLambda comp:resources.aws Things related to OpenTelemetry.Resources.AWS comp:sampler.aws Things related to OpenTelemetry.Samplers.AWS documentation Improvements or additions to documentation infra Infra work - CI/CD, code coverage, linters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants