Merge branch 'main' into serverless/support-link-add-deployment-id

elastic · Nov 22, 2023 · 2101dbf · 2101dbf
2 parents b9c1312 + 5cbc93c
commit 2101dbf
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 49 deletions.
diff --git a/.buildkite/pull_requests.json b/.buildkite/pull_requests.json
@@ -55,14 +55,14 @@
       "repoName": "kibana",
       "pipelineSlug": "kibana-kme-test",
 
-      "enabled": false,
+      "enabled": true,
       "allow_org_users": true,
       "allowed_repo_permissions": ["admin", "write"],
       "allowed_list": ["barlowm", "renovate[bot]"],
       "set_commit_status": true,
-      "commit_status_context": "kibana-ci",
+      "commit_status_context": "kibana-ci-test",
       "build_on_commit": true,
-      "build_on_comment": true,
+      "build_on_comment": false,
       "trigger_comment_regex": "^(?:(?:buildkite\\W+)?(?:build|test)\\W+(?:this|it))",
       "always_trigger_comment_regex": "^(?:(?:buildkite\\W+)?(?:build|test)\\W+(?:this|it))",
       "skip_ci_labels": [],

diff --git a/.buildkite/scripts/steps/storybooks/build_and_upload.ts b/.buildkite/scripts/steps/storybooks/build_and_upload.ts
@@ -16,7 +16,6 @@ const STORYBOOKS = [
   'canvas',
   'cases',
   'cell_actions',
-  'ci_composite',
   'cloud_chat',
   'coloring',
   'chart_icons',
@@ -93,14 +92,12 @@ const upload = () => {
     console.log('--- Generating Storybooks HTML');
 
     process.chdir(path.join('.', 'built_assets', 'storybook'));
-    fs.renameSync('ci_composite', 'composite');
 
     const storybooks = execSync(`ls -1d */`)
       .toString()
       .trim()
       .split('\n')
-      .map((filePath) => filePath.replace('/', ''))
-      .filter((filePath) => filePath !== 'composite');
+      .map((filePath) => filePath.replace('/', ''));
 
     const listHtml = storybooks
       .map((storybook) => `<li><a href="${STORYBOOK_BASE_URL}/${storybook}">${storybook}</a></li>`)
@@ -110,8 +107,6 @@ const upload = () => {
       <html>
         <body>
           <h1>Storybooks</h1>
-          <p><a href="${STORYBOOK_BASE_URL}/composite">Composite Storybook</a></p>
-          <h2>All</h2>
           <ul>
             ${listHtml}
           </ul>

diff --git a/.ci/.storybook/main.js b/.ci/.storybook/main.js
diff --git a/renovate.json b/renovate.json
@@ -462,7 +462,8 @@
         "Team:Operations",
         "release_note:skip"
       ],
-      "enabled": true
+      "enabled": true,
+      "allowedVersions": "<7.0"
     },
     {
       "groupName": "react-query",
@@ -647,4 +648,4 @@
       "enabled": true
     }
   ]
-}
+}
diff --git a/src/dev/storybook/aliases.ts b/src/dev/storybook/aliases.ts
@@ -15,7 +15,6 @@ export const storybookAliases = {
   canvas: 'x-pack/plugins/canvas/storybook',
   cases: 'packages/kbn-cases-components/.storybook',
   cell_actions: 'packages/kbn-cell-actions/.storybook',
-  ci_composite: '.ci/.storybook',
   cloud_chat: 'x-pack/plugins/cloud_integrations/cloud_chat/.storybook',
   coloring: 'packages/kbn-coloring/.storybook',
   language_documentation_popover: 'packages/kbn-language-documentation-popover/.storybook',

diff --git a/x-pack/packages/security-solution/data_table/store/data_table/reducer.ts b/x-pack/packages/security-solution/data_table/store/data_table/reducer.ts
@@ -95,7 +95,7 @@ export const dataTableReducer = reducerWithInitialState(initialDataTableState)
         [action.id]: {
           ...state.tableById[action.id],
           expandedDetail: {
-            ...state.tableById[action.id].expandedDetail,
+            ...state.tableById[action.id]?.expandedDetail,
             ...updateTableDetailsPanel(action),
           },
         },

diff --git a/x-pack/plugins/security_solution/scripts/junit_transformer/lib.ts b/x-pack/plugins/security_solution/scripts/junit_transformer/lib.ts
@@ -16,6 +16,7 @@ import * as t from 'io-ts';
 import { isLeft } from 'fp-ts/lib/Either';
 import { PathReporter } from 'io-ts/lib/PathReporter';
 import globby from 'globby';
+import del from 'del';
 
 /**
  * Updates the `name` and `classname` attributes of each testcase.
@@ -195,6 +196,9 @@ ${boilerplate}
 
     if ('error' in maybeValidationResult) {
       logError(maybeValidationResult.error);
+      // Sending broken XML to Failed Test Reporter will cause a job to fail
+      await del(path);
+      log.warning(`${path} file was deleted.`);
       // If there is an error, continue trying to process other files.
       continue;
     }
@@ -203,9 +207,11 @@ ${boilerplate}
 
     const { processed, hadTestCases } = isReportAlreadyProcessed(reportJson);
     if (hadTestCases === false) {
-      log.warning(`${path} had no test cases. Skipping it.
+      log.warning(`${path} had no test cases.
 ${boilerplate}
 `);
+      await del(path);
+      log.warning(`${path} file was deleted.`);
       // If there is an error, continue trying to process other files.
       continue;
     }
@@ -222,6 +228,9 @@ ${boilerplate}
 
     if ('error' in maybeSpecFilePath) {
       logError(maybeSpecFilePath.error);
+      // Sending broken XML to Failed Test Reporter will cause a job to fail
+      await del(path);
+      log.warning(`${path} file was deleted.`);
       // If there is an error, continue trying to process other files.
       continue;
     }

diff --git a/x-pack/plugins/serverless_search/public/application/components/languages/dotnet.ts b/x-pack/plugins/serverless_search/public/application/components/languages/dotnet.ts
@@ -23,7 +23,7 @@ export const dotnetDefinition: LanguageDefinition = {
   installClient: 'dotnet add package Elastic.Clients.Elasticsearch.Serverless',
   configureClient: ({ apiKey, cloudId }) => `using System;
 using Elastic.Clients.Elasticsearch.Serverless;
-using Elastic.Clients.Elasticsearch.QueryDsl;
+using Elastic.Clients.Elasticsearch.Serverless.QueryDsl;
 
 var client = new ElasticsearchClient("${cloudId}", new ApiKey("${apiKey}"));`,
   testConnection: `var info = await client.InfoAsync();`,