[APM] Improve open transaction/span/error in Discover (#125431)

This commit is contained in:
Tobias Stadler 2022-02-16 18:03:29 +01:00 committed by GitHub
parent 82c21484b3
commit d68a0d390a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
9 changed files with 24 additions and 34 deletions

View file

@ -151,7 +151,7 @@ Learn more about a trace sample in the *Metadata* tab:
* User - Requires additional configuration, but allows you to see which user experienced the current transaction. * User - Requires additional configuration, but allows you to see which user experienced the current transaction.
TIP: All of this data is stored in documents in Elasticsearch. TIP: All of this data is stored in documents in Elasticsearch.
This means you can select "Actions - View sample document" to see the actual Elasticsearch document under the discover tab. This means you can select "Actions - View transaction in Discover" to see the actual Elasticsearch document under the discover tab.
*Trace sample logs* *Trace sample logs*

View file

@ -34,11 +34,7 @@ exports[`DetailView should render Discover button 1`] = `
} }
kuery="" kuery=""
> >
<EuiButtonEmpty
iconType="discoverApp"
>
View 10 occurrences in Discover. View 10 occurrences in Discover.
</EuiButtonEmpty>
</DiscoverErrorLink> </DiscoverErrorLink>
`; `;

View file

@ -6,7 +6,6 @@
*/ */
import { import {
EuiButtonEmpty,
EuiIcon, EuiIcon,
EuiPanel, EuiPanel,
EuiSpacer, EuiSpacer,
@ -100,7 +99,6 @@ export function DetailView({ errorGroup, urlParams, kuery }: Props) {
</h3> </h3>
</EuiTitle> </EuiTitle>
<DiscoverErrorLink error={error} kuery={kuery}> <DiscoverErrorLink error={error} kuery={kuery}>
<EuiButtonEmpty iconType="discoverApp">
{i18n.translate( {i18n.translate(
'xpack.apm.errorGroupDetails.viewOccurrencesInDiscoverButtonLabel', 'xpack.apm.errorGroupDetails.viewOccurrencesInDiscoverButtonLabel',
{ {
@ -109,7 +107,6 @@ export function DetailView({ errorGroup, urlParams, kuery }: Props) {
values: { occurrencesCount }, values: { occurrencesCount },
} }
)} )}
</EuiButtonEmpty>
</DiscoverErrorLink> </DiscoverErrorLink>
</HeaderContainer> </HeaderContainer>

View file

@ -7,7 +7,6 @@
import { import {
EuiBadge, EuiBadge,
EuiButtonEmpty,
EuiCallOut, EuiCallOut,
EuiFlexGroup, EuiFlexGroup,
EuiFlexItem, EuiFlexItem,
@ -126,14 +125,12 @@ export function SpanFlyout({
</EuiFlexItem> </EuiFlexItem>
<EuiFlexItem grow={false}> <EuiFlexItem grow={false}>
<DiscoverSpanLink span={span}> <DiscoverSpanLink span={span}>
<EuiButtonEmpty iconType="discoverApp">
{i18n.translate( {i18n.translate(
'xpack.apm.transactionDetails.spanFlyout.viewSpanInDiscoverButtonLabel', 'xpack.apm.transactionDetails.spanFlyout.viewSpanInDiscoverButtonLabel',
{ {
defaultMessage: 'View span in Discover', defaultMessage: 'View span in Discover',
} }
)} )}
</EuiButtonEmpty>
</DiscoverSpanLink> </DiscoverSpanLink>
</EuiFlexItem> </EuiFlexItem>
</EuiFlexGroup> </EuiFlexGroup>

View file

@ -67,7 +67,7 @@ describe('Transaction action menu', () => {
actions: [ actions: [
{ {
key: 'sampleDocument', key: 'sampleDocument',
label: 'View sample document', label: 'View transaction in Discover',
href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))', href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))',
condition: true, condition: true,
}, },
@ -134,7 +134,7 @@ describe('Transaction action menu', () => {
actions: [ actions: [
{ {
key: 'sampleDocument', key: 'sampleDocument',
label: 'View sample document', label: 'View transaction in Discover',
href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))', href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))',
condition: true, condition: true,
}, },
@ -200,7 +200,7 @@ describe('Transaction action menu', () => {
actions: [ actions: [
{ {
key: 'sampleDocument', key: 'sampleDocument',
label: 'View sample document', label: 'View transaction in Discover',
href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))', href: 'some-basepath/app/discover#/?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-24h,to:now))&_a=(index:apm_static_index_pattern_id,interval:auto,query:(language:kuery,query:\'processor.event:"transaction" AND transaction.id:"123" AND trace.id:"123"\'))',
condition: true, condition: true,
}, },

View file

@ -191,7 +191,7 @@ export const getSections = ({
label: i18n.translate( label: i18n.translate(
'xpack.apm.transactionActionMenu.viewSampleDocumentLinkLabel', 'xpack.apm.transactionActionMenu.viewSampleDocumentLinkLabel',
{ {
defaultMessage: 'View sample document', defaultMessage: 'View transaction in Discover',
} }
), ),
href: getDiscoverHref({ href: getDiscoverHref({

View file

@ -74,7 +74,7 @@ describe('TransactionActionMenu component', () => {
Transactions.transactionWithMinimalData Transactions.transactionWithMinimalData
); );
expect(queryByText('View sample document')).not.toBeNull(); expect(queryByText('View transaction in Discover')).not.toBeNull();
}); });
it('always renders the trace logs link', async () => { it('always renders the trace logs link', async () => {

View file

@ -35,8 +35,8 @@ describe('SeriesActions', function () {
expect(screen.getByLabelText('Series actions list')).toBeVisible(); expect(screen.getByLabelText('Series actions list')).toBeVisible();
}); });
it('should display a view sample link', function () { it('should display a view transaction link', function () {
expect(screen.getByLabelText('View sample documents')).toBeVisible(); expect(screen.getByLabelText('View transaction in Discover')).toBeVisible();
}); });
it('should display a hide series link', function () { it('should display a hide series link', function () {

View file

@ -182,7 +182,7 @@ const DELETE_SERIES_TOOLTIP_LABEL = i18n.translate(
const VIEW_SAMPLE_DOCUMENTS_LABEL = i18n.translate( const VIEW_SAMPLE_DOCUMENTS_LABEL = i18n.translate(
'xpack.observability.seriesEditor.sampleDocuments', 'xpack.observability.seriesEditor.sampleDocuments',
{ {
defaultMessage: 'View sample documents', defaultMessage: 'View transaction in Discover',
} }
); );