Skip to content

Commit

Permalink
Add schema tag to db_fragmentation metrics
Browse files Browse the repository at this point in the history
  • Loading branch information
azhou-datadog committed Dec 17, 2024
1 parent ac01180 commit dde6b06
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"query": """SELECT
DB_NAME(DDIPS.database_id) as database_name,
OBJECT_NAME(DDIPS.object_id, DDIPS.database_id) as object_name,
OBJECT_SCHEMA_NAME(DDIPS.object_id, DDIPS.database_id) as "schema",
DDIPS.index_id as index_id,
I.name as index_name,
DDIPS.fragment_count as fragment_count,
Expand All @@ -28,6 +29,7 @@
"columns": [
{"name": "database_name", "type": "tag"},
{"name": "object_name", "type": "tag"},
{"name": "schema", "type": "tag"},
{"name": "index_id", "type": "tag"},
{"name": "index_name", "type": "tag"},
{"name": "database.fragment_count", "type": "gauge"},
Expand Down
8 changes: 4 additions & 4 deletions sqlserver/metadata.csv
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,15 @@ sqlserver.buffer.page_writes,gauge,,page,,Indicates the number of physical datab
sqlserver.cache.object_counts,gauge,,object,,Number of cache objects in the cache. (Perf. Counter: `Plan Cache - Cache Object Counts`),0,sql_server,cache obj counts,,
sqlserver.cache.pages,gauge,,object,,Number of 8-kilobyte (KB) pages used by cache objects. (Perf. Counter: `Plan Cache - Cache Pages`),0,sql_server,cache pages,,
sqlserver.database.active_transactions,gauge,,transaction,,Number of active transactions across all databases on the SQL Server instance. Tags: `db`. (Perf. Counter: `Databases - Active Transactions`).,0,sql_server,database trans active,,
sqlserver.database.avg_fragment_size_in_pages,gauge,,,,"The average number of pages in one fragment on the leaf level of an IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `index_id`, `index_name`",0,sql_server,avg fragment size in pages,,
sqlserver.database.avg_fragmentation_in_percent,gauge,,,,"Logical fragmentation for indexes, or extent fragmentation for heaps in the IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `index_id`, `index_name`",0,sql_server,avg fragmentation percent,,
sqlserver.database.avg_fragment_size_in_pages,gauge,,,,"The average number of pages in one fragment on the leaf level of an IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `schema`, `index_id`, `index_name`",0,sql_server,avg fragment size in pages,,
sqlserver.database.avg_fragmentation_in_percent,gauge,,,,"Logical fragmentation for indexes, or extent fragmentation for heaps in the IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `schema`, `index_id`, `index_name`",0,sql_server,avg fragmentation percent,,
sqlserver.database.backup_count,gauge,,,,The total count of successful backups made for a database. **Note:** This metric is not emitted on Azure managed databases. Tags: `db`,0,sql_server,backup count,,
sqlserver.database.backup_restore_throughput,gauge,,,,Read/write throughput for backup and restore operations of a database per second. Tags: `db`. (Perf. Counter: `Databases - Backup/Restore Throughput/sec`),0,sql_server,database restore throughput,,
sqlserver.database.files.size,gauge,,kibibyte,,"Current size of the database file. Tags: `db`, `file_id`, `file_type`, `file_name`, `file_location`, `database_files_state_desc`",0,sql_server,database file size,,
sqlserver.database.files.space_used,gauge,,kibibyte,,"Current used space of the database file. Tags: `db`, `file_id`, `file_type`, `file_name`, `file_location`, `database_files_state_desc`",0,sql_server,database file used size,,
sqlserver.database.files.state,gauge,,,,"Database file state: 0 = Online, 1 = Restoring, 2 = Recovering, 3 = Recovery_Pending, 4 = Suspect, 5 = Unknown, 6 = Offline, 7 = Defunct. Tags: `db`, `file_id`, `file_type`, `file_name`, `file_location`, `database_files_state_desc`",0,sql_server,database file status,,
sqlserver.database.fragment_count,gauge,,,,"The number of fragments in the leaf level of an IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `index_id`, `index_name`",0,sql_server,fragment count,,
sqlserver.database.index_page_count,gauge,,,,"Total number of index or data pages. Tags: `db`, `object_name`, `index_id`, `index_name`",0,sql_server,page count per index,,
sqlserver.database.fragment_count,gauge,,,,"The number of fragments in the leaf level of an IN_ROW_DATA allocation unit. Tags: `db`, `object_name`, `schema`, `index_id`, `index_name`",0,sql_server,fragment count,,
sqlserver.database.index_page_count,gauge,,,,"Total number of index or data pages. Tags: `db`, `object_name`, `schema`, `index_id`, `index_name`",0,sql_server,page count per index,,
sqlserver.database.is_in_standby,gauge,,,,"Whether or not the database is read-only for restore log. Tags: `db`, `database_state_desc`, `database_recovery_model_desc`",0,sql_server,is in standby,,
sqlserver.database.is_read_only,gauge,,,,"Whether or not the database is marked as READ_ONLY. 0 = READ_WRITE, 1 = READ_ONLY. Tags: `db`, `database_state_desc`, `database_recovery_model_desc`",0,sql_server,is read-only,,
sqlserver.database.is_sync_with_backup,gauge,,,,"Whether or not the database is marked for replication synchronization with backup. 0 = Not marked for replication sync, 1 = Marked for replication sync. Tags: `db`, `database_state_desc`, `database_recovery_model_desc`",0,sql_server,is sync with backup,,
Expand Down
27 changes: 14 additions & 13 deletions sqlserver/tests/test_database_metrics.py
Original file line number Diff line number Diff line change
Expand Up @@ -922,29 +922,29 @@ def test_sqlserver_db_fragmentation_metrics(
print(instance_docker_metrics)
mocked_results = [
[
('master', 'spt_fallback_db', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_dev', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_usg', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_monitor', 0, None, 1, 1.0, 1, 0.0),
('master', 'MSreplication_options', 0, None, 1, 1.0, 1, 0.0),
('master', 'spt_fallback_db', 'dbo', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_dev', 'dbo', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_usg', 'dbo', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_monitor', 'dbo', 0, None, 1, 1.0, 1, 0.0),
('master', 'MSreplication_options', 'dbo', 0, None, 1, 1.0, 1, 0.0),
],
[
('msdb', 'syscachedcredentials', 1, 'PK__syscache__F6D56B562DA81DC6', 0, 0.0, 0, 0.0),
('msdb', 'syscollector_blobs_internal', 1, 'PK_syscollector_blobs_internal_paremeter_name', 0, 0.0, 0, 0.0),
('msdb', 'syscachedcredentials', 'dbo', 1, 'PK__syscache__F6D56B562DA81DC6', 0, 0.0, 0, 0.0),
('msdb', 'syscollector_blobs_internal', 'dbo', 1, 'PK_syscollector_blobs_internal_paremeter_name', 0, 0.0, 0, 0.0),
],
[('datadog_test-1', 'ϑings', 1, 'thingsindex', 1, 1.0, 1, 0.0)],
[('datadog_test-1', 'ϑings', 'dbo', 1, 'thingsindex', 1, 1.0, 1, 0.0)],
]
mocked_results_tempdb = [
[('tempdb', '#TempExample__000000000008', 1, 'PK__#TempExa__3214EC278A26D67E', 1, 1.0, 1, 0.0)],
[('tempdb', '#TempExample__000000000008', 'dbo', 1, 'PK__#TempExa__3214EC278A26D67E', 1, 1.0, 1, 0.0)],
]

if db_fragmentation_object_names:
instance_docker_metrics['db_fragmentation_object_names'] = db_fragmentation_object_names
mocked_results = [
[
('master', 'spt_fallback_db', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_dev', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_usg', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_db', 'dbo', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_dev', 'dbo', 0, None, 0, 0.0, 0, 0.0),
('master', 'spt_fallback_usg', 'dbo', 0, None, 0, 0.0, 0, 0.0),
],
[],
[],
Expand Down Expand Up @@ -983,12 +983,13 @@ def test_sqlserver_db_fragmentation_metrics(
tags = sqlserver_check._config.tags
for result in mocked_results:
for row in result:
database_name, object_name, index_id, index_name, *metric_values = row
database_name, object_name, schema, index_id, index_name, *metric_values = row
metrics = zip(db_fragmentation_metrics.metric_names()[0], metric_values)
expected_tags = [
f'db:{database_name}',
f'database_name:{database_name}',
f'object_name:{object_name}',
f'schema:{schema}',
f'index_id:{index_id}',
f'index_name:{index_name}',
] + tags
Expand Down
2 changes: 1 addition & 1 deletion sqlserver/tests/test_metrics.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,7 @@ def test_check_db_fragmentation_metrics(
for metric_name, _, _ in DATABASE_FRAGMENTATION_METRICS:
for tag in db_tags:
aggregator.assert_metric_has_tag(metric_name, tag=tag)
for tag_prefix in ('index_id', 'index_name', 'object_name'):
for tag_prefix in ('index_id', 'index_name', 'object_name', 'schema'):
aggregator.assert_metric_has_tag_prefix(metric_name, tag_prefix=tag_prefix)


Expand Down

0 comments on commit dde6b06

Please sign in to comment.