Implements logging using OpenTelemetry as per #67

[he1senbrg]

closes #67

[ivinjabraham]

Hefty commit, I will have to learn how all of this works myself before I review it. Will try and get it done before the weekend.

[Copilot]

Pull Request Overview

This PR implements comprehensive OpenTelemetry logging and tracing as per issue #67. The changes introduce distributed tracing capabilities with proper instrumentation throughout the application.

Key changes:

• Added OpenTelemetry dependencies and configuration for tracing and metrics
• Instrumented key functions with #[tracing::instrument] attributes
• Implemented graceful shutdown with proper telemetry cleanup
• Added Debug derives to structs for better tracing support

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
Cargo.toml	Added OpenTelemetry dependencies for tracing, metrics, and OTLP export
src/main.rs	Implemented complete OpenTelemetry setup with tracer/meter providers and graceful shutdown
src/routes.rs	Added tracing instrumentation to router setup function
src/models/*.rs	Added Debug derives to structs for better tracing support
src/graphql/queries/*.rs	Added tracing instrumentation and logging to GraphQL query functions
src/graphql/mutations/attendance_mutations.rs	Added tracing instrumentation to mutation functions
src/daily_task/mod.rs	Added tracing instrumentation to daily task functions

Comments suppressed due to low confidence (1)

src/main.rs:1

• The trace sampler is set to 1.0 (100% sampling) which may generate excessive telemetry data in production. Consider making the sampling rate configurable or using a lower default value for production environments.

use async_graphql::EmptySubscription;

---

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

[Copilot]

[nitpick] The Debug derive should be placed on its own line for consistency with other derive attributes in the codebase. Consider separating it: #[derive(InputObject)] and #[derive(Debug)] on separate lines.

Suggested change

	#[derive(InputObject, Debug)]
	#[derive(InputObject)]
	#[derive(Debug)]

[Copilot]

The deployment environment is hardcoded to 'develop'. This should be configurable through environment variables or the Config struct to support different deployment environments (development, staging, production).

[Copilot]

The metrics export interval is hardcoded to 30 seconds. Consider making this configurable through environment variables or the Config struct to allow tuning for different environments.

[ivinjabraham]

Too many changes in one commit, too annoying to review.