Show warning in standalone mode for policies with fleet server

elastic · Apr 29, 2021 · a8f5ead · a8f5ead
1 parent 60f088c
commit a8f5ead
Show file tree

Hide file tree

Showing 3 changed files with 44 additions and 6 deletions.
diff --git a/...tions/fleet/sections/agents/components/agent_enrollment_flyout/agent_policy_selection.tsx b/...tions/fleet/sections/agents/components/agent_enrollment_flyout/agent_policy_selection.tsx
@@ -18,6 +18,7 @@ import { AgentPolicyPackageBadges } from '../agent_policy_package_badges';
 type Props = {
   agentPolicies?: AgentPolicy[];
   onAgentPolicyChange?: (key: string) => void;
+  excludeFleetServer?: boolean;
 } & (
   | {
       withKeySelection: true;
@@ -30,7 +31,7 @@ type Props = {
 
 export const EnrollmentStepAgentPolicy: React.FC<Props> = (props) => {
   const { notifications } = useStartServices();
-  const { withKeySelection, agentPolicies, onAgentPolicyChange } = props;
+  const { withKeySelection, agentPolicies, onAgentPolicyChange, excludeFleetServer } = props;
   const onKeyChange = props.withKeySelection && props.onKeyChange;
 
   const [isAuthenticationSettingsOpen, setIsAuthenticationSettingsOpen] = useState(false);
@@ -182,7 +183,10 @@ export const EnrollmentStepAgentPolicy: React.FC<Props> = (props) => {
       />
       <EuiSpacer size="m" />
       {selectedState.agentPolicyId && (
-        <AgentPolicyPackageBadges agentPolicyId={selectedState.agentPolicyId} />
+        <AgentPolicyPackageBadges
+          agentPolicyId={selectedState.agentPolicyId}
+          excludeFleetServer={excludeFleetServer}
+        />
       )}
       {withKeySelection && onKeyChange && (
         <>

diff --git a/...et/public/applications/fleet/sections/agents/components/agent_enrollment_flyout/steps.tsx b/...et/public/applications/fleet/sections/agents/components/agent_enrollment_flyout/steps.tsx
@@ -98,6 +98,7 @@ export const AgentPolicySelectionStep = ({
         withKeySelection={setSelectedAPIKeyId ? true : false}
         onKeyChange={setSelectedAPIKeyId}
         onAgentPolicyChange={onAgentPolicyChange}
+        excludeFleetServer={excludeFleetServer}
       />
     ),
   };

diff --git a/...leet/public/applications/fleet/sections/agents/components/agent_policy_package_badges.tsx b/...leet/public/applications/fleet/sections/agents/components/agent_policy_package_badges.tsx
@@ -6,8 +6,11 @@
  */
 
 import React, { useMemo } from 'react';
+import { i18n } from '@kbn/i18n';
 import { FormattedMessage } from '@kbn/i18n/react';
-import { EuiSpacer, EuiText, EuiFlexGroup, EuiFlexItem, EuiBadge } from '@elastic/eui';
+import { EuiSpacer, EuiText, EuiFlexGroup, EuiFlexItem, EuiBadge, EuiCallOut } from '@elastic/eui';
+
+import { FLEET_SERVER_PACKAGE } from '../../../../../../common/constants';
 
 import type { PackagePolicy, PackagePolicyPackage } from '../../../types';
 import { useGetOneAgentPolicy } from '../../../hooks';
@@ -16,11 +19,13 @@ import { PackageIcon } from '../../../components/package_icon';
 interface Props {
   agentPolicyId: string;
   hideTitle?: boolean;
+  excludeFleetServer?: boolean;
 }
 
 export const AgentPolicyPackageBadges: React.FunctionComponent<Props> = ({
   agentPolicyId,
   hideTitle,
+  excludeFleetServer,
 }) => {
   const agentPolicyRequest = useGetOneAgentPolicy(agentPolicyId);
   const agentPolicy = agentPolicyRequest.data ? agentPolicyRequest.data.item : null;
@@ -49,6 +54,13 @@ export const AgentPolicyPackageBadges: React.FunctionComponent<Props> = ({
     return null;
   }
 
+  const showFleetServerWarning =
+    excludeFleetServer && packages.some((pkg) => pkg.name === FLEET_SERVER_PACKAGE);
+
+  const collectedIntegrationsCount = packages.filter(
+    (pkg) => !excludeFleetServer || pkg.name !== FLEET_SERVER_PACKAGE
+  ).length;
+
   return (
     <>
       {!hideTitle && (
@@ -58,8 +70,8 @@ export const AgentPolicyPackageBadges: React.FunctionComponent<Props> = ({
               id="xpack.fleet.agentReassignPolicy.policyDescription"
               defaultMessage="The selected agent policy will collect data for {count, plural, one {{countValue} integration} other {{countValue} integrations}}:"
               values={{
-                count: packages.length,
-                countValue: <b>{packages.length}</b>,
+                count: collectedIntegrationsCount,
+                countValue: <b>{collectedIntegrationsCount}</b>,
               }}
             />
           </EuiText>
@@ -68,7 +80,11 @@ export const AgentPolicyPackageBadges: React.FunctionComponent<Props> = ({
       )}
       {packages.map((pkg, idx) => {
         return (
-          <EuiBadge key={idx} color="hollow">
+          <EuiBadge
+            key={idx}
+            color="hollow"
+            isDisabled={excludeFleetServer && pkg.name === FLEET_SERVER_PACKAGE}
+          >
             <EuiFlexGroup direction="row" gutterSize="xs" alignItems="center">
               <EuiFlexItem grow={false}>
                 <PackageIcon
@@ -89,6 +105,23 @@ export const AgentPolicyPackageBadges: React.FunctionComponent<Props> = ({
           </EuiBadge>
         );
       })}
+      {showFleetServerWarning && (
+        <>
+          <EuiSpacer size="s" />
+          <EuiCallOut
+            size="s"
+            color="warning"
+            iconType="alert"
+            title={i18n.translate(
+              'xpack.fleet.agentReassignPolicy.packageBadgeFleetServerWarning',
+              {
+                defaultMessage:
+                  'The Fleet Server integration will not collect data in standalone mode.',
+              }
+            )}
+          />
+        </>
+      )}
     </>
   );
 };