Replace logs/metrics strings with const

elastic · Nov 3, 2020 · 61a78dd · 61a78dd
1 parent c2af3aa
commit 61a78dd
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 9 deletions.
diff --git a/x-pack/plugins/ingest_manager/common/types/models/agent_policy.ts b/x-pack/plugins/ingest_manager/common/types/models/agent_policy.ts
@@ -5,6 +5,7 @@
  */
 import { PackagePolicy, PackagePolicyPackage } from './package_policy';
 import { Output } from './output';
+import { DataType, ValueOf } from '../../index';
 
 export enum AgentPolicyStatus {
   Active = 'active',
@@ -16,7 +17,7 @@ export interface NewAgentPolicy {
   namespace: string;
   description?: string;
   is_default?: boolean;
-  monitoring_enabled?: Array<'logs' | 'metrics'>;
+  monitoring_enabled?: Array<ValueOf<DataType>>;
 }
 
 export interface AgentPolicy extends NewAgentPolicy {

diff --git a/...public/applications/ingest_manager/sections/agent_policy/components/agent_policy_form.tsx b/...public/applications/ingest_manager/sections/agent_policy/components/agent_policy_form.tsx
@@ -23,6 +23,7 @@ import {
 import { FormattedMessage } from '@kbn/i18n/react';
 import { i18n } from '@kbn/i18n';
 import styled from 'styled-components';
+import { dataTypes } from '../../../../../../common';
 import { NewAgentPolicy, AgentPolicy } from '../../../types';
 import { isValidNamespace } from '../../../services';
 import { AgentPolicyDeleteProvider } from './agent_policy_delete_provider';
@@ -211,7 +212,7 @@ export const AgentPolicyForm: React.FunctionComponent<Props> = ({
         <EuiCheckboxGroup
           options={[
             {
-              id: 'logs',
+              id: dataTypes.Logs,
               label: (
                 <>
                   <FormattedMessage
@@ -233,7 +234,7 @@ export const AgentPolicyForm: React.FunctionComponent<Props> = ({
               ),
             },
             {
-              id: 'metrics',
+              id: dataTypes.Metrics,
               label: (
                 <>
                   <FormattedMessage
@@ -263,7 +264,7 @@ export const AgentPolicyForm: React.FunctionComponent<Props> = ({
             { logs: false, metrics: false }
           )}
           onChange={(id) => {
-            if (id !== 'logs' && id !== 'metrics') {
+            if (id !== dataTypes.Logs && id !== dataTypes.Metrics) {
               return;
             }
 

diff --git a/...cations/ingest_manager/sections/agent_policy/list_page/components/create_agent_policy.tsx b/...cations/ingest_manager/sections/agent_policy/list_page/components/create_agent_policy.tsx
@@ -21,6 +21,7 @@ import {
   EuiFlyoutProps,
   EuiSpacer,
 } from '@elastic/eui';
+import { dataTypes } from '../../../../../../../common';
 import { NewAgentPolicy, AgentPolicy } from '../../../../types';
 import { useCapabilities, useCore, sendCreateAgentPolicy } from '../../../../hooks';
 import { AgentPolicyForm, agentPolicyFormValidation } from '../../components';
@@ -44,7 +45,7 @@ export const CreateAgentPolicyFlyout: React.FunctionComponent<Props> = ({
     description: '',
     namespace: 'default',
     is_default: undefined,
-    monitoring_enabled: ['logs', 'metrics'],
+    monitoring_enabled: Object.values(dataTypes),
   });
   const [isLoading, setIsLoading] = useState<boolean>(false);
   const [withSysMonitoring, setWithSysMonitoring] = useState<boolean>(true);

diff --git a/x-pack/plugins/ingest_manager/server/services/agent_policy.ts b/x-pack/plugins/ingest_manager/server/services/agent_policy.ts
@@ -25,6 +25,7 @@ import {
   DeleteAgentPolicyResponse,
   Settings,
   storedPackagePoliciesToAgentInputs,
+  dataTypes,
 } from '../../common';
 import { AgentPolicyNameExistsError } from '../errors';
 import { createAgentPolicyAction, listAgents } from './agents';
@@ -538,8 +539,8 @@ class AgentPolicyService {
               monitoring: {
                 use_output: defaultOutput.name,
                 enabled: true,
-                logs: agentPolicy.monitoring_enabled.indexOf('logs') >= 0,
-                metrics: agentPolicy.monitoring_enabled.indexOf('metrics') >= 0,
+                logs: agentPolicy.monitoring_enabled.includes(dataTypes.Logs),
+                metrics: agentPolicy.monitoring_enabled.includes(dataTypes.Metrics),
               },
             },
           }

diff --git a/x-pack/plugins/ingest_manager/server/types/models/agent_policy.ts b/x-pack/plugins/ingest_manager/server/types/models/agent_policy.ts
@@ -5,14 +5,16 @@
  */
 import { schema } from '@kbn/config-schema';
 import { PackagePolicySchema, NamespaceSchema } from './package_policy';
-import { AgentPolicyStatus } from '../../../common';
+import { AgentPolicyStatus, dataTypes } from '../../../common';
 
 const AgentPolicyBaseSchema = {
   name: schema.string({ minLength: 1 }),
   namespace: NamespaceSchema,
   description: schema.maybe(schema.string()),
   monitoring_enabled: schema.maybe(
-    schema.arrayOf(schema.oneOf([schema.literal('logs'), schema.literal('metrics')]))
+    schema.arrayOf(
+      schema.oneOf([schema.literal(dataTypes.Logs), schema.literal(dataTypes.Metrics)])
+    )
   ),
 };