From d9d84f45bc103775f0d6133408ea809e7025ad62 Mon Sep 17 00:00:00 2001 From: Erik Sundell Date: Mon, 3 Dec 2018 16:58:46 +0100 Subject: [PATCH] only make it possible to scan for older logs if there is at least one non failing selector --- public/app/features/explore/Explore.tsx | 2 ++ public/app/features/explore/Logs.tsx | 16 ++++++++++++++-- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/public/app/features/explore/Explore.tsx b/public/app/features/explore/Explore.tsx index 3e35a68bf84..17ee35fda9b 100644 --- a/public/app/features/explore/Explore.tsx +++ b/public/app/features/explore/Explore.tsx @@ -836,6 +836,7 @@ export class Explore extends React.PureComponent { const graphLoading = queryTransactions.some(qt => qt.resultType === 'Graph' && !qt.done); const tableLoading = queryTransactions.some(qt => qt.resultType === 'Table' && !qt.done); const logsLoading = queryTransactions.some(qt => qt.resultType === 'Logs' && !qt.done); + const logExpressionExists = queryTransactions.some(qt => qt.resultType === 'Logs' && qt.query.expr && !qt.error); const loading = queryTransactions.some(qt => !qt.done); return ( @@ -969,6 +970,7 @@ export class Explore extends React.PureComponent { range={range} scanning={scanning} scanRange={scanRange} + logExpressionExists={logExpressionExists} /> )} diff --git a/public/app/features/explore/Logs.tsx b/public/app/features/explore/Logs.tsx index 6447ec895ed..5cadfb13b0d 100644 --- a/public/app/features/explore/Logs.tsx +++ b/public/app/features/explore/Logs.tsx @@ -140,6 +140,7 @@ interface LogsProps { range?: RawTimeRange; scanning?: boolean; scanRange?: RawTimeRange; + logExpressionExists?: boolean; onChangeTime?: (range: RawTimeRange) => void; onClickLabel?: (label: string, value: string) => void; onStartScanning?: () => void; @@ -239,7 +240,17 @@ export default class Logs extends PureComponent { }; render() { - const { className = '', data, loading = false, onClickLabel, position, range, scanning, scanRange } = this.props; + const { + className = '', + data, + loading = false, + onClickLabel, + position, + range, + scanning, + scanRange, + logExpressionExists, + } = this.props; const { dedup, deferLogs, hiddenLogLevels, renderAll, showLocalTime, showUtc } = this.state; let { showLabels } = this.state; const hasData = data && data.rows && data.rows.length > 0; @@ -357,7 +368,8 @@ export default class Logs extends PureComponent { ))} {hasData && deferLogs && Rendering {dedupedData.rows.length} rows...} - {!loading && + {logExpressionExists && + !loading && !hasData && !scanning && (