PMM-12284 Remove alertmanager from settings (#697)

* PMM-12284 Remove alertmanager from settings * PMM-12284 Trigger PR checks * PMM-12284 Fix CI --------- Co-authored-by: Artem Gavrilov <[email protected]>
percona · Dec 5, 2023 · 533d1f1 · 533d1f1
1 parent 4a59c23
commit 533d1f1
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 271 deletions.
diff --git a/.github/workflows/ui-tests.yml b/.github/workflows/ui-tests.yml
@@ -74,7 +74,7 @@ jobs:
 
       - name: Attaching artifacts
         if: ${{ always() }}
-        uses: percona-platform/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         with:
           name: ui-tests-output
           path: ./pmm-ui-tests/tests/output
diff --git a/public/app/percona/settings/Settings.messages.ts b/public/app/percona/settings/Settings.messages.ts
@@ -48,18 +48,6 @@ export const Messages = {
       '                  about feature status',
     technicalPreviewLinkText: 'here',
   },
-  alertmanager: {
-    warningPre: "Note: integration with Alertmanager is needed only in cases when you can't use",
-    warningLinkContent: 'Percona Alerting',
-    warningPost: 'but you need to manage and configure alerts.',
-    action: 'Apply Alertmanager settings',
-    rulesLabel: 'Prometheus Alerting rules',
-    rulesLink: `https://per.co.na/alertmanager`,
-    rulesTooltip: 'Alerting rules in the YAML configuration format.',
-    urlLabel: 'Alertmanager URL',
-    urlLink: `https://per.co.na/alertmanager`,
-    urlTooltip: 'The URL of the external Alertmanager to use.',
-  },
   diagnostics: {
     action: 'Download server diagnostics',
     label: 'Diagnostics',
@@ -96,7 +84,6 @@ export const Messages = {
     metrics: 'Metrics Resolution',
     advanced: 'Advanced Settings',
     ssh: 'SSH Key',
-    alertManager: 'Alertmanager Integration',
     perconaPlatform: 'Percona Platform',
     communication: 'Communication',
   },

diff --git a/public/app/percona/settings/components/AlertManager/AlertManager.styles.ts b/public/app/percona/settings/components/AlertManager/AlertManager.styles.ts
diff --git a/public/app/percona/settings/components/AlertManager/AlertManager.test.tsx b/public/app/percona/settings/components/AlertManager/AlertManager.test.tsx
diff --git a/public/app/percona/settings/components/AlertManager/AlertManager.tsx b/public/app/percona/settings/components/AlertManager/AlertManager.tsx
diff --git a/public/app/percona/settings/components/index.ts b/public/app/percona/settings/components/index.ts
@@ -1,5 +1,4 @@
 export { Advanced } from './Advanced/Advanced';
-export { AlertManager } from './AlertManager/AlertManager';
 export { Diagnostics } from './Diagnostics/Diagnostics';
 export { MetricsResolution } from './MetricsResolution/MetricsResolution';
 export { SSHKey } from './SSHKey/SSHKey';

diff --git a/...na/shared/components/PerconaBootstrapper/PerconaNavigation/PerconaNavigation.constants.ts b/...na/shared/components/PerconaBootstrapper/PerconaNavigation/PerconaNavigation.constants.ts
@@ -179,11 +179,6 @@ export const getPmmSettingsPage = (alertingEnabled = false): NavModelItem => {
       text: 'SSH Key',
       url: `${config.appSubUrl}/settings/ssh-key`,
     },
-    {
-      id: 'settings-alert-manager',
-      text: 'Alertmanager Integration',
-      url: `${config.appSubUrl}/settings/am-integration`,
-    },
     {
       id: 'settings-percona-platform',
       text: 'Percona Platform',

diff --git a/public/app/routes/routes.tsx b/public/app/routes/routes.tsx
@@ -694,15 +694,6 @@ export function getAppRoutes(): RouteDescriptor[] {
         () => import(/* webpackChunkName: "SSHKeySettingsPage" */ 'app/percona/settings/components/SSHKey/SSHKey')
       ),
     },
-    {
-      path: '/settings/am-integration',
-      component: SafeDynamicImport(
-        () =>
-          import(
-            /* webpackChunkName: "AMIntegrationSettingsPage" */ 'app/percona/settings/components/AlertManager/AlertManager'
-          )
-      ),
-    },
     {
       path: '/settings/percona-platform',
       component: SafeDynamicImport(