Merge branch 'main' into v3

percona · Jan 23, 2024 · 6653256 · 6653256
1 parent 9389120
commit 6653256
Show file tree

Hide file tree

Showing 70 changed files with 2,754 additions and 134 deletions.
diff --git a/.betterer.results b/.betterer.results
diff --git a/packages/grafana-data/src/types/icon.ts b/packages/grafana-data/src/types/icon.ts
@@ -10,6 +10,7 @@ export const availableIconsIndex = {
   'google-hangouts-alt': true,
   pagerduty: true,
   line: true,
+  brain: true,
   anchor: true,
   'adjust-circle': true,
   'angle-double-down': true,

diff --git a/public/app/core/components/Footer/Footer.tsx b/public/app/core/components/Footer/Footer.tsx
@@ -16,6 +16,13 @@ export interface FooterLink {
 export let getFooterLinks = (): FooterLink[] => {
   // @PERCONA
   return [
+    {
+      id: 'pmm-dump',
+      text: 'PMM Dump',
+      icon: 'brain',
+      url: '/graph/pmm-dump',
+      target: '_self',
+    },
     {
       id: 'pmm-logs',
       text: 'PMM Logs',

diff --git a/public/app/core/icons/cached.json b/public/app/core/icons/cached.json
@@ -27,6 +27,7 @@
   "unicons/book",
   "unicons/book-open",
   "unicons/brackets-curly",
+  "unicons/brain",
   "unicons/bug",
   "unicons/building",
   "unicons/calculator-alt",

diff --git a/public/app/percona/add-instance/components/AddRemoteInstance/AddRemoteInstance.types.ts b/public/app/percona/add-instance/components/AddRemoteInstance/AddRemoteInstance.types.ts
@@ -102,6 +102,7 @@ export interface PostgreSQLPayload extends RemoteCommonPayload, TLSCommon {
   tls_key: string;
   agent_password: string;
   max_query_length: number;
+  auto_discovery_limit: number;
 }
 
 export interface MySQLPayload extends RemoteCommonPayload, TLSCommon {

diff --git a/...e/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.constants.ts b/...e/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.constants.ts
@@ -1,5 +1,5 @@
 import { Messages } from './AdditionalOptions.messages';
-import { TablestatOptionsInterface } from './AdditionalOptions.types';
+import { AutoDiscoveryOptionsInterface, TablestatOptionsInterface } from './AdditionalOptions.types';
 
 export const tablestatOptions = [
   {
@@ -15,3 +15,18 @@ export const tablestatOptions = [
     label: Messages.form.tablestatOptions.custom,
   },
 ];
+
+export const autoDiscoveryOptions = [
+  {
+    value: AutoDiscoveryOptionsInterface.enabled,
+    label: Messages.form.autoDiscoveryOptions.enabled,
+  },
+  {
+    value: AutoDiscoveryOptionsInterface.disabled,
+    label: Messages.form.autoDiscoveryOptions.disabled,
+  },
+  {
+    value: AutoDiscoveryOptionsInterface.custom,
+    label: Messages.form.autoDiscoveryOptions.custom,
+  },
+];
diff --git a/...ce/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.messages.ts b/...ce/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.messages.ts
@@ -5,5 +5,10 @@ export const Messages = {
       default: 'Default',
       custom: 'Custom',
     },
+    autoDiscoveryOptions: {
+      enabled: 'Enabled',
+      disabled: 'Disabled',
+      custom: 'Custom',
+    },
   },
 };
diff --git a/...d-instance/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.tsx b/...d-instance/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.tsx
@@ -14,8 +14,8 @@ import { Messages } from '../FormParts.messages';
 import { getStyles } from '../FormParts.styles';
 import { AdditionalOptionsFormPartProps, PostgreSQLAdditionalOptionsProps } from '../FormParts.types';
 
-import { tablestatOptions } from './AdditionalOptions.constants';
-import { TablestatOptionsInterface } from './AdditionalOptions.types';
+import { autoDiscoveryOptions, tablestatOptions } from './AdditionalOptions.constants';
+import { AutoDiscoveryOptionsInterface, TablestatOptionsInterface } from './AdditionalOptions.types';
 import { MongodbTLSCertificate } from './MongodbTLSCertificate';
 import { MysqlTLSCertificate } from './MysqlTLSCertificate';
 import { PostgreTLSCertificate } from './PostgreTLSCertificate';
@@ -41,16 +41,55 @@ export const AdditionalOptionsFormPart: FC<AdditionalOptionsFormPartProps> = ({
   );
 };
 
-export const PostgreSQLAdditionalOptions: FC<PostgreSQLAdditionalOptionsProps> = ({ isRDS, isAzure }) => (
-  <>
-    <h4>{Messages.form.labels.trackingOptions}</h4>
-    <RadioButtonGroupField
-      name="tracking"
-      data-testid="tracking-options-radio-button-group"
-      options={isRDS || isAzure ? rdsTrackingOptions : trackingOptions}
-    />
-  </>
-);
+export const PostgreSQLAdditionalOptions: FC<PostgreSQLAdditionalOptionsProps> = ({ form, isRDS, isAzure }) => {
+  const selectedOption = form.getState()?.values?.autoDiscoveryOptions;
+  const [selectedValue, setSelectedValue] = useState<string>(selectedOption || AutoDiscoveryOptionsInterface.enabled);
+  const styles = useStyles2(getStyles);
+  const validators = [platformCoreValidators.containsNumber, ...platformCoreValidators.int32];
+
+  const getAutoDiscoveryLimitValue = (type: AutoDiscoveryOptionsInterface) =>
+    type === AutoDiscoveryOptionsInterface.disabled ? -1 : 10;
+
+  useEffect(() => {
+    setSelectedValue(selectedOption);
+    form.change('autoDiscoveryLimit', getAutoDiscoveryLimitValue(selectedOption));
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [selectedOption]);
+
+  return (
+    <>
+      <h4>{Messages.form.labels.trackingOptions}</h4>
+      <RadioButtonGroupField
+        name="tracking"
+        data-testid="tracking-options-radio-button-group"
+        options={isRDS || isAzure ? rdsTrackingOptions : trackingOptions}
+        className={styles.radioField}
+        fullWidth
+      />
+      <h4>{Messages.form.labels.postgresqlDetails.autoDiscovery}</h4>
+      <div className={styles.group}>
+        <RadioButtonGroupField
+          name="autoDiscoveryOptions"
+          data-testid="auto-discovery-options-radio-button-group"
+          defaultValue={selectedValue}
+          options={autoDiscoveryOptions}
+          className={styles.radioField}
+          label={Messages.form.labels.postgresqlDetails.autoDiscoveryLimitOptions}
+          fullWidth
+        />
+        <NumberInputField
+          key="autoDiscoveryLimit"
+          name="autoDiscoveryLimit"
+          defaultValue={0}
+          disabled={selectedValue !== AutoDiscoveryOptionsInterface.custom}
+          validators={validators}
+          label={Messages.form.labels.postgresqlDetails.autoDiscoveryLimit}
+          tooltipText={Messages.form.tooltips.postgresqlDetails.autoDiscoveryLimit}
+        />
+      </div>
+    </>
+  );
+};
 
 const getTablestatValues = (type: TablestatOptionsInterface) => {
   switch (type) {
@@ -69,7 +108,8 @@ const MySQLOptions = ({ form }: { form: FormApi }) => {
   useEffect(() => {
     setSelectedValue(selectedOption);
     form.change('tablestats_group_table_limit', getTablestatValues(selectedOption));
-  }, [selectedOption, form]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [selectedOption]);
 
   return (
     <>
@@ -115,6 +155,7 @@ export const getAdditionalOptions = (
             />
           </>
           <PostgreSQLAdditionalOptions
+            form={form}
             isRDS={remoteInstanceCredentials.isRDS}
             isAzure={remoteInstanceCredentials.isAzure}
           />

diff --git a/...tance/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.types.ts b/...tance/components/AddRemoteInstance/FormParts/AdditionalOptions/AdditionalOptions.types.ts
@@ -3,3 +3,9 @@ export enum TablestatOptionsInterface {
   default = 'default',
   custom = 'custom',
 }
+
+export enum AutoDiscoveryOptionsInterface {
+  enabled = 'enabled',
+  disabled = 'disabled',
+  custom = 'custom',
+}
diff --git a/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.messages.ts b/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.messages.ts
@@ -29,6 +29,9 @@ export const Messages = {
       postgresqlDetails: {
         database: 'Database',
         maxQueryLength: 'Max query length',
+        autoDiscovery: 'Auto-discovery',
+        autoDiscoveryLimit: 'Auto-discovery limit',
+        autoDiscoveryLimitOptions: 'State',
       },
       mysqlDetails: {
         maxQueryLength: 'Max query length',
@@ -87,6 +90,7 @@ export const Messages = {
       postgresqlDetails: {
         database: 'Database (default: postgres)',
         maxQueryLength: 'Max query length',
+        autoDiscoveryLimit: 'Auto-discovery limit',
       },
       mysqlDetails: {
         maxQueryLength: 'Max query length',
@@ -126,6 +130,7 @@ export const Messages = {
       postgresqlDetails: {
         database: 'Database name',
         maxQueryLength: 'Full Example (Fingerprint) storage is not allowed by default to have the best performance',
+        autoDiscoveryLimit: 'Turn off auto-discovery when the total count of databases exceeds the limit.',
       },
       mysqlDetails: {
         maxQueryLength: 'Full Example (Fingerprint) storage is not allowed by default to have the best performance',

diff --git a/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.test.tsx b/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.test.tsx
@@ -181,7 +181,7 @@ describe('getAdditionalOptions ::', () => {
     const fields = container.querySelectorAll('input');
     const trakingFields = screen.getAllByTestId('tracking-radio-button');
     expect(trakingFields.length).toBe(trackingOptions.length);
-    expect(fields.length).toBe(7);
+    expect(fields.length).toBe(12);
   });
   it('should render correct for RDS PostgreSQL', async () => {
     const type = Databases.postgresql;
@@ -198,6 +198,6 @@ describe('getAdditionalOptions ::', () => {
     const fields = container.querySelectorAll('input');
     const trakingFields = screen.getAllByTestId('tracking-radio-button');
     expect(trakingFields.length).toBe(rdsTrackingOptions.length);
-    expect(fields.length).toBe(8);
+    expect(fields.length).toBe(13);
   });
 });
diff --git a/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.types.ts b/public/app/percona/add-instance/components/AddRemoteInstance/FormParts/FormParts.types.ts
@@ -21,6 +21,7 @@ export interface AdditionalOptionsFormPartProps {
 export interface PostgreSQLAdditionalOptionsProps {
   isRDS?: boolean;
   isAzure?: boolean;
+  form: FormApi;
 }
 
 export enum Schema {

diff --git a/public/app/percona/backup/Backup.messages.ts b/public/app/percona/backup/Backup.messages.ts
@@ -14,7 +14,20 @@ export const Messages = {
         created: 'Created',
         location: 'Location',
         vendor: 'DB Technology',
-        status: 'Status',
+        status: {
+          name: 'Status',
+          options: {
+            success: 'Success',
+            error: 'Error',
+            pending: 'Pending',
+            paused: 'Paused',
+            invalid: 'Invalid',
+            inProgress: 'In Progress',
+            failedToDelete: 'Failed To Delete',
+            failedNotSupportedByAgent: 'Failed Not Supported By Agent',
+            deleting: 'Deleting',
+          },
+        },
         actions: 'Actions',
         type: 'Type',
       },

diff --git a/public/app/percona/backup/Backup.utils.ts b/public/app/percona/backup/Backup.utils.ts
@@ -34,15 +34,15 @@ export const formatDataModel = (model: DataModel): string => {
   return map[model] ?? '';
 };
 
-export const formatBackupMode = (mode: BackupMode): string => {
-  const map: Record<BackupMode, string> = {
-    [BackupMode.SNAPSHOT]: backupModeMsg.full,
-    [BackupMode.INCREMENTAL]: backupModeMsg.incremental,
-    [BackupMode.PITR]: backupModeMsg.pitr,
-    [BackupMode.INVALID]: backupModeMsg.invalid,
-  };
+export const BackupModeMap: Record<BackupMode, string> = {
+  [BackupMode.SNAPSHOT]: backupModeMsg.full,
+  [BackupMode.INCREMENTAL]: backupModeMsg.incremental,
+  [BackupMode.PITR]: backupModeMsg.pitr,
+  [BackupMode.INVALID]: backupModeMsg.invalid,
+};
 
-  return map[mode] || map[BackupMode.INVALID];
+export const formatBackupMode = (mode: BackupMode): string => {
+  return BackupModeMap[mode] || BackupModeMap[BackupMode.INVALID];
 };
 
 export const formatLocationsToMap = (locations: StorageLocation[]) =>

diff --git a/public/app/percona/backup/components/AddBackupPage/AddBackupPage.service.ts b/public/app/percona/backup/components/AddBackupPage/AddBackupPage.service.ts
@@ -18,7 +18,7 @@ export const AddBackupPageService = {
       if (supportedServices.includes(serviceName)) {
         result.push(
           ...newServices
-            .filter((service) => service.name.toLowerCase().includes(query))
+            .filter((service) => service.name.toLowerCase().includes(query.toLowerCase()))
             .map(
               ({ id, name, cluster }): SelectableValue<SelectableService> => ({
                 label: name,

diff --git a/public/app/percona/backup/components/BackupInventory/BackupInventory.test.tsx b/public/app/percona/backup/components/BackupInventory/BackupInventory.test.tsx
@@ -11,6 +11,12 @@ import { BackupInventory } from './BackupInventory';
 jest.mock('./BackupInventory.service');
 jest.mock('app/percona/backup/components/StorageLocations/StorageLocations.service');
 jest.mock('../../hooks/recurringCall.hook');
+jest.mock('react-router-dom', () => ({
+  ...jest.requireActual('react-router-dom'),
+  useLocation: () => ({
+    pathname: '/',
+  }),
+}));
 
 describe('BackupInventory', () => {
   it('should send correct data to Table', async () => {