Merge pull request #637 from PeterYurkovich/ou-1058

[release-4.17] OU-1058: Don't show TP button in non-administrator

Author: openshift-merge-bot[bot]

web/src/components/alerting/AlertsDetailPage.tsx

@@ -17,6 +17,7 @@ import {
   Silence,
   TableColumn,
   VirtualizedTable,
+  useActivePerspective,
 } from '@openshift-console/dynamic-plugin-sdk';
 import {
   AlertingRuleChartExtension,
@@ -81,6 +82,7 @@ const AlertsDetailsPage_: React.FC<AlertsDetailsPageProps> = ({ history, match }
   const { t } = useTranslation('plugin__monitoring-plugin');
 
   const { alertsKey, isDev } = usePerspective();
+  const [perspective] = useActivePerspective();
 
   const namespace = match.params?.ns;
   const hideGraphs = useSelector(({ observe }: RootState) => !!observe.get('hideGraphs'));
@@ -134,6 +136,14 @@ const AlertsDetailsPage_: React.FC<AlertsDetailsPageProps> = ({ history, match }
     [customExtensions, isDev],
   );
 
+  // Keep the default name for the admin perspective while providing specific
+  // entrypoints for all other perspectives as wel
+  const alertActionContextName = `alert-detail-toolbar-actions${
+    perspective === 'admin' ? '' : '-' + perspective
+  }`;
+  const alertActionContext = {};
+  alertActionContext[alertActionContextName] = { alert };
+
   useRulesAlertsPoller(namespace, dispatch, alertsSource);
 
   return (
@@ -189,7 +199,7 @@ const AlertsDetailsPage_: React.FC<AlertsDetailsPageProps> = ({ history, match }
                 <SectionHeading text={t('Alert details')} />
               </ToolbarItem>
               <ToolbarGroup alignment={{ default: 'alignRight' }}>
-                <ActionServiceProvider context={{ 'alert-detail-toolbar-actions': { alert } }}>
+                <ActionServiceProvider context={alertActionContext}>
                   {({ actions, loaded }) =>
                     loaded
                       ? actions.map((action) => {