fix for archived plr loader logic

anwesha-palit-redhat · arvindk-softwaredev · commit 8e91e58f5619 · 2026-04-06T11:40:17.000+05:30
diff --git a/src/components/logs/Logs.tsx b/src/components/logs/Logs.tsx
@@ -411,7 +411,7 @@ const Logs: FC<LogsProps> = ({
   }, [logData, activeStep, findTargetRowForActiveStep]);
 
   return (
-    <div className="pf-v5-u-h-100 pf-v5-u-w-100">
+    <div className="pf-v6-u-h-100 pf-v6-u-w-100">
       {error && (
         <Alert
           variant="danger"
@@ -422,8 +422,8 @@ const Logs: FC<LogsProps> = ({
       <LogViewer
         useAnsiClasses={true}
         header={
-          <Banner className="pf-v5-l-flex pf-v5-l-gap-md">
-            <span data-test-id="logs-taskName" className="pf-v5-u-font-size-md">
+          <Banner className="pf-v6-l-flex pf-v6-l-gap-md">
+            <span data-test-id="logs-taskName" className="pf-v6-u-font-size-md">
               {taskName}
             </span>
             {stillFetching ? (
diff --git a/src/components/logs/TektonTaskRunLog.scss b/src/components/logs/TektonTaskRunLog.scss
@@ -3,7 +3,7 @@
 .odc-tekton-taskrun-log {
   overflow: hidden;
 
-  .pf-v5-c-log-viewer__text {
+  .pf-v6-c-log-viewer__text {
     white-space: pre;
   }
 
diff --git a/src/components/pipelineRuns-details/PipelineRunDetailsPage.tsx b/src/components/pipelineRuns-details/PipelineRunDetailsPage.tsx
@@ -48,7 +48,7 @@ const PipelineRunDetailsPage: FC<PipelineRunDetailsPageProps> = ({
   const { t } = useTranslation('plugin__pipelines-console-plugin');
   const [pipelineRun, k8sLoaded, trLoaded] = usePipelineRun(namespace, name);
   /* this needs decoupling */
-  const pipelineRunLoaded = k8sLoaded && trLoaded;
+  const pipelineRunLoaded = k8sLoaded || trLoaded;
 
   const customActionMenu = useCallback((_kindObj, obj) => {
     const reference = getReferenceForModel(PipelineRunModel);
diff --git a/src/components/pipelineRuns-details/PipelineRunLogs.scss b/src/components/pipelineRuns-details/PipelineRunLogs.scss
@@ -9,7 +9,7 @@
   bottom: 0;
   overflow: hidden;
 
-  .pf-v5-c-log-viewer__text {
+  .pf-v6-c-log-viewer__text {
     white-space: pre;
   }
 
diff --git a/src/components/pipelineRuns-details/PipelineRunLogs.tsx b/src/components/pipelineRuns-details/PipelineRunLogs.tsx
@@ -292,7 +292,7 @@ export const PipelineRunLogsWithActiveTask: FC<
     plrStatus !== ComputedStatus.Running &&
     plrStatus !== ComputedStatus.Pending &&
     plrStatus !== ComputedStatus.Cancelling;
-  const [taskRuns, taskRunsLoaded, , , pendingAdmission, proxyUnavailable] =
+  const [taskRuns, k8sLoaded, trLoaded, , pendingAdmission, proxyUnavailable] =
     useTaskRuns(
       obj?.metadata?.namespace,
       obj?.metadata?.name,
@@ -303,6 +303,8 @@ export const PipelineRunLogsWithActiveTask: FC<
         pipelineRunManagedBy: obj?.spec?.managedBy,
       },
     );
+  /* this needs decoupling */
+  const taskRunsLoaded = k8sLoaded || trLoaded;
   const { isResourceManagedByKueue } = useMultiClusterProxyService({
     managedBy: obj?.spec?.managedBy,
   });
diff --git a/src/components/pipelineRuns-details/PipelineRunVisualization.tsx b/src/components/pipelineRuns-details/PipelineRunVisualization.tsx
@@ -35,7 +35,7 @@ const PipelineRunVisualization: FC<PipelineRunVisualizationProps> = ({
       },
     );
   /* this needs decoupling */
-  const taskRunsLoaded = k8sLoaded && trLoaded;
+  const taskRunsLoaded = k8sLoaded || trLoaded;
   const pipeline: PipelineKind = usePipelineFromPipelineRun(pipelineRun);
   if (!pipeline) {
     return (
diff --git a/src/components/pipelines-metrics/PipelinesAverageDurationK8s.tsx b/src/components/pipelines-metrics/PipelinesAverageDurationK8s.tsx
@@ -86,7 +86,7 @@ const PipelinesAverageDurationK8s: FC<PipelinesAverageDurationProps> = ({
   interval,
   parentName,
   namespace,
-  width = 530
+  width = 530,
 }) => {
   const { t } = useTranslation('plugin__pipelines-console-plugin');
   const startTimespan = timespan - parsePrometheusDuration('1d');
@@ -299,16 +299,18 @@ const PipelinesAverageDurationK8s: FC<PipelinesAverageDurationProps> = ({
     <>
       <Card
         className={classNames({
-          'pf-v6-u-h-100 pipeline-overview__min-width-full pipeline-overview__overflow-hidden pf-v6-u-display-flex pf-v6-u-flex-direction-column': !averageDurationError,
+          'pf-v6-u-h-100 pipeline-overview__min-width-full pipeline-overview__overflow-hidden pf-v6-u-display-flex pf-v6-u-flex-direction-column':
+            !averageDurationError,
           'card-border': bordered,
         })}
       >
         <CardTitle className="pf-v6-u-pb-0">
           <span>{t('Average duration')}</span>
         </CardTitle>
-        <CardBody 
+        <CardBody
           className={classNames({
-            'pf-v6-u-flex-1 pipeline-overview__min-height-0 pf-v6-u-display-flex pf-v6-u-flex-direction-column pf-v6-u-justify-content-flex-end pf-v6-u-align-items-flex-start pf-v6-u-p-0': !averageDurationError,
+            'pf-v6-u-flex-1 pipeline-overview__min-height-0 pf-v6-u-display-flex pf-v6-u-flex-direction-column pf-v6-u-justify-content-flex-end pf-v6-u-align-items-flex-start pf-v6-u-p-0':
+              !averageDurationError,
           })}
         >
           {averageDurationError ? (
@@ -357,7 +359,7 @@ const PipelinesAverageDurationK8s: FC<PipelinesAverageDurationProps> = ({
                   </ChartGroup>
                 </Chart>
               ) : (
-                <div className="pipeline-overview__number-of-plr-card__loading pf-v5-u-h-100">
+                <div className="pipeline-overview__number-of-plr-card__loading pf-v6-u-h-100">
                   <LoadingInline />
                 </div>
               )}
diff --git a/src/components/pipelines-overview/PipelinesOverviewPageK8s.tsx b/src/components/pipelines-overview/PipelinesOverviewPageK8s.tsx
@@ -60,7 +60,7 @@ const PipelinesOverviewPageK8s: FC = () => {
       <PageSection hasBodyWrapper={false} className="pf-v6-u-pl-md">
         <Title headingLevel="h2">{t('Overview')}</Title>
       </PageSection>
-      <div className="pf-v5-u-m-md">
+      <div className="pf-v6-u-m-md">
         <K8sDataLimitationAlert />
       </div>
       <Flex className="pf-v6-u-pl-md pf-v6-u-pr-md pf-v6-u-mb-md">
diff --git a/src/components/pipelines-tasks/tasks-details-pages/TaskRunLog.scss b/src/components/pipelines-tasks/tasks-details-pages/TaskRunLog.scss
@@ -5,7 +5,7 @@
   flex: 1;
   padding: var(--pf-t--global--spacer--xl);
 
-  .pf-v5-c-log-viewer__text {
+  .pf-v6-c-log-viewer__text {
     white-space: pre;
   }
 
diff --git a/src/components/quick-search/QuickSearchBar.scss b/src/components/quick-search/QuickSearchBar.scss
@@ -48,7 +48,10 @@
     margin-left: auto !important;
   }
   &__icon {
-    background-color: var(--pf-t--global--background--color--secondary--default);
-    border-bottom: solid var(--pf-v5-global--BorderWidth--sm) var(--pf-v5-global--BorderColor--200);
+    background-color: var(
+      --pf-t--global--background--color--secondary--default
+    );
+    border-bottom: solid var(--pf-v6-global--BorderWidth--sm)
+      var(--pf-v6-global--BorderColor--200);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`.odc-tekton-taskrun-log {`
`4`	`4`	`overflow: hidden;`
`5`	`5`
`6`		`- .pf-v5-c-log-viewer__text {`
	`6`	`+ .pf-v6-c-log-viewer__text {`
`7`	`7`	`white-space: pre;`
`8`	`8`	`}`
`9`	`9`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`bottom: 0;`
`10`	`10`	`overflow: hidden;`
`11`	`11`
`12`		`- .pf-v5-c-log-viewer__text {`
	`12`	`+ .pf-v6-c-log-viewer__text {`
`13`	`13`	`white-space: pre;`
`14`	`14`	`}`
`15`	`15`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`flex: 1;`
`6`	`6`	`padding: var(--pf-t--global--spacer--xl);`
`7`	`7`
`8`		`- .pf-v5-c-log-viewer__text {`
	`8`	`+ .pf-v6-c-log-viewer__text {`
`9`	`9`	`white-space: pre;`
`10`	`10`	`}`
`11`	`11`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,10 @@`
`48`	`48`	`margin-left: auto !important;`
`49`	`49`	`}`
`50`	`50`	`&__icon {`
`51`		`- background-color: var(--pf-t--global--background--color--secondary--default);`
`52`		`- border-bottom: solid var(--pf-v5-global--BorderWidth--sm) var(--pf-v5-global--BorderColor--200);`
	`51`	`+ background-color: var(`
	`52`	`+ --pf-t--global--background--color--secondary--default`
	`53`	`+ );`
	`54`	`+ border-bottom: solid var(--pf-v6-global--BorderWidth--sm)`
	`55`	`+ var(--pf-v6-global--BorderColor--200);`
`53`	`56`	`}`
`54`	`57`	`}`