Merge branch '7.x' into backport/7.x/pr-101068

elastic · Jun 2, 2021 · b02a01e · b02a01e
2 parents b305da5 + 654e46c
commit b02a01e
Show file tree

Hide file tree

Showing 99 changed files with 2,247 additions and 1,760 deletions.
diff --git a/docs/CHANGELOG.asciidoc b/docs/CHANGELOG.asciidoc
@@ -57,8 +57,6 @@ Review important information about the {kib} 7.x releases.
 [[release-notes-7.13.1]]
 == {kib} 7.13.1
 
-coming::[7.13.1]
-
 The 7.13.1 release includes the following bug fixes.
 
 [float]

diff --git a/docs/developer/getting-started/index.asciidoc b/docs/developer/getting-started/index.asciidoc
@@ -92,6 +92,10 @@ may need to run:
 yarn kbn clean
 ----
 
+NOTE: Running this command is only necessary in rare circumstance where you need to recover
+a consistent state when problems arise. If you need to run this command often, complete
+this form to provide feedback: https://ela.st/yarn-kbn-clean
+
 If you have failures during `yarn kbn bootstrap` you may have some
 corrupted packages in your yarn cache which you can clean with:
 

diff --git a/.../core/server/kibana-plugin-core-server.deprecationsdetails.correctiveactions.md b/.../core/server/kibana-plugin-core-server.deprecationsdetails.correctiveactions.md
@@ -15,6 +15,6 @@ correctiveActions: {
                 [key: string]: any;
             };
         };
-        manualSteps?: string[];
+        manualSteps: string[];
     };
 ```
diff --git a/docs/development/core/server/kibana-plugin-core-server.deprecationsdetails.md b/docs/development/core/server/kibana-plugin-core-server.deprecationsdetails.md
@@ -14,7 +14,7 @@ export interface DeprecationsDetails
 
 |  Property | Type | Description |
 |  --- | --- | --- |
-|  [correctiveActions](./kibana-plugin-core-server.deprecationsdetails.correctiveactions.md) | <code>{</code><br/><code>        api?: {</code><br/><code>            path: string;</code><br/><code>            method: 'POST' &#124; 'PUT';</code><br/><code>            body?: {</code><br/><code>                [key: string]: any;</code><br/><code>            };</code><br/><code>        };</code><br/><code>        manualSteps?: string[];</code><br/><code>    }</code> |  |
+|  [correctiveActions](./kibana-plugin-core-server.deprecationsdetails.correctiveactions.md) | <code>{</code><br/><code>        api?: {</code><br/><code>            path: string;</code><br/><code>            method: 'POST' &#124; 'PUT';</code><br/><code>            body?: {</code><br/><code>                [key: string]: any;</code><br/><code>            };</code><br/><code>        };</code><br/><code>        manualSteps: string[];</code><br/><code>    }</code> |  |
 |  [deprecationType](./kibana-plugin-core-server.deprecationsdetails.deprecationtype.md) | <code>'config' &#124; 'feature'</code> | (optional) Used to identify between different deprecation types. Example use case: in Upgrade Assistant, we may want to allow the user to sort by deprecation type or show each type in a separate tab.<!-- -->Feel free to add new types if necessary. Predefined types are necessary to reduce having similar definitions with different keywords across kibana deprecations. |
 |  [documentationUrl](./kibana-plugin-core-server.deprecationsdetails.documentationurl.md) | <code>string</code> |  |
 |  [level](./kibana-plugin-core-server.deprecationsdetails.level.md) | <code>'warning' &#124; 'critical' &#124; 'fetch_error'</code> | levels: - warning: will not break deployment upon upgrade - critical: needs to be addressed before upgrade. - fetch\_error: Deprecations service failed to grab the deprecation details for the domain. |

diff --git a/...ic/kibana-plugin-plugins-data-public.indexpattern.getaggregationrestrictions.md b/...ic/kibana-plugin-plugins-data-public.indexpattern.getaggregationrestrictions.md
@@ -8,26 +8,30 @@
 
 ```typescript
 getAggregationRestrictions(): Record<string, Record<string, {
-        agg?: string | undefined;
+        agg?: string | undefined; /**
+         * Get last saved saved object fields
+         */
         interval?: number | undefined;
         fixed_interval?: string | undefined;
         calendar_interval?: string | undefined;
-        delay?: string | undefined; /**
+        delay?: string | undefined;
+        time_zone?: string | undefined; /**
          * Reset last saved saved object fields. used after saving
          */
-        time_zone?: string | undefined;
     }>> | undefined;
 ```
 <b>Returns:</b>
 
 `Record<string, Record<string, {
-        agg?: string | undefined;
+        agg?: string | undefined; /**
+         * Get last saved saved object fields
+         */
         interval?: number | undefined;
         fixed_interval?: string | undefined;
         calendar_interval?: string | undefined;
-        delay?: string | undefined; /**
+        delay?: string | undefined;
+        time_zone?: string | undefined; /**
          * Reset last saved saved object fields. used after saving
          */
-        time_zone?: string | undefined;
     }>> | undefined`
 
diff --git a/...a/public/kibana-plugin-plugins-data-public.indexpattern.getassavedobjectbody.md b/...a/public/kibana-plugin-plugins-data-public.indexpattern.getassavedobjectbody.md
@@ -9,33 +9,9 @@ Returns index pattern as saved object body for saving
 <b>Signature:</b>
 
 ```typescript
-getAsSavedObjectBody(): {
-        fieldAttrs: string | undefined;
-        title: string;
-        timeFieldName: string | undefined;
-        intervalName: string | undefined;
-        sourceFilters: string | undefined;
-        fields: string | undefined;
-        fieldFormatMap: string | undefined;
-        type: string | undefined;
-        typeMeta: string | undefined;
-        allowNoIndex: true | undefined;
-        runtimeFieldMap: string | undefined;
-    };
+getAsSavedObjectBody(): IndexPatternAttributes;
 ```
 <b>Returns:</b>
 
-`{
-        fieldAttrs: string | undefined;
-        title: string;
-        timeFieldName: string | undefined;
-        intervalName: string | undefined;
-        sourceFilters: string | undefined;
-        fields: string | undefined;
-        fieldFormatMap: string | undefined;
-        type: string | undefined;
-        typeMeta: string | undefined;
-        allowNoIndex: true | undefined;
-        runtimeFieldMap: string | undefined;
-    }`
+`IndexPatternAttributes`
 
diff --git a/...er/kibana-plugin-plugins-data-server.indexpattern.getaggregationrestrictions.md b/...er/kibana-plugin-plugins-data-server.indexpattern.getaggregationrestrictions.md
@@ -8,26 +8,30 @@
 
 ```typescript
 getAggregationRestrictions(): Record<string, Record<string, {
-        agg?: string | undefined;
+        agg?: string | undefined; /**
+         * Get last saved saved object fields
+         */
         interval?: number | undefined;
         fixed_interval?: string | undefined;
         calendar_interval?: string | undefined;
-        delay?: string | undefined; /**
+        delay?: string | undefined;
+        time_zone?: string | undefined; /**
          * Reset last saved saved object fields. used after saving
          */
-        time_zone?: string | undefined;
     }>> | undefined;
 ```
 <b>Returns:</b>
 
 `Record<string, Record<string, {
-        agg?: string | undefined;
+        agg?: string | undefined; /**
+         * Get last saved saved object fields
+         */
         interval?: number | undefined;
         fixed_interval?: string | undefined;
         calendar_interval?: string | undefined;
-        delay?: string | undefined; /**
+        delay?: string | undefined;
+        time_zone?: string | undefined; /**
          * Reset last saved saved object fields. used after saving
          */
-        time_zone?: string | undefined;
     }>> | undefined`
 
diff --git a/...a/server/kibana-plugin-plugins-data-server.indexpattern.getassavedobjectbody.md b/...a/server/kibana-plugin-plugins-data-server.indexpattern.getassavedobjectbody.md
@@ -9,33 +9,9 @@ Returns index pattern as saved object body for saving
 <b>Signature:</b>
 
 ```typescript
-getAsSavedObjectBody(): {
-        fieldAttrs: string | undefined;
-        title: string;
-        timeFieldName: string | undefined;
-        intervalName: string | undefined;
-        sourceFilters: string | undefined;
-        fields: string | undefined;
-        fieldFormatMap: string | undefined;
-        type: string | undefined;
-        typeMeta: string | undefined;
-        allowNoIndex: true | undefined;
-        runtimeFieldMap: string | undefined;
-    };
+getAsSavedObjectBody(): IndexPatternAttributes;
 ```
 <b>Returns:</b>
 
-`{
-        fieldAttrs: string | undefined;
-        title: string;
-        timeFieldName: string | undefined;
-        intervalName: string | undefined;
-        sourceFilters: string | undefined;
-        fields: string | undefined;
-        fieldFormatMap: string | undefined;
-        type: string | undefined;
-        typeMeta: string | undefined;
-        allowNoIndex: true | undefined;
-        runtimeFieldMap: string | undefined;
-    }`
+`IndexPatternAttributes`
 
diff --git a/packages/kbn-config/src/config_service.test.ts b/packages/kbn-config/src/config_service.test.ts
@@ -418,8 +418,14 @@ test('logs deprecation warning during validation', async () => {
   const configService = new ConfigService(rawConfig, defaultEnv, logger);
   mockApplyDeprecations.mockImplementationOnce((config, deprecations, createAddDeprecation) => {
     const addDeprecation = createAddDeprecation!('');
-    addDeprecation({ message: 'some deprecation message' });
-    addDeprecation({ message: 'another deprecation message' });
+    addDeprecation({
+      message: 'some deprecation message',
+      correctiveActions: { manualSteps: ['do X'] },
+    });
+    addDeprecation({
+      message: 'another deprecation message',
+      correctiveActions: { manualSteps: ['do Y'] },
+    });
     return { config, changedPaths: mockedChangedPaths };
   });
 
@@ -444,13 +450,24 @@ test('does not log warnings for silent deprecations during validation', async ()
   mockApplyDeprecations
     .mockImplementationOnce((config, deprecations, createAddDeprecation) => {
       const addDeprecation = createAddDeprecation!('');
-      addDeprecation({ message: 'some deprecation message', silent: true });
-      addDeprecation({ message: 'another deprecation message' });
+      addDeprecation({
+        message: 'some deprecation message',
+        correctiveActions: { manualSteps: ['do X'] },
+        silent: true,
+      });
+      addDeprecation({
+        message: 'another deprecation message',
+        correctiveActions: { manualSteps: ['do Y'] },
+      });
       return { config, changedPaths: mockedChangedPaths };
     })
     .mockImplementationOnce((config, deprecations, createAddDeprecation) => {
       const addDeprecation = createAddDeprecation!('');
-      addDeprecation({ message: 'I am silent', silent: true });
+      addDeprecation({
+        message: 'I am silent',
+        silent: true,
+        correctiveActions: { manualSteps: ['do Z'] },
+      });
       return { config, changedPaths: mockedChangedPaths };
     });
 
@@ -519,7 +536,11 @@ describe('getHandledDeprecatedConfigs', () => {
     mockApplyDeprecations.mockImplementationOnce((config, deprecations, createAddDeprecation) => {
       deprecations.forEach((deprecation) => {
         const addDeprecation = createAddDeprecation!(deprecation.path);
-        addDeprecation({ message: `some deprecation message`, documentationUrl: 'some-url' });
+        addDeprecation({
+          message: `some deprecation message`,
+          documentationUrl: 'some-url',
+          correctiveActions: { manualSteps: ['do X'] },
+        });
       });
       return { config, changedPaths: mockedChangedPaths };
     });
@@ -532,6 +553,11 @@ describe('getHandledDeprecatedConfigs', () => {
           "base",
           Array [
             Object {
+              "correctiveActions": Object {
+                "manualSteps": Array [
+                  "do X",
+                ],
+              },
               "documentationUrl": "some-url",
               "message": "some deprecation message",
             },

diff --git a/packages/kbn-config/src/deprecation/types.ts b/packages/kbn-config/src/deprecation/types.ts
@@ -25,8 +25,8 @@ export interface DeprecatedConfigDetails {
   silent?: boolean;
   /* (optional) link to the documentation for more details on the deprecation. */
   documentationUrl?: string;
-  /* (optional) corrective action needed to fix this deprecation. */
-  correctiveActions?: {
+  /* corrective action needed to fix this deprecation. */
+  correctiveActions: {
     /**
      * Specify a list of manual steps our users need to follow
      * to fix the deprecation before upgrade.

diff --git a/src/core/public/deprecations/deprecations_client.test.ts b/src/core/public/deprecations/deprecations_client.test.ts
@@ -90,6 +90,7 @@ describe('DeprecationsClient', () => {
             path: 'some-path',
             method: 'POST',
           },
+          manualSteps: ['manual-step'],
         },
       };
 
@@ -104,7 +105,9 @@ describe('DeprecationsClient', () => {
         domainId: 'testPluginId-1',
         message: 'some-message',
         level: 'warning',
-        correctiveActions: {},
+        correctiveActions: {
+          manualSteps: ['manual-step'],
+        },
       };
 
       const isResolvable = deprecationsClient.isDeprecationResolvable(mockDeprecationDetails);
@@ -120,7 +123,9 @@ describe('DeprecationsClient', () => {
         domainId: 'testPluginId-1',
         message: 'some-message',
         level: 'warning',
-        correctiveActions: {},
+        correctiveActions: {
+          manualSteps: ['manual-step'],
+        },
       };
       const result = await deprecationsClient.resolveDeprecation(mockDeprecationDetails);
 
@@ -144,6 +149,7 @@ describe('DeprecationsClient', () => {
               extra_param: 123,
             },
           },
+          manualSteps: ['manual-step'],
         },
       };
       const result = await deprecationsClient.resolveDeprecation(mockDeprecationDetails);
@@ -176,6 +182,7 @@ describe('DeprecationsClient', () => {
               extra_param: 123,
             },
           },
+          manualSteps: ['manual-step'],
         },
       };
       http.fetch.mockRejectedValue({ body: { message: mockResponse } });

diff --git a/src/core/server/config/deprecation/core_deprecations.test.ts b/src/core/server/config/deprecation/core_deprecations.test.ts
@@ -24,7 +24,7 @@ describe('core deprecations', () => {
       const { messages } = applyCoreDeprecations();
       expect(messages).toMatchInlineSnapshot(`
         Array [
-          "Environment variable CONFIG_PATH is deprecated. It has been replaced with KBN_PATH_CONF pointing to a config folder",
+          "Environment variable \\"CONFIG_PATH\\" is deprecated. It has been replaced with \\"KBN_PATH_CONF\\" pointing to a config folder",
         ]
       `);
     });
@@ -425,7 +425,7 @@ describe('core deprecations', () => {
       });
       expect(messages).toMatchInlineSnapshot(`
         Array [
-          "\\"logging.events.log\\" has been deprecated and will be removed in 8.0. Moving forward, log levels can be customized on a per-logger basis using the new logging configuration. ",
+          "\\"logging.events.log\\" has been deprecated and will be removed in 8.0. Moving forward, log levels can be customized on a per-logger basis using the new logging configuration.",
         ]
       `);
     });
@@ -438,7 +438,7 @@ describe('core deprecations', () => {
       });
       expect(messages).toMatchInlineSnapshot(`
         Array [
-          "\\"logging.events.error\\" has been deprecated and will be removed in 8.0. Moving forward, you can use \\"logging.root.level: error\\" in your logging configuration. ",
+          "\\"logging.events.error\\" has been deprecated and will be removed in 8.0. Moving forward, you can use \\"logging.root.level: error\\" in your logging configuration.",
         ]
       `);
     });