do not log error for cmk, since that will show up as a failure on our…

… dashboards (#1035)
microsoft · Sep 10, 2024 · d69c12b · d69c12b
1 parent ff171b2
commit d69c12b
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/Microsoft.Health.SqlServer/Features/Storage/SqlServerSchemaDataStore.cs b/src/Microsoft.Health.SqlServer/Features/Storage/SqlServerSchemaDataStore.cs
@@ -117,6 +117,12 @@ public async Task<int> UpsertInstanceSchemaInformationAsync(string name, SchemaI
                 throw;
             }
 
+            if (sqlEx.IsCMKError())
+            {
+                // do not log error if it is a CMK error, since it is the customer's issue and not a system issue
+                throw;
+            }
+
             _logger.LogError(sqlEx, "Error from SQL database on upserting InstanceSchema information");
             throw;
         }