fix: service account is not properly propagated from the execution in…

…fo (#374) * fix: service account is not properly propagated from the execution info Signed-off-by: eugenejahn <[email protected]>
flyteorg · Apr 13, 2022 · 2e250bb · 2e250bb
1 parent 96a162d
commit 2e250bb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/src/components/Executions/ExecutionDetails/ExecutionMetadataExtra.tsx b/src/components/Executions/ExecutionDetails/ExecutionMetadataExtra.tsx
@@ -38,7 +38,7 @@ export const ExecutionMetadataExtra: React.FC<{
     launchPlan: launchPlanId,
     maxParallelism,
     rawOutputDataConfig,
-    authRole,
+    securityContext,
   } = execution.spec;
 
   const [launchPlanSpec, setLaunchPlanSpec] = React.useState<Partial<LaunchPlanSpec>>({});
@@ -50,11 +50,12 @@ export const ExecutionMetadataExtra: React.FC<{
   const details: DetailItem[] = [
     {
       label: ExecutionMetadataLabels.iam,
-      value: authRole?.assumableIamRole || ExecutionMetadataLabels.securityContextDefault,
+      value: securityContext?.runAs?.iamRole || ExecutionMetadataLabels.securityContextDefault,
     },
     {
       label: ExecutionMetadataLabels.serviceAccount,
-      value: authRole?.kubernetesServiceAccount || ExecutionMetadataLabels.securityContextDefault,
+      value:
+        securityContext?.runAs?.k8sServiceAccount || ExecutionMetadataLabels.securityContextDefault,
     },
     {
       label: ExecutionMetadataLabels.rawOutputPrefix,

diff --git a/src/models/Execution/api.ts b/src/models/Execution/api.ts
@@ -122,10 +122,6 @@ export const createWorkflowExecution = (
     annotations,
   };
 
-  if (authRole?.assumableIamRole || authRole?.kubernetesServiceAccount) {
-    spec.authRole = authRole;
-  }
-
   if (securityContext?.runAs?.iamRole || securityContext?.runAs?.k8sServiceAccount) {
     spec.securityContext = securityContext;
   }