Skip to content

Comments

feat: add DynamoDB Distributed Lock Observability package#30

Merged
ncipollina merged 1 commit intomainfrom
feature/observability-package
Aug 28, 2025
Merged

feat: add DynamoDB Distributed Lock Observability package#30
ncipollina merged 1 commit intomainfrom
feature/observability-package

Conversation

@ncipollina
Copy link
Collaborator

Summary

  • Add new DynamoDb.DistributedLock.Observability package with OpenTelemetry integration
  • Implement MeterProviderBuilder.AddDynamoDbDistributedLock() extension method for simplified metrics configuration
  • Update documentation with comprehensive observability examples and usage patterns

Changes Made

  • ✅ Created new observability project targeting .NET 8.0 and 9.0
  • ✅ Added MeterProviderBuilder extension methods with proper XML documentation
  • ✅ Updated solution file to include new project
  • ✅ Enhanced README.md with observability package documentation
  • ✅ Updated CLAUDE.md with new architecture components
  • ✅ Bumped version to 1.2.1 in Directory.Build.props

Usage

using DynamoDb.DistributedLock.Observability;

services.AddOpenTelemetry()
    .WithMetrics(metrics => 
        metrics.AddDynamoDbDistributedLock() // <-- New extension method
    );

Test plan

  • Solution builds successfully
  • New package structure follows existing conventions
  • Documentation is comprehensive and includes examples
  • Extension method properly references MetricNames.MeterName constant

🤖 Generated with Claude Code

- Add DynamoDb.DistributedLock.Observability project with OpenTelemetry integration
- Implement MeterProviderBuilder extension methods for easy metrics configuration
- Update README with comprehensive observability documentation
- Add observability package to solution and build configuration
- Update CLAUDE.md with new architecture components

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ncipollina ncipollina requested a review from Copilot August 28, 2025 12:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive observability support for the DynamoDB Distributed Lock library through a new dedicated package and updated documentation. It enables simplified OpenTelemetry metrics integration while also updating package dependencies to their latest versions.

  • Introduces a new DynamoDb.DistributedLock.Observability package with OpenTelemetry integration
  • Adds MeterProviderBuilder.AddDynamoDbDistributedLock() extension method for simplified metrics configuration
  • Updates package dependencies across the solution and enhances documentation with observability examples

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/DynamoDb.DistributedLock.Observability/DynamoDb.DistributedLock.Observability.csproj New project file for the observability package targeting .NET 8.0 and 9.0
src/DynamoDb.DistributedLock.Observability/MeterProviderBuilderExtensions.cs Extension method implementation for simplified OpenTelemetry metrics registration
DynamoDb.DistributedLock.sln Solution file updated to include the new observability project
Directory.Build.props Centralized NuGet packaging properties and version bump to 1.2.1
src/DynamoDb.DistributedLock/DynamoDb.DistributedLock.csproj Package dependency updates and moved common properties to Directory.Build.props
test/DynamoDb.DistributedLock.Tests/DynamoDb.DistributedLock.Tests.csproj Updated test dependencies to latest versions
README.md Enhanced documentation with observability package usage examples and restructured telemetry section
CLAUDE.md Updated architecture documentation to reflect new observability components

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ncipollina ncipollina merged commit 41d6d30 into main Aug 28, 2025
1 check passed
@ncipollina ncipollina deleted the feature/observability-package branch August 28, 2025 12:54
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.

1 participant