Addressing feedback from PR adding some comments

Author: leenanofal

pet_clinic_billing_service/billing_service/views.py

@@ -175,7 +175,9 @@ def list(self, request, pk=None):
             # Cache miss
             span.set_attribute("billing_summary_cache_hit", 0)
 
+            # Sleep to simulate high latency when there's a cache miss 
             time.sleep(2)
+            
             billings = Billing.objects.all()
 
             summary = {

pet_clinic_billing_service/create-metric-filters.sh

@@ -5,6 +5,7 @@ LOG_GROUP_NAME="aws/spans"
 REGION=${AWS_REGION:-us-east-1}
 NAMESPACE="CustomBilling"
 
+# TODO: [@lnnofal] convert it to CDK and hooked up with other CDK stacks (for EKS)
 echo "Creating log group if it doesn't exist..."
 aws logs create-log-group --log-group-name "$LOG_GROUP_NAME" --region "$REGION" 2>/dev/null || echo "Log group already exists or creation failed"