Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
643d3cd
WIP: Adding in new reporting diag tool
Jul 7, 2020
6ee796a
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Jul 13, 2020
ae1a3e6
WIP: chrome-binary test + log capturing/error handling
Jul 14, 2020
8bcd274
More wip on diagnostic tool
Jul 27, 2020
b10c351
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Aug 3, 2020
61fe997
More work adding in diagnose routes
Aug 4, 2020
77ad9c5
Alter link in description + minor rename of chrome => browser
Aug 4, 2020
0d434aa
Wiring UI to API + some polish on UI flow
Aug 5, 2020
0e661c8
WIP: Add in screenshot diag route
Aug 10, 2020
a395b42
Merge branch 'master' into reporting/diagnostics
Aug 11, 2020
b3131c9
Adding in screenshot diag route, hooking up client to it
Aug 17, 2020
5bd3f2f
Merge branch 'master' into reporting/diagnostics
Aug 20, 2020
7f6ac93
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Aug 21, 2020
6a03654
Add missing lib check + memory check
Aug 21, 2020
09b9e00
Working screenshot test + config check for RAM
Aug 21, 2020
f0ead22
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
joelgriffith Aug 24, 2020
c5aab98
Small test helper consolidation + screenshot diag test
joelgriffith Aug 24, 2020
0cddac1
Delete old i18n translations
joelgriffith Aug 24, 2020
21638ec
PR feedback, browser tests, rename, re-organize import statements and…
joelgriffith Aug 25, 2020
b67ce10
Lite rename for consistency
Aug 31, 2020
9a303eb
Remove old validate check i18n
Aug 31, 2020
5ab2442
Add config check
Aug 31, 2020
4e092ba
i18n all the things!
Sep 1, 2020
451a679
Docs on diagnostics tool
Sep 1, 2020
c1db6e9
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Sep 1, 2020
f9b81f3
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Sep 2, 2020
14e0712
Fixes, better readability, spelling and more for diagnostic tool
Sep 2, 2020
b72349e
Translate a few error messages
Sep 2, 2020
9ba6761
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Sep 3, 2020
8bc5b26
Rename of test => start_logs for clarity. Move to observables
Sep 4, 2020
0b8dec0
Merge branch 'master' into reporting/diagnostics
Sep 4, 2020
5cce4f8
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Sep 4, 2020
032442d
Merge branch 'master' into reporting/diagnostics
elasticmachine Sep 8, 2020
9200c04
Gathering logs even during process exit or crash
Sep 8, 2020
04c9276
Adds a test case for the browser exiting during the diag check
Sep 8, 2020
c4eecf8
Tap into browser logs for checking output
Sep 9, 2020
2c54609
Merge remote-tracking branch 'upstream/master' into reporting/diagnos…
Sep 9, 2020
1c86e84
Rename asciidoc diag id
Sep 9, 2020
27ab7c4
Remove duplicate shared object message
Sep 9, 2020
902a72b
Add better comment as to why we merge events + wait for a period of time
Sep 9, 2020
fefcfa8
Cloning logger for mirroring browser stderr to kibana output
Sep 9, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions docs/user/reporting/reporting-troubleshooting.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@

Having trouble? Here are solutions to common problems you might encounter while using Reporting.

* <<reporting-diagnostics>>
* <<reporting-troubleshooting-system-dependencies>>
* <<reporting-troubleshooting-text-incorrect>>
* <<reporting-troubleshooting-missing-data>>
Expand All @@ -15,6 +16,11 @@ Having trouble? Here are solutions to common problems you might encounter while
* <<reporting-troubleshooting-puppeteer-debug-logs>>
* <<reporting-troubleshooting-system-requirements>>

[float]
[[reporting-diagnostics]]
=== Reporting Diagnostics
Reporting comes with a built-in utility to try to automatically find common issues. When Kibana is running, navigate to the Report Listing page, and click the "Run reporting diagnostics..." button. This will open up a diagnostic tool that checks various parts of the Kibana deployment to come up with any relevant recommendations.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any thoughts or feelings about having the label just be Reporting diagnostics instead of the longer phrase?


[float]
[[reporting-troubleshooting-system-dependencies]]
=== System dependencies
Expand Down
1 change: 1 addition & 0 deletions x-pack/plugins/reporting/common/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ export const API_BASE_URL_V1 = '/api/reporting/v1'; //
export const API_BASE_GENERATE_V1 = `${API_BASE_URL_V1}/generate`;
export const API_LIST_URL = '/api/reporting/jobs';
export const API_GENERATE_IMMEDIATE = `${API_BASE_URL_V1}/generate/immediate/csv/saved-object`;
export const API_DIAGNOSE_URL = `${API_BASE_URL}/diagnose`;

export const CONTENT_TYPE_CSV = 'text/csv';
export const CSV_REPORTING_ACTION = 'downloadCsvReport';
Expand Down
281 changes: 281 additions & 0 deletions x-pack/plugins/reporting/public/components/report_diagnostic.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,281 @@
/*
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
* or more contributor license agreements. Licensed under the Elastic License;
* you may not use this file except in compliance with the Elastic License.
*/

import { i18n } from '@kbn/i18n';
import React, { useState, Fragment } from 'react';
import { FormattedMessage } from '@kbn/i18n/react';
import {
EuiButton,
EuiButtonEmpty,
EuiCallOut,
EuiCodeBlock,
EuiFlyout,
EuiFlyoutBody,
EuiFlyoutHeader,
EuiSpacer,
EuiSteps,
EuiText,
EuiTitle,
} from '@elastic/eui';
import { ReportingAPIClient, DiagnoseResponse } from '../lib/reporting_api_client';

interface Props {
apiClient: ReportingAPIClient;
}

type ResultStatus = 'danger' | 'incomplete' | 'complete';

enum statuses {
configStatus = 'configStatus',
chromeStatus = 'chromeStatus',
screenshotStatus = 'screenshotStatus',
}

interface State {
isFlyoutVisible: boolean;
configStatus: ResultStatus;
chromeStatus: ResultStatus;
screenshotStatus: ResultStatus;
help: string[];
logs: string;
isBusy: boolean;
success: boolean;
}

const initialState: State = {
[statuses.configStatus]: 'incomplete',
[statuses.chromeStatus]: 'incomplete',
[statuses.screenshotStatus]: 'incomplete',
isFlyoutVisible: false,
help: [],
logs: '',
isBusy: false,
success: true,
};

export const ReportDiagnostic = ({ apiClient }: Props) => {
const [state, setStateBase] = useState(initialState);
const setState = (s: Partial<typeof state>) =>
setStateBase({
...state,
...s,
});
const {
configStatus,
isBusy,
screenshotStatus,
chromeStatus,
isFlyoutVisible,
help,
logs,
success,
} = state;

const closeFlyout = () => setState({ ...initialState, isFlyoutVisible: false });
const showFlyout = () => setState({ isFlyoutVisible: true });
const apiWrapper = (apiMethod: () => Promise<DiagnoseResponse>, statusProp: statuses) => () => {
setState({ isBusy: true, [statusProp]: 'incomplete' });
apiMethod()
.then((response) => {
setState({
isBusy: false,
help: response.help,
logs: response.logs,
success: response.success,
[statusProp]: response.success ? 'complete' : 'danger',
});
})
.catch((error) => {
setState({
isBusy: false,
help: [
i18n.translate('xpack.reporting.listing.diagnosticApiCallFailure', {
defaultMessage: `There was a problem running the diagnostic: {error}`,
values: { error },
}),
],
logs: `${error.message}`,
success: false,
[statusProp]: 'danger',
});
});
};

const steps = [
{
title: i18n.translate('xpack.reporting.listing.diagnosticConfigTitle', {
defaultMessage: 'Verify Kibana Configuration',
}),
children: (
<Fragment>
<FormattedMessage
id="xpack.reporting.listing.diagnosticConfigMessage"
defaultMessage="This check ensures your Kibana configuration is setup properly for reports."
/>
<EuiSpacer />
<EuiButton
disabled={isBusy || configStatus === 'complete'}
isLoading={isBusy && configStatus === 'incomplete'}
onClick={apiWrapper(apiClient.verifyConfig, statuses.configStatus)}
iconType={configStatus === 'complete' ? 'check' : undefined}
>
<FormattedMessage
id="xpack.reporting.listing.diagnosticConfigButton"
defaultMessage="Verify Configuration"
/>
</EuiButton>
</Fragment>
),
status: !success && configStatus !== 'complete' ? 'danger' : configStatus,
},
];

if (configStatus === 'complete') {
steps.push({
title: i18n.translate('xpack.reporting.listing.diagnosticBrowserTitle', {
defaultMessage: 'Check Browser',
}),
children: (
<Fragment>
<FormattedMessage
id="xpack.reporting.listing.diagnosticBrowserMessage"
defaultMessage="Reporting utilizes a headless browser to generate PDF and PNGS. This check validates
that the browser can launch successfully."
/>
<EuiSpacer />
<EuiButton
disabled={isBusy || chromeStatus === 'complete'}
onClick={apiWrapper(apiClient.verifyBrowser, statuses.chromeStatus)}
isLoading={isBusy && chromeStatus === 'incomplete'}
iconType={chromeStatus === 'complete' ? 'check' : undefined}
>
<FormattedMessage
id="xpack.reporting.listing.diagnosticBrowserButton"
defaultMessage="Check Browser"
/>
</EuiButton>
</Fragment>
),
status: !success && chromeStatus !== 'complete' ? 'danger' : chromeStatus,
});
}

if (chromeStatus === 'complete') {
steps.push({
title: i18n.translate('xpack.reporting.listing.diagnosticScreenshotTitle', {
defaultMessage: 'Check Screen Capture Capabilities',
}),
children: (
<Fragment>
<FormattedMessage
id="xpack.reporting.listing.diagnosticScreenshotMessage"
defaultMessage="This check ensures the headless browser can capture a screenshot of a page."
/>
<EuiSpacer />
<EuiButton
disabled={isBusy || screenshotStatus === 'complete'}
onClick={apiWrapper(apiClient.verifyScreenCapture, statuses.screenshotStatus)}
isLoading={isBusy && screenshotStatus === 'incomplete'}
iconType={screenshotStatus === 'complete' ? 'check' : undefined}
>
<FormattedMessage
id="xpack.reporting.listing.diagnosticScreenshotButton"
defaultMessage="Capture Screenshot"
/>
</EuiButton>
</Fragment>
),
status: !success && screenshotStatus !== 'complete' ? 'danger' : screenshotStatus,
});
}

if (screenshotStatus === 'complete') {
steps.push({
title: i18n.translate('xpack.reporting.listing.diagnosticSuccessTitle', {
defaultMessage: 'All set!',
}),
children: (
<Fragment>
<FormattedMessage
id="xpack.reporting.listing.diagnosticSuccessMessage"
defaultMessage="Excellent! Everything looks like shipshape for reporting to function!"
/>
</Fragment>
),
status: !success ? 'danger' : screenshotStatus,
});
}

if (!success) {
steps.push({
title: i18n.translate('xpack.reporting.listing.diagnosticFailureTitle', {
defaultMessage: "Whoops! Looks like something isn't working properly.",
}),
children: (
<Fragment>
{help.length ? (
<Fragment>
<EuiCallOut color="danger" iconType="alert">
<p>{help.join('\n')}</p>
</EuiCallOut>
</Fragment>
) : null}
{logs.length ? (
<Fragment>
<EuiSpacer />
<FormattedMessage
id="xpack.reporting.listing.diagnosticFailureDescription"
defaultMessage="Here are some more details about the issue:"
/>
<EuiSpacer />
<EuiCodeBlock>{logs}</EuiCodeBlock>
</Fragment>
) : null}
</Fragment>
),
status: 'danger',
});
}

let flyout;
if (isFlyoutVisible) {
flyout = (
<EuiFlyout onClose={closeFlyout} aria-labelledby="reportingHelperTitle" size="m">
<EuiFlyoutHeader hasBorder>
<EuiTitle size="m">
<h2>
<FormattedMessage
id="xpack.reporting.listing.diagnosticTitle"
defaultMessage="Reporting Diagnostics"
/>
</h2>
</EuiTitle>
<EuiSpacer size="s" />
<EuiText color="subdued">
<FormattedMessage
id="xpack.reporting.listing.diagnosticDescription"
defaultMessage="Automatically run a series of diagnostics to troubleshoot common reporting problems."
/>
</EuiText>
</EuiFlyoutHeader>
<EuiFlyoutBody>
<EuiSteps steps={steps} />
</EuiFlyoutBody>
</EuiFlyout>
);
}
return (
<div>
{flyout}
<EuiButtonEmpty size="xs" flush="left" onClick={showFlyout}>
<FormattedMessage
id="xpack.reporting.listing.diagnosticButton"
defaultMessage="Run reporting diagnostics..."
/>
</EuiButtonEmpty>
</div>
);
};
52 changes: 35 additions & 17 deletions x-pack/plugins/reporting/public/components/report_listing.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@

import {
EuiBasicTable,
EuiFlexItem,
EuiFlexGroup,
EuiPageContent,
EuiSpacer,
EuiText,
Expand All @@ -31,6 +33,7 @@ import {
ReportErrorButton,
ReportInfoButton,
} from './buttons';
import { ReportDiagnostic } from './report_diagnostic';

export interface Job {
id: string;
Expand Down Expand Up @@ -134,23 +137,38 @@ class ReportListingUi extends Component<Props, State> {

public render() {
return (
<EuiPageContent horizontalPosition="center" className="euiPageBody--restrictWidth-default">
<EuiTitle>
<h1>
<FormattedMessage id="xpack.reporting.listing.reportstitle" defaultMessage="Reports" />
</h1>
</EuiTitle>
<EuiText color="subdued" size="s">
<p>
<FormattedMessage
id="xpack.reporting.listing.reports.subtitle"
defaultMessage="Find reports generated in Kibana applications here"
/>
</p>
</EuiText>
<EuiSpacer />
{this.renderTable()}
</EuiPageContent>
<div>
<EuiPageContent horizontalPosition="center" className="euiPageBody--restrictWidth-default">
<EuiFlexGroup justifyContent="spaceBetween">
<EuiFlexItem grow={false}>
<EuiTitle>
<h1>
<FormattedMessage
id="xpack.reporting.listing.reportstitle"
defaultMessage="Reports"
/>
</h1>
</EuiTitle>
<EuiText color="subdued" size="s">
<p>
<FormattedMessage
id="xpack.reporting.listing.reports.subtitle"
defaultMessage="Find reports generated in Kibana applications here"
/>
</p>
</EuiText>
</EuiFlexItem>
</EuiFlexGroup>
<EuiSpacer />
{this.renderTable()}
</EuiPageContent>
<EuiSpacer size="s" />
<EuiFlexGroup justifyContent="spaceBetween" direction="rowReverse">
<EuiFlexItem grow={false}>
<ReportDiagnostic apiClient={this.props.apiClient} />
</EuiFlexItem>
</EuiFlexGroup>
</div>
);
}

Expand Down
Loading