fix(#14022): restrict multiple run now calls

openmetadata-ui/src/main/resources/ui/src/components/Applications/AppRunsHistory/AppRunsHistory.component.tsx

@@ -59,7 +59,12 @@ import {
 
 const AppRunsHistory = forwardRef(
   (
-    { appData, maxRecords, showPagination = true }: AppRunsHistoryProps,
+    {
+      appData,
+      maxRecords,
+      showPagination = true,
+      runsData,
+    }: AppRunsHistoryProps,
     ref
   ) => {
     const { t } = useTranslation();
@@ -271,8 +276,13 @@ const AppRunsHistory = forwardRef(
     }));
 
     useEffect(() => {
-      fetchAppHistory();
-    }, [fqn, pageSize]);
+      if (runsData) {
+        setAppRunsHistoryData(runsData);
+        setIsLoading(false);
+      } else {
+        fetchAppHistory();
+      }
+    }, [fqn, pageSize, runsData]);
 
     return (
       <Row gutter={[16, 16]}>

openmetadata-ui/src/main/resources/ui/src/components/Applications/AppRunsHistory/AppRunsHistory.interface.ts

@@ -26,4 +26,5 @@ export interface AppRunsHistoryProps {
   maxRecords?: number;
   appData?: App;
   showPagination?: boolean;
+  runsData?: AppRunRecordWithId[];
 }

openmetadata-ui/src/main/resources/ui/src/components/Applications/AppSchedule/AppSchedule.component.tsx

@@ -11,6 +11,7 @@
  *  limitations under the License.
  */
 import { Button, Col, Divider, Modal, Row, Space, Typography } from 'antd';
+import { AxiosError } from 'axios';
 import cronstrue from 'cronstrue';
 import React, {
   useCallback,
@@ -24,13 +25,26 @@ import {
   AppScheduleClass,
   AppType,
 } from '../../../generated/entity/applications/app';
-import { PipelineType } from '../../../generated/entity/services/ingestionPipelines/ingestionPipeline';
+import { Status } from '../../../generated/entity/applications/appRunRecord';
+import {
+  PipelineState,
+  PipelineStatus,
+  PipelineType,
+} from '../../../generated/entity/services/ingestionPipelines/ingestionPipeline';
+import { Paging } from '../../../generated/type/paging';
+import { getApplicationRuns } from '../../../rest/applicationAPI';
 import { getIngestionPipelineByFqn } from '../../../rest/ingestionPipelineAPI';
+import { getStatusFromPipelineState } from '../../../utils/ApplicationUtils';
 import { getIngestionFrequency } from '../../../utils/CommonUtils';
+import { getEpochMillisForPastDays } from '../../../utils/date-time/DateTimeUtils';
+import { showErrorToast } from '../../../utils/ToastUtils';
 import TestSuiteScheduler from '../../AddDataQualityTest/components/TestSuiteScheduler';
 import Loader from '../../Loader/Loader';
 import AppRunsHistory from '../AppRunsHistory/AppRunsHistory.component';
-import { AppRunsHistoryRef } from '../AppRunsHistory/AppRunsHistory.interface';
+import {
+  AppRunRecordWithId,
+  AppRunsHistoryRef,
+} from '../AppRunsHistory/AppRunsHistory.interface';
 import { AppScheduleProps } from './AppScheduleProps.interface';
 
 const AppSchedule = ({
@@ -44,6 +58,22 @@ const AppSchedule = ({
   const appRunsHistoryRef = useRef<AppRunsHistoryRef>(null);
   const [isPipelineDeployed, setIsPipelineDeployed] = useState(false);
   const [isLoading, setIsLoading] = useState(true);
+  const [appRunsHistoryData, setAppRunsHistoryData] = useState<
+    AppRunRecordWithId[]
+  >([]);
+
+  const isExternalApp = useMemo(
+    () => appData?.appType === AppType.External,
+    [appData]
+  );
+
+  const isAppRunning = useMemo(() => {
+    if (appRunsHistoryData.length > 0) {
+      return appRunsHistoryData[0].status === Status.Running;
+    } else {
+      return false;
+    }
+  }, [appRunsHistoryData]);
 
   const fetchPipelineDetails = useCallback(async () => {
     setIsLoading(true);
@@ -67,6 +97,60 @@ const AppSchedule = ({
     }
   }, [appData]);
 
+  const fetchAppHistory = useCallback(
+    async (pagingOffset?: Paging) => {
+      try {
+        setIsLoading(true);
+
+        if (isExternalApp) {
+          const currentTime = Date.now();
+          // past 30 days
+          const startDay = getEpochMillisForPastDays(30);
+
+          const { data } = await getApplicationRuns(
+            appData.fullyQualifiedName ?? '',
+            {
+              startTs: startDay,
+              endTs: currentTime,
+            }
+          );
+
+          setAppRunsHistoryData(
+            data
+              .map((item) => ({
+                ...item,
+                status: getStatusFromPipelineState(
+                  (item as PipelineStatus).pipelineState ?? PipelineState.Failed
+                ),
+                id: (item as PipelineStatus).runId ?? '',
+              }))
+              .slice(0, 1)
+          );
+        } else {
+          const { data } = await getApplicationRuns(
+            appData.fullyQualifiedName ?? '',
+            {
+              offset: pagingOffset?.offset ?? 0,
+              limit: 1,
+            }
+          );
+
+          setAppRunsHistoryData(
+            data.map((item) => ({
+              ...item,
+              id: `${item.appId}-${item.runType}-${item.timestamp}`,
+            }))
+          );
+        }
+      } catch (err) {
+        showErrorToast(err as AxiosError);
+      } finally {
+        setIsLoading(false);
+      }
+    },
+    [appData]
+  );
+
   const cronString = useMemo(() => {
     if (appData.appSchedule) {
       const cronExp =
@@ -91,7 +175,7 @@ const AppSchedule = ({
 
   const onAppTrigger = async () => {
     await onDemandTrigger();
-    appRunsHistoryRef.current?.refreshAppHistory();
+    await fetchAppHistory();
   };
 
   const appRunHistory = useMemo(() => {
@@ -104,6 +188,7 @@ const AppSchedule = ({
           appData={appData}
           maxRecords={1}
           ref={appRunsHistoryRef}
+          runsData={appRunsHistoryData}
           showPagination={false}
         />
       );
@@ -122,10 +207,11 @@ const AppSchedule = ({
         {t('message.no-ingestion-pipeline-found')}
       </Typography.Text>
     );
-  }, [appData, isPipelineDeployed, appRunsHistoryRef]);
+  }, [appData, isPipelineDeployed, appRunsHistoryRef, appRunsHistoryData]);
 
   useEffect(() => {
     fetchPipelineDetails();
+    fetchAppHistory();
   }, []);
 
   if (isLoading) {
@@ -170,7 +256,6 @@ const AppSchedule = ({
               {appData.appType === AppType.External && (
                 <Button
                   data-testid="deploy-button"
-                  disabled={appData.deleted}
                   type="primary"
                   onClick={onDeployTrigger}>
                   {t('label.deploy')}
@@ -179,15 +264,14 @@ const AppSchedule = ({
 
               <Button
                 data-testid="edit-button"
-                disabled={appData.deleted}
                 type="primary"
                 onClick={() => setShowModal(true)}>
                 {t('label.edit')}
               </Button>
 
               <Button
                 data-testid="run-now-button"
-                disabled={appData.deleted}
+                disabled={isAppRunning}
                 type="primary"
                 onClick={onAppTrigger}>
                 {t('label.run-now')}