Merge branch 'main' into rule-url-variable

elastic · Nov 14, 2022 · 3dd3236 · 3dd3236
2 parents d7edbb8 + bebcd35
commit 3dd3236
Show file tree

Hide file tree

Showing 118 changed files with 3,023 additions and 1,206 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -15,8 +15,6 @@
 /test/functional/apps/discover/ @elastic/kibana-data-discovery
 /test/functional/apps/context/ @elastic/kibana-data-discovery
 /test/api_integration/apis/unified_field_list/ @elastic/kibana-data-discovery
-/x-pack/plugins/graph/ @elastic/kibana-data-discovery
-/x-pack/test/functional/apps/graph @elastic/kibana-data-discovery
 /src/plugins/unified_field_list/ @elastic/kibana-data-discovery
 /src/plugins/unified_histogram/ @elastic/kibana-data-discovery
 /src/plugins/saved_objects_finder/ @elastic/kibana-data-discovery
@@ -49,6 +47,8 @@
 /test/functional/apps/visualize/ @elastic/kibana-visualizations
 /src/plugins/expressions/ @elastic/kibana-visualizations
 /src/plugins/unified_search/ @elastic/kibana-visualizations
+/x-pack/plugins/graph/ @elastic/kibana-visualizations
+/x-pack/test/functional/apps/graph @elastic/kibana-visualizations
 
 # Application Services
 /examples/dashboard_embeddable_examples/ @elastic/kibana-app-services

diff --git a/packages/core/apps/core-apps-browser-internal/src/status/lib/load_status.test.ts b/packages/core/apps/core-apps-browser-internal/src/status/lib/load_status.test.ts
@@ -62,17 +62,9 @@ const mockedResponse: StatusResponse = {
       },
     },
     elasticsearch_client: {
-      protocol: 'https',
-      connectedNodes: 3,
-      nodesWithActiveSockets: 3,
-      nodesWithIdleSockets: 1,
       totalActiveSockets: 25,
       totalIdleSockets: 2,
       totalQueuedRequests: 0,
-      mostActiveNodeSockets: 15,
-      averageActiveSocketsPerNode: 8,
-      mostIdleNodeSockets: 2,
-      averageIdleSocketsPerNode: 0.5,
     },
     process: {
       pid: 1,

diff --git a/...core/metrics/core-metrics-collectors-server-internal/src/get_agents_sockets_stats.test.ts b/...core/metrics/core-metrics-collectors-server-internal/src/get_agents_sockets_stats.test.ts
@@ -32,7 +32,7 @@ describe('getAgentsSocketsStats()', () => {
       },
     });
 
-    const agent2 = getHttpAgentMock({
+    const agent2 = getHttpsAgentMock({
       sockets: {
         node1: [mockSocket, mockSocket, mockSocket],
         node4: [mockSocket],
@@ -47,101 +47,9 @@ describe('getAgentsSocketsStats()', () => {
 
     const stats = getAgentsSocketsStats(new Set<Agent>([agent1, agent2]));
     expect(stats).toEqual({
-      averageActiveSocketsPerNode: 2.6666666666666665,
-      averageIdleSocketsPerNode: 4.5,
-      connectedNodes: 4,
-      mostActiveNodeSockets: 6,
-      mostIdleNodeSockets: 8,
-      nodesWithActiveSockets: 3,
-      nodesWithIdleSockets: 2,
-      protocol: 'http',
       totalActiveSockets: 8,
       totalIdleSockets: 9,
       totalQueuedRequests: 6,
     });
   });
-
-  it('takes into account Agent types to determine the `protocol`', () => {
-    const httpAgent = getHttpAgentMock({
-      sockets: { node1: [mockSocket] },
-      freeSockets: {},
-      requests: {},
-    });
-
-    const httpsAgent = getHttpsAgentMock({
-      sockets: { node1: [mockSocket] },
-      freeSockets: {},
-      requests: {},
-    });
-
-    const noAgents = new Set<Agent>();
-    const httpAgents = new Set<Agent>([httpAgent, httpAgent]);
-    const httpsAgents = new Set<Agent>([httpsAgent, httpsAgent]);
-    const mixedAgents = new Set<Agent>([httpAgent, httpsAgent]);
-
-    expect(getAgentsSocketsStats(noAgents).protocol).toEqual('none');
-    expect(getAgentsSocketsStats(httpAgents).protocol).toEqual('http');
-    expect(getAgentsSocketsStats(httpsAgents).protocol).toEqual('https');
-    expect(getAgentsSocketsStats(mixedAgents).protocol).toEqual('mixed');
-  });
-
-  it('does not take into account those Agents that have not had any connection to any node', () => {
-    const pristineAgentProps = {
-      sockets: {},
-      freeSockets: {},
-      requests: {},
-    };
-    const agent1 = getHttpAgentMock(pristineAgentProps);
-    const agent2 = getHttpAgentMock(pristineAgentProps);
-    const agent3 = getHttpAgentMock(pristineAgentProps);
-
-    const stats = getAgentsSocketsStats(new Set<Agent>([agent1, agent2, agent3]));
-
-    expect(stats).toEqual({
-      averageActiveSocketsPerNode: 0,
-      averageIdleSocketsPerNode: 0,
-      connectedNodes: 0,
-      mostActiveNodeSockets: 0,
-      mostIdleNodeSockets: 0,
-      nodesWithActiveSockets: 0,
-      nodesWithIdleSockets: 0,
-      protocol: 'none',
-      totalActiveSockets: 0,
-      totalIdleSockets: 0,
-      totalQueuedRequests: 0,
-    });
-  });
-
-  it('takes into account those Agents that have hold mappings to one or more nodes, but that do not currently have any pending requests, active connections or idle connections', () => {
-    const emptyAgentProps = {
-      sockets: {
-        node1: [],
-      },
-      freeSockets: {
-        node2: [],
-      },
-      requests: {
-        node3: [],
-      },
-    };
-
-    const agent1 = getHttpAgentMock(emptyAgentProps);
-    const agent2 = getHttpAgentMock(emptyAgentProps);
-
-    const stats = getAgentsSocketsStats(new Set<Agent>([agent1, agent2]));
-
-    expect(stats).toEqual({
-      averageActiveSocketsPerNode: 0,
-      averageIdleSocketsPerNode: 0,
-      connectedNodes: 3,
-      mostActiveNodeSockets: 0,
-      mostIdleNodeSockets: 0,
-      nodesWithActiveSockets: 0,
-      nodesWithIdleSockets: 0,
-      protocol: 'http',
-      totalActiveSockets: 0,
-      totalIdleSockets: 0,
-      totalQueuedRequests: 0,
-    });
-  });
 });
diff --git a/...ages/core/metrics/core-metrics-collectors-server-internal/src/get_agents_sockets_stats.ts b/...ages/core/metrics/core-metrics-collectors-server-internal/src/get_agents_sockets_stats.ts
@@ -7,33 +7,20 @@
  */
 
 import { NetworkAgent } from '@kbn/core-elasticsearch-client-server-internal';
-import { Agent as HttpsAgent } from 'https';
-import { mean } from 'lodash';
-import type {
-  ElasticsearchClientProtocol,
-  ElasticsearchClientsMetrics,
-} from '@kbn/core-metrics-server';
+import type { ElasticsearchClientsMetrics } from '@kbn/core-metrics-server';
 
 export const getAgentsSocketsStats = (agents: Set<NetworkAgent>): ElasticsearchClientsMetrics => {
   const nodes = new Set<string>();
   let totalActiveSockets = 0;
   let totalIdleSockets = 0;
   let totalQueuedRequests = 0;
-  let http: boolean = false;
-  let https: boolean = false;
-
-  const nodesWithActiveSockets: Record<string, number> = {};
-  const nodesWithIdleSockets: Record<string, number> = {};
 
   agents.forEach((agent) => {
     const agentRequests = Object.entries(agent.requests) ?? [];
     const agentSockets = Object.entries(agent.sockets) ?? [];
     const agentFreeSockets = Object.entries(agent.freeSockets) ?? [];
 
     if (agentRequests.length || agentSockets.length || agentFreeSockets.length) {
-      if (agent instanceof HttpsAgent) https = true;
-      else http = true;
-
       agentRequests.forEach(([node, queue]) => {
         nodes.add(node);
         totalQueuedRequests += queue?.length ?? 0;
@@ -43,39 +30,19 @@ export const getAgentsSocketsStats = (agents: Set<NetworkAgent>): ElasticsearchC
         nodes.add(node);
         const activeSockets = sockets?.length ?? 0;
         totalActiveSockets += activeSockets;
-        nodesWithActiveSockets[node] = (nodesWithActiveSockets[node] ?? 0) + activeSockets;
       });
 
       agentFreeSockets.forEach(([node, freeSockets]) => {
         nodes.add(node);
         const idleSockets = freeSockets?.length ?? 0;
         totalIdleSockets += idleSockets;
-        nodesWithIdleSockets[node] = (nodesWithIdleSockets[node] ?? 0) + idleSockets;
       });
     }
   });
 
-  const activeSocketCounters = Object.values(nodesWithActiveSockets);
-  const idleSocketCounters = Object.values(nodesWithIdleSockets);
-  const protocol: ElasticsearchClientProtocol = http
-    ? https
-      ? 'mixed'
-      : 'http'
-    : https
-    ? 'https'
-    : 'none';
-
   return {
-    protocol,
-    connectedNodes: nodes.size,
-    nodesWithActiveSockets: activeSocketCounters.filter(Boolean).length,
-    nodesWithIdleSockets: idleSocketCounters.filter(Boolean).length,
     totalActiveSockets,
     totalIdleSockets,
     totalQueuedRequests,
-    mostActiveNodeSockets: activeSocketCounters.length ? Math.max(...activeSocketCounters) : 0,
-    averageActiveSocketsPerNode: activeSocketCounters.length ? mean(activeSocketCounters) : 0,
-    mostIdleNodeSockets: idleSocketCounters.length ? Math.max(...idleSocketCounters) : 0,
-    averageIdleSocketsPerNode: idleSocketCounters.length ? mean(idleSocketCounters) : 0,
   };
 };
diff --git a/packages/core/metrics/core-metrics-server-mocks/src/metrics_service.mock.ts b/packages/core/metrics/core-metrics-server-mocks/src/metrics_service.mock.ts
@@ -24,17 +24,9 @@ import type {
 } from '@kbn/core-metrics-server';
 
 export const sampleEsClientMetrics: ElasticsearchClientsMetrics = {
-  protocol: 'https',
-  connectedNodes: 3,
-  nodesWithActiveSockets: 3,
-  nodesWithIdleSockets: 1,
   totalActiveSockets: 25,
   totalIdleSockets: 2,
   totalQueuedRequests: 0,
-  mostActiveNodeSockets: 15,
-  averageActiveSocketsPerNode: 8,
-  mostIdleNodeSockets: 2,
-  averageIdleSocketsPerNode: 0.5,
 };
 
 const createInternalSetupContractMock = () => {

diff --git a/packages/core/metrics/core-metrics-server/src/metrics.ts b/packages/core/metrics/core-metrics-server/src/metrics.ts
@@ -52,30 +52,12 @@ export type ElasticsearchClientProtocol = 'none' | 'http' | 'https' | 'mixed';
  * @public
  */
 export interface ElasticsearchClientsMetrics {
-  /** The protocol (or protocols) that these Agents are using */
-  protocol: ElasticsearchClientProtocol;
-  /** Number of ES nodes that ES-js client is connecting to */
-  connectedNodes: number;
-  /** Number of nodes with active connections */
-  nodesWithActiveSockets: number;
-  /** Number of nodes with available connections (alive but idle).
-   * Note that a node can have both active and idle connections at the same time
-   */
-  nodesWithIdleSockets: number;
   /** Total number of active sockets (all nodes, all connections) */
   totalActiveSockets: number;
   /** Total number of available sockets (alive but idle, all nodes, all connections) */
   totalIdleSockets: number;
   /** Total number of queued requests (all nodes, all connections) */
   totalQueuedRequests: number;
-  /** Number of active connections of the node with most active connections */
-  mostActiveNodeSockets: number;
-  /** Average of active sockets per node (all connections) */
-  averageActiveSocketsPerNode: number;
-  /** Number of idle connections of the node with most idle connections */
-  mostIdleNodeSockets: number;
-  /** Average of available (idle) sockets per node (all connections) */
-  averageIdleSocketsPerNode: number;
 }
 
 /**

diff --git a/src/core/server/integration_tests/saved_objects/migrations/check_registered_types.test.ts b/src/core/server/integration_tests/saved_objects/migrations/check_registered_types.test.ts
@@ -113,8 +113,8 @@ describe('checking migration metadata changes on all registered SO types', () =>
         "ml-trained-model": "e39dd10b2da827e194ddcaaf3db141ad1daf0201",
         "monitoring-telemetry": "af508cea8e22edaa909e462069390650fbbf01b7",
         "osquery-manager-usage-metric": "fbe3cbea25a96e2ca522ca436878e0162c94dcc2",
-        "osquery-pack": "afb3b46c5e23fc24ad438e9c4317ff37e4e5164a",
-        "osquery-pack-asset": "32421669c87c49dfabd4d3957f044e5eb7f7fb20",
+        "osquery-pack": "a2d675c7af4208e54a5b28d23d324d7c599a5491",
+        "osquery-pack-asset": "de8783298eb33a577bf1fa0caacd42121dcfae91",
         "osquery-saved-query": "7b213b4b7a3e59350e99c50e8df9948662ed493a",
         "query": "4640ef356321500a678869f24117b7091a911cb6",
         "sample-data-telemetry": "8b10336d9efae6f3d5593c4cc89fb4abcdf84e04",
@@ -130,7 +130,7 @@ describe('checking migration metadata changes on all registered SO types', () =>
         "siem-ui-timeline-pinned-event": "e2697b38751506c7fce6e8b7207a830483dc4283",
         "space": "c4a0acce1bd4b9cce85154f2a350624a53111c59",
         "spaces-usage-stats": "922d3235bbf519e3fb3b260e27248b1df8249b79",
-        "synthetics-monitor": "30f1cd04016a37095de60554cbf7fff89aaad177",
+        "synthetics-monitor": "111811218f7e34f40980665a4eb99976f457bb23",
         "synthetics-privates-locations": "dd00385f4a27ef062c3e57312eeb3799872fa4af",
         "tag": "39413f4578cc2128c9a0fda97d0acd1c8862c47a",
         "task": "ef53d0f070bd54957b8fe22fae3b1ff208913f76",

diff --git a/...age_collection/server/collectors/ops_stats/__snapshots__/ops_stats_collector.test.ts.snap b/...age_collection/server/collectors/ops_stats/__snapshots__/ops_stats_collector.test.ts.snap
diff --git a/x-pack/plugins/cloud_security_posture/common/constants.ts b/x-pack/plugins/cloud_security_posture/common/constants.ts
@@ -40,7 +40,6 @@ export const INTERNAL_FEATURE_FLAGS = {
   showManageRulesMock: false,
   showFindingFlyoutEvidence: false,
   showFindingsGroupBy: true,
-  showNewDashboard: false,
 } as const;
 
 export const CSP_RULE_SAVED_OBJECT_TYPE = 'csp_rule';

diff --git a/x-pack/plugins/cloud_security_posture/public/common/api/use_cis_kubernetes_integration.tsx b/x-pack/plugins/cloud_security_posture/public/common/api/use_cis_kubernetes_integration.tsx
@@ -21,8 +21,6 @@ export const useCisKubernetesIntegration = () => {
   const { http } = useKibana().services;
 
   return useQuery<GetInfoResponse, DefaultPackagesInstallationError>(['integrations'], () =>
-    http.get<GetInfoResponse>(epmRouteService.getInfoPath(CLOUD_SECURITY_POSTURE_PACKAGE_NAME), {
-      query: { experimental: true },
-    })
+    http.get<GetInfoResponse>(epmRouteService.getInfoPath(CLOUD_SECURITY_POSTURE_PACKAGE_NAME))
   );
 };
diff --git a/x-pack/plugins/cloud_security_posture/public/components/chart_panel.tsx b/x-pack/plugins/cloud_security_posture/public/components/chart_panel.tsx
@@ -60,7 +60,7 @@ export const ChartPanel: React.FC<ChartPanelProps> = ({
 
   return (
     <EuiPanel hasBorder={hasBorder} hasShadow={false} data-test-subj="chart-panel">
-      <EuiFlexGroup direction="column" gutterSize="none" style={{ height: '100%' }}>
+      <EuiFlexGroup direction="column" gutterSize="m" style={{ height: '100%' }}>
         <EuiFlexItem grow={false}>
           {title && (
             <EuiTitle size="xs">

diff --git a/x-pack/plugins/cloud_security_posture/public/components/cis_benchmark_icon.tsx b/x-pack/plugins/cloud_security_posture/public/components/cis_benchmark_icon.tsx
@@ -30,6 +30,6 @@ const getBenchmarkIdIconType = (props: Props): string => {
 
 export const CISBenchmarkIcon = (props: Props) => (
   <EuiToolTip content={props.name}>
-    <EuiIcon type={getBenchmarkIdIconType(props)} size="xxl" css={props.style} />
+    <EuiIcon type={getBenchmarkIdIconType(props)} size="xl" css={props.style} />
   </EuiToolTip>
 );