-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[APM] Reinstate ML multi-metric job #117836
[APM] Reinstate ML multi-metric job #117836
Conversation
Pinging @elastic/apm-ui (Team:apm) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly nits
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/manifest.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/manifest.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
x-pack/plugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/apm_metrics.json
Outdated
Show resolved
Hide resolved
"field" : "@timestamp" | ||
} | ||
}, | ||
"transactions_per_min" : { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need changing to whatever is chosen for the field_name
in the job config.
...lugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/datafeed_apm_metrics.json
Outdated
Show resolved
Hide resolved
...lugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/datafeed_apm_metrics.json
Outdated
Show resolved
Hide resolved
...lugins/ml/server/models/data_recognizer/modules/apm_transaction/ml/datafeed_apm_metrics.json
Outdated
Show resolved
Hide resolved
{ exists: { field: TRANSACTION_DURATION } }, | ||
{ term: { [PROCESSOR_EVENT]: ProcessorEvent.metric } }, | ||
{ term: { [METRICSET_NAME]: 'transaction' } }, | ||
{ terms: { _tier: ['content', 'hot', 'warm'] } }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When the jobs are started from within the APM UI, what start time is used for the job? Unless you are running them over the full time range of data available, this _tier
filter isn't needed. Assuming the start date is close to now
, the time range part of the datafeed query will ensure the cold / frozen tiers won't be searched.
{ "term": { "processor.event": "transaction" } }, | ||
{ "exists": { "field": "transaction.duration" } } | ||
{ "term": { "processor.event": "metric" } }, | ||
{ "term": { "metricset.name": "transaction" } } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The manifest query will need to be
"query": {
"bool": {
"filter": [
{ "term": { "processor.event": "metric" } },
{ "term": { "metricset.name": "transaction" } }
],
"must_not": { "terms": { "_tier": [ "data_frozen", "data_cold" ] } }
}
},
but as discussed, feel free to leave the _tier
part out of the changes in this PR and I can include them when editing all the other ML modules as part of #116696.
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested creating the job from the ML and APM UIs and LGTM
Closes elastic#101734. This reverts commit 008421f. Additionally, incorporate suggested changes from elastic#101734 (comment).
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Closes elastic#101734. This reverts commit 008421f. Additionally, incorporate suggested changes from elastic#101734 (comment).
Closes elastic#101734. This reverts commit 008421f. Additionally, incorporate suggested changes from elastic#101734 (comment).
Closes elastic#101734. This reverts commit 008421f. Additionally, incorporate suggested changes from elastic#101734 (comment).
Closes #101734.
This reverts commit 008421f.
Additionally, incorporate suggested changes from #101734 (comment).