Support registry show_user var definition property (elastic/package…

…-registry#266)
jen-huang · Mar 16, 2020 · e66fb3b · e66fb3b
1 parent 4a8fd0a
commit e66fb3b
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 6 deletions.
diff --git a/x-pack/plugins/ingest_manager/common/types/models/epm.ts b/x-pack/plugins/ingest_manager/common/types/models/epm.ts
@@ -190,6 +190,7 @@ export interface RegistryVarsEntry {
   description?: string;
   type: string;
   required?: boolean;
+  show_user?: boolean;
   multi?: boolean;
   default?: string | string[];
   os?: {

diff --git a/...nager/sections/agent_config/create_datasource_page/components/datasource_input_config.tsx b/...nager/sections/agent_config/create_datasource_page/components/datasource_input_config.tsx
@@ -15,6 +15,7 @@ import {
   EuiTitle,
 } from '@elastic/eui';
 import { DatasourceInput, RegistryVarsEntry } from '../../../../types';
+import { isAdvancedVar } from '../services';
 import { DatasourceInputVarField } from './datasource_input_var_field';
 
 export const DatasourceInputConfig: React.FunctionComponent<{
@@ -30,10 +31,10 @@ export const DatasourceInputConfig: React.FunctionComponent<{
 
   if (packageInputVars) {
     packageInputVars.forEach(varDef => {
-      if (varDef.required && !varDef.default) {
-        requiredVars.push(varDef);
-      } else {
+      if (isAdvancedVar(varDef)) {
         advancedVars.push(varDef);
+      } else {
+        requiredVars.push(varDef);
       }
     });
   }

diff --git a/...ections/agent_config/create_datasource_page/components/datasource_input_stream_config.tsx b/...ections/agent_config/create_datasource_page/components/datasource_input_stream_config.tsx
@@ -16,6 +16,7 @@ import {
   EuiButtonEmpty,
 } from '@elastic/eui';
 import { DatasourceInputStream, RegistryStream, RegistryVarsEntry } from '../../../../types';
+import { isAdvancedVar } from '../services';
 import { DatasourceInputVarField } from './datasource_input_var_field';
 
 export const DatasourceInputStreamConfig: React.FunctionComponent<{
@@ -31,10 +32,10 @@ export const DatasourceInputStreamConfig: React.FunctionComponent<{
 
   if (packageInputStream.vars && packageInputStream.vars.length) {
     packageInputStream.vars.forEach(varDef => {
-      if (varDef.required && !varDef.default) {
-        requiredVars.push(varDef);
-      } else {
+      if (isAdvancedVar(varDef)) {
         advancedVars.push(varDef);
+      } else {
+        requiredVars.push(varDef);
       }
     });
   }

diff --git a/...pplications/ingest_manager/sections/agent_config/create_datasource_page/services/index.ts b/...pplications/ingest_manager/sections/agent_config/create_datasource_page/services/index.ts
@@ -0,0 +1,6 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+export { isAdvancedVar } from './is_advanced_var';
diff --git a/...s/ingest_manager/sections/agent_config/create_datasource_page/services/is_advanced_var.ts b/...s/ingest_manager/sections/agent_config/create_datasource_page/services/is_advanced_var.ts
@@ -0,0 +1,13 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License;
+ * you may not use this file except in compliance with the Elastic License.
+ */
+import { RegistryVarsEntry } from '../../../../types';
+
+export const isAdvancedVar = (varDef: RegistryVarsEntry): boolean => {
+  if (varDef.show_user || (varDef.required && !varDef.default)) {
+    return false;
+  }
+  return true;
+};