Skip to content

Commit 7455bde

Browse files
authored
Add levels for registered deprecations (#117691)
1 parent 6fe0052 commit 7455bde

File tree

1 file changed

+25
-10
lines changed

1 file changed

+25
-10
lines changed

x-pack/plugins/monitoring/server/deprecations.ts

Lines changed: 25 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,27 +23,39 @@ export const deprecations = ({
2323
}: ConfigDeprecationFactory): ConfigDeprecation[] => {
2424
return [
2525
// This order matters. The "blanket rename" needs to happen at the end
26-
renameFromRoot('xpack.monitoring.max_bucket_size', 'monitoring.ui.max_bucket_size'),
27-
renameFromRoot('xpack.monitoring.min_interval_seconds', 'monitoring.ui.min_interval_seconds'),
26+
renameFromRoot('xpack.monitoring.max_bucket_size', 'monitoring.ui.max_bucket_size', {
27+
level: 'warning',
28+
}),
29+
renameFromRoot('xpack.monitoring.min_interval_seconds', 'monitoring.ui.min_interval_seconds', {
30+
level: 'warning',
31+
}),
2832
renameFromRoot(
2933
'xpack.monitoring.show_license_expiration',
30-
'monitoring.ui.show_license_expiration'
34+
'monitoring.ui.show_license_expiration',
35+
{ level: 'warning' }
3136
),
3237
renameFromRoot(
3338
'xpack.monitoring.ui.container.elasticsearch.enabled',
34-
'monitoring.ui.container.elasticsearch.enabled'
39+
'monitoring.ui.container.elasticsearch.enabled',
40+
{ level: 'warning' }
3541
),
3642
renameFromRoot(
3743
'xpack.monitoring.ui.container.logstash.enabled',
38-
'monitoring.ui.container.logstash.enabled'
44+
'monitoring.ui.container.logstash.enabled',
45+
{ level: 'warning' }
3946
),
40-
renameFromRoot('xpack.monitoring.elasticsearch', 'monitoring.ui.elasticsearch'),
41-
renameFromRoot('xpack.monitoring.ccs.enabled', 'monitoring.ui.ccs.enabled'),
47+
renameFromRoot('xpack.monitoring.elasticsearch', 'monitoring.ui.elasticsearch', {
48+
level: 'warning',
49+
}),
50+
renameFromRoot('xpack.monitoring.ccs.enabled', 'monitoring.ui.ccs.enabled', {
51+
level: 'warning',
52+
}),
4253
renameFromRoot(
4354
'xpack.monitoring.elasticsearch.logFetchCount',
44-
'monitoring.ui.elasticsearch.logFetchCount'
55+
'monitoring.ui.elasticsearch.logFetchCount',
56+
{ level: 'warning' }
4557
),
46-
renameFromRoot('xpack.monitoring', 'monitoring'),
58+
renameFromRoot('xpack.monitoring', 'monitoring', { level: 'warning' }),
4759
(config, fromPath, addDeprecation) => {
4860
const emailNotificationsEnabled = get(config, 'cluster_alerts.email_notifications.enabled');
4961
if (emailNotificationsEnabled && !get(config, CLUSTER_ALERTS_ADDRESS_CONFIG_KEY)) {
@@ -55,11 +67,14 @@ export const deprecations = ({
5567
`Add [${fromPath}.${CLUSTER_ALERTS_ADDRESS_CONFIG_KEY}] to your kibana configs."`,
5668
],
5769
},
70+
level: 'critical',
5871
});
5972
}
6073
return config;
6174
},
62-
rename('xpack_api_polling_frequency_millis', 'licensing.api_polling_frequency'),
75+
rename('xpack_api_polling_frequency_millis', 'licensing.api_polling_frequency', {
76+
level: 'warning',
77+
}),
6378

6479
// TODO: Add deprecations for "monitoring.ui.elasticsearch.username: elastic" and "monitoring.ui.elasticsearch.username: kibana".
6580
// TODO: Add deprecations for using "monitoring.ui.elasticsearch.ssl.certificate" without "monitoring.ui.elasticsearch.ssl.key", and

0 commit comments

Comments
 (0)