fix: Fixed missing Authrole values on ExecutionMetaDataExtra (#255)

Signed-off-by: Jason Porter <[email protected]>
flyteorg · Jan 5, 2022 · 71af238 · 71af238
1 parent ec84ffd
commit 71af238
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/src/components/Executions/ExecutionDetails/ExecutionMetadataExtra.tsx b/src/components/Executions/ExecutionDetails/ExecutionMetadataExtra.tsx
@@ -34,7 +34,12 @@ export const ExecutionMetadataExtra: React.FC<{
     const commonStyles = useCommonStyles();
     const styles = useStyles();
 
-    const { launchPlan: launchPlanId, maxParallelism } = execution.spec;
+    const {
+        launchPlan: launchPlanId,
+        maxParallelism,
+        authRole
+    } = execution.spec;
+
     const [launchPlanSpec, setLaunchPlanSpec] = React.useState<
         Partial<LaunchPlanSpec>
     >({});
@@ -44,9 +49,17 @@ export const ExecutionMetadataExtra: React.FC<{
     }, [launchPlanId]);
 
     const details: DetailItem[] = [
+        {
+            label: ExecutionMetadataLabels.iam,
+            value:
+                authRole?.assumableIamRole ||
+                ExecutionMetadataLabels.securityContextDefault
+        },
         {
             label: ExecutionMetadataLabels.serviceAccount,
-            value: launchPlanSpec?.authRole?.kubernetesServiceAccount
+            value:
+                authRole?.kubernetesServiceAccount ||
+                ExecutionMetadataLabels.securityContextDefault
         },
         {
             label: ExecutionMetadataLabels.rawOutputPrefix,

diff --git a/src/components/Executions/ExecutionDetails/constants.ts b/src/components/Executions/ExecutionDetails/constants.ts
@@ -6,8 +6,10 @@ export enum ExecutionMetadataLabels {
     relatedTo = 'Related to',
     version = 'Version',
     serviceAccount = 'Service Account',
+    iam = 'IAM Role',
     rawOutputPrefix = 'Raw Output Prefix',
-    parallelism = 'Parallelism'
+    parallelism = 'Parallelism',
+    securityContextDefault = 'default'
 }
 
 export const tabs = {