fix(aa): refactor Automated Analysis Recording Setting (#1014)

* show current config Signed-off-by: Max Cao <macao@redhat.com> * prevent setting empty template Signed-off-by: Max Cao <macao@redhat.com> * add reset Signed-off-by: Max Cao <macao@redhat.com> * fix some handling Signed-off-by: Max Cao <macao@redhat.com> * redesign Signed-off-by: Max Cao <macao@redhat.com> * redesign 2 Signed-off-by: Max Cao <macao@redhat.com> * update tests and add confirmation Signed-off-by: Max Cao <macao@redhat.com> * re: fixes Signed-off-by: Max Cao <macao@redhat.com> * fix tests Signed-off-by: Max Cao <macao@redhat.com> * localize Signed-off-by: Max Cao <macao@redhat.com> --------- Signed-off-by: Max Cao <macao@redhat.com>
cryostatio · May 5, 2023 · 093e003 · 093e003
1 parent 00e311e
commit 093e003
Show file tree

Hide file tree

Showing 13 changed files with 458 additions and 307 deletions.
diff --git a/locales/en/common.json b/locales/en/common.json
@@ -20,6 +20,7 @@
   "DESCRIPTION": "Description",
   "DEVELOPMENT": "DEVELOPMENT",
   "DOWNLOAD": "Download",
+  "EDIT": "Edit",
   "FILTER_NAME": "Name",
   "HELP": "Help",
   "HOUR": "Hour",
@@ -54,7 +55,6 @@
   "SUBMITTING": "Submitting",
   "SUGGESTED": "Suggested",
   "TEMPLATE": "Template",
-  "TEMPLATE_HELPER_TEXT_INVALID": "Template must be selected",
   "TIME": "Time",
   "UPLOAD": "Upload",
   "USER_SUBMITTED": "User-submitted",

diff --git a/locales/en/public.json b/locales/en/public.json
@@ -72,8 +72,12 @@
     }
   },
   "AutomatedAnalysisConfigForm": {
-    "ERROR_TITLE": "Error displaying recording configuration settings",
+    "CURRENT_CONFIG": "Current Configuration",
     "FORM_TITLE": "Profiling Recording Configuration",
+    "FORMATTED_TEMPLATE": "Name: {{template.name}}, Type: {{template.type}}",
+    "MAXIMUM_AGE": "Maximum age ({{unit}})",
+    "MAXIMUM_SIZE": "Maximum size ({{unit}})",
+    "SAVE_CHANGES": "Save changes",
     "TEMPLATE_HELPER_TEXT": "The Event Template to be applied to Automated Analysis recordings.",
     "TEMPLATE_INVALID_WARNING": "WARNING: Setting a Target Template as a default template type configuration may not apply to all Target JVMs if the JVMs do not support them."
   },
@@ -230,6 +234,9 @@
     "ERROR_MESSAGE": "Reason: {{message}}",
     "RESOLVE_MESSAGE": "Reload the page and try again. If the error still persists, see the list of <issue>{{knownIssue}}</issue> or <report>{{fileReport}}</report>."
   },
+  "ErrorView": {
+    "EVENT_TEMPLATES": "Error retrieving event templates"
+  },
   "JvmDetailsCard": {
     "CARD_DESCRIPTION": "Display details about the selected target JVM.",
     "CARD_DESCRIPTION_FULL": "View information such as the connection URL, labels, and annotations belonging to the selected target JVM.",

diff --git a/src/app/AppLayout/AppLayout.tsx b/src/app/AppLayout/AppLayout.tsx
@@ -619,7 +619,12 @@ const AppLayout: React.FC<AppLayoutProps> = ({ children }) => {
         >
           {children}
         </Page>
-        <AuthModal visible={showAuthModal} onDismiss={dismissAuthModal} onSave={authModalOnSave} />
+        <AuthModal
+          visible={showAuthModal}
+          onDismiss={dismissAuthModal}
+          onSave={authModalOnSave}
+          targetObs={serviceContext.target.target()}
+        />
         <SslErrorModal visible={showSslErrorModal} onDismiss={dismissSslErrorModal} />
       </CryostatJoyride>
     </GlobalQuickStartDrawer>

diff --git a/src/app/AppLayout/AuthModal.tsx b/src/app/AppLayout/AuthModal.tsx
@@ -36,21 +36,22 @@
  * SOFTWARE.
  */
 import { ServiceContext } from '@app/Shared/Services/Services';
-import { NO_TARGET } from '@app/Shared/Services/Target.service';
+import { NO_TARGET, Target } from '@app/Shared/Services/Target.service';
 import { useSubscriptions } from '@app/utils/useSubscriptions';
 import { Modal, ModalVariant, Text } from '@patternfly/react-core';
 import * as React from 'react';
 import { Link } from 'react-router-dom';
-import { filter, first, map, mergeMap } from 'rxjs';
+import { Observable, filter, first, map, mergeMap } from 'rxjs';
 import { CredentialAuthForm } from './CredentialAuthForm';
 
 export interface AuthModalProps {
   visible: boolean;
   onDismiss: () => void;
   onSave: () => void;
+  targetObs: Observable<Target>;
 }
 
-export const AuthModal: React.FC<AuthModalProps> = ({ onDismiss, onSave: onPropsSave, ...props }) => {
+export const AuthModal: React.FC<AuthModalProps> = ({ onDismiss, onSave: onPropsSave, targetObs, ...props }) => {
   const context = React.useContext(ServiceContext);
   const [loading, setLoading] = React.useState(false);
   const addSubscription = useSubscriptions();
@@ -59,8 +60,7 @@ export const AuthModal: React.FC<AuthModalProps> = ({ onDismiss, onSave: onProps
     (username: string, password: string) => {
       setLoading(true);
       addSubscription(
-        context.target
-          .target()
+        targetObs
           .pipe(
             filter((target) => target !== NO_TARGET),
             first(),
@@ -75,7 +75,7 @@ export const AuthModal: React.FC<AuthModalProps> = ({ onDismiss, onSave: onProps
           })
       );
     },
-    [addSubscription, context.authCredentials, context.target, setLoading, onPropsSave]
+    [addSubscription, context.authCredentials, targetObs, setLoading, onPropsSave]
   );
 
   return (