Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EEM] Undefined failedTransactionRate metric from service entity definition #188765

Closed
kpatticha opened this issue Jul 19, 2024 · 2 comments · Fixed by #189065
Closed

[EEM] Undefined failedTransactionRate metric from service entity definition #188765

kpatticha opened this issue Jul 19, 2024 · 2 comments · Fixed by #189065
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:obs-knowledge Observability Experience Knowledge team

Comments

@kpatticha
Copy link
Contributor

Issue

name: 'failedTransactionRate',
equation: 'A / B',
metrics: [
{
name: 'A',
aggregation: 'doc_count',
filter: serviceTransactionFilter(['event.outcome: "failure"']),
},
{
name: 'B',
aggregation: 'doc_count',
filter: serviceTransactionFilter(['event.outcome: *']),
},
],

The service entity definition is using event.outcome for calculating failedTransactionRate which doesn't exist in the service transaction metric.

The event.success_count should be used instead.

https://github.com/elastic/apm-server/blob/main/docs/data/elasticsearch/service_transaction_metric.json#L18-L21

related

@botelastic botelastic bot added the needs-team Issues missing a team label label Jul 19, 2024
@kpatticha kpatticha added bug Fixes for quality problems that affect the customer experience Team:obs-knowledge Observability Experience Knowledge team labels Jul 19, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-knowledge-team (Team:obs-knowledge)

@klacabane
Copy link
Contributor

ok, the new metric should be

      {
        name: 'failedTransactionRate',
        equation: '1 - (A / B)',
        metrics: [
          {
            name: 'A',
            aggregation: 'sum',
            filter: serviceTransactionFilter(),
            field: 'event.success_count',
          },
          {
            name: 'B',
            aggregation: 'value_count',
            filter: serviceTransactionFilter(),
            field: 'event.success_count',
          },
        ],
      },

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:obs-knowledge Observability Experience Knowledge team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants