Skip to content

Commit ec19732

Browse files
authored
[DBMON-5572] Enable collect_settings by default for DBM enabled integrations (DataDog#21172)
* Mysql: enable collect_settings by default * Postgres: enable collect_settings by default * SqlServer: enable collect_settings by default * Add changelog
1 parent 0647aff commit ec19732

12 files changed

Lines changed: 16 additions & 13 deletions

File tree

mysql/assets/configuration/spec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ files:
395395
Enable collection of performance_schema.global_variables. Requires `dbm: true`.
396396
value:
397397
type: boolean
398-
example: false
398+
example: true
399399
- name: collection_interval
400400
description: |
401401
Set the database settings collection interval (in seconds). Each collection involves a single query to

mysql/changelog.d/21172.added

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Enable collect_settings by default for DBM enabled integrations

mysql/datadog_checks/mysql/data/conf.yaml.example

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -385,10 +385,10 @@ instances:
385385
#
386386
# collect_settings:
387387

388-
## @param enabled - boolean - optional - default: false
388+
## @param enabled - boolean - optional - default: true
389389
## Enable collection of performance_schema.global_variables. Requires `dbm: true`.
390390
#
391-
# enabled: false
391+
# enabled: true
392392

393393
## @param collection_interval - number - optional - default: 600
394394
## Set the database settings collection interval (in seconds). Each collection involves a single query to

mysql/datadog_checks/mysql/metadata.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def __init__(self, check, config, connection_args):
5252
self._databases_data_collection_interval = config.schemas_config.get(
5353
"collection_interval", DEFAULT_DATABASES_DATA_COLLECTION_INTERVAL
5454
)
55-
self._settings_enabled = is_affirmative(config.settings_config.get('enabled', False))
55+
self._settings_enabled = is_affirmative(config.settings_config.get('enabled', True))
5656

5757
self._settings_collection_interval = float(
5858
config.settings_config.get('collection_interval', DEFAULT_SETTINGS_COLLECTION_INTERVAL)

postgres/assets/configuration/spec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -609,7 +609,7 @@ files:
609609
Enable collection of pg_settings. Requires `dbm: true`.
610610
value:
611611
type: boolean
612-
example: false
612+
example: true
613613
- name: collection_interval
614614
description: |
615615
Set the database settings collection interval (in seconds). Each collection involves a single query to

postgres/changelog.d/21172.added

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Enable collect_settings by default for DBM enabled integrations

postgres/datadog_checks/postgres/data/conf.yaml.example

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -530,10 +530,10 @@ instances:
530530
#
531531
# collect_settings:
532532

533-
## @param enabled - boolean - optional - default: false
533+
## @param enabled - boolean - optional - default: true
534534
## Enable collection of pg_settings. Requires `dbm: true`.
535535
#
536-
# enabled: false
536+
# enabled: true
537537

538538
## @param collection_interval - number - optional - default: 600
539539
## Set the database settings collection interval (in seconds). Each collection involves a single query to

postgres/datadog_checks/postgres/metadata.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -263,8 +263,8 @@ def __init__(self, check, config, shutdown_callback):
263263
self._check = check
264264
self._config = config
265265
self.db_pool = self._check.db_pool
266-
self._collect_extensions_enabled = is_affirmative(config.settings_metadata_config.get("enabled", False))
267-
self._collect_pg_settings_enabled = is_affirmative(config.settings_metadata_config.get("enabled", False))
266+
self._collect_extensions_enabled = is_affirmative(config.settings_metadata_config.get("enabled", True))
267+
self._collect_pg_settings_enabled = is_affirmative(config.settings_metadata_config.get("enabled", True))
268268
self._collect_schemas_enabled = is_affirmative(config.schemas_metadata_config.get("enabled", False))
269269
self._is_schemas_collection_in_progress = False
270270
self._pg_settings_cached = None

sqlserver/assets/configuration/spec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -534,7 +534,7 @@ files:
534534
Enable collection of sys.configurations. Requires `dbm: true`.
535535
value:
536536
type: boolean
537-
example: false
537+
example: true
538538
- name: collection_interval
539539
description: |
540540
Set the database settings collection interval (in seconds). Each collection involves a single query to

sqlserver/changelog.d/21172.added

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Enable collect_settings by default for DBM enabled integrations

0 commit comments

Comments
 (0)