Fix from time parameters on tracing tabs (kiali#2021)

mtho11 · Dec 3, 2020 · 77c981c · 77c981c
1 parent b6d36ac
commit 77c981c
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 8 deletions.
diff --git a/src/components/JaegerIntegration/TracesComponent.tsx b/src/components/JaegerIntegration/TracesComponent.tsx
@@ -28,16 +28,16 @@ import { config } from '../../config';
 import { TracesFetcher } from './TracesFetcher';
 import { getTimeRangeMicros, buildTags } from './JaegerHelper';
 import { SpanDetails } from './JaegerResults/SpanDetails';
-import { TargetKind, TimeInMilliseconds } from 'types/Common';
-import { durationSelector } from '../../store/Selectors';
+import { isEqualTimeRange, TargetKind, TimeInMilliseconds, TimeRange } from 'types/Common';
+import { timeRangeSelector } from '../../store/Selectors';
 
 interface TracesProps {
   namespace: string;
   target: string;
   targetKind: TargetKind;
   urlJaeger: string;
   namespaceSelector: boolean;
-  duration: number;
+  timeRange: TimeRange;
   selectedTrace?: JaegerTrace;
   lastRefreshAt: TimeInMilliseconds;
 }
@@ -143,7 +143,12 @@ class TracesComponent extends React.Component<TracesProps, TracesState> {
         this.setState({ traces: traces });
       }
     }
-    if (prevProps.duration !== this.props.duration || prevProps.lastRefreshAt !== this.props.lastRefreshAt) {
+
+    const changedTimeRange = !isEqualTimeRange(this.props.timeRange, prevProps.timeRange);
+    if (this.props.lastRefreshAt !== prevProps.lastRefreshAt || changedTimeRange) {
+      if (changedTimeRange) {
+        this.fetcher.resetLastFetchTime();
+      }
       this.fetchTraces();
     }
   }
@@ -418,7 +423,7 @@ class TracesComponent extends React.Component<TracesProps, TracesState> {
 
 const mapStateToProps = (state: KialiAppState) => {
   return {
-    duration: durationSelector(state),
+    timeRange: timeRangeSelector(state),
     urlJaeger: state.jaegerState.info ? state.jaegerState.info.url : '',
     namespaceSelector: state.jaegerState.info ? state.jaegerState.info.namespaceSelector : true,
     selectedTrace: state.jaegerState.selectedTrace,

diff --git a/src/pages/AppDetails/AppDetailsPage.tsx b/src/pages/AppDetails/AppDetailsPage.tsx
@@ -194,11 +194,11 @@ class AppDetails extends React.Component<AppDetailsProps, AppDetailsState> {
     switch (this.state.currentTab) {
       case 'info':
       case 'traffic':
-      case 'traces':
         useCustomTime = false;
         break;
       case 'in_metrics':
       case 'out_metrics':
+      case 'traces':
         useCustomTime = true;
         break;
     }

diff --git a/src/pages/ServiceDetails/ServiceDetailsPage.tsx b/src/pages/ServiceDetails/ServiceDetailsPage.tsx
@@ -188,10 +188,10 @@ class ServiceDetails extends React.Component<ServiceDetailsProps, ServiceDetails
     switch (this.state.currentTab) {
       case 'info':
       case 'traffic':
-      case 'traces':
         useCustomTime = false;
         break;
       case 'metrics':
+      case 'traces':
         useCustomTime = true;
         break;
     }

diff --git a/src/pages/WorkloadDetails/WorkloadDetailsPage.tsx b/src/pages/WorkloadDetails/WorkloadDetailsPage.tsx
@@ -201,12 +201,12 @@ class WorkloadDetails extends React.Component<WorkloadDetailsPageProps, Workload
     switch (this.state.currentTab) {
       case 'info':
       case 'traffic':
-      case 'traces':
         useCustomTime = false;
         break;
       case 'in_metrics':
       case 'out_metrics':
       case 'logs':
+      case 'traces':
         useCustomTime = true;
         break;
     }