-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Metricbeat] Add FailOnRequired when applying schema #14143
Conversation
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.
LGTM. Glad to see the optional Apply
types getting used, there's probably lots of other places where they could be useful.
Yeah I see in mongodb module, we are not using |
Thanks for the fix. Is it released in metricbeat 7.4.1? |
@digz6666 I don't think this is in 7.4.1 unfortunately. But it will definitely be in 7.5 release in near future! |
* Add FailOnRequired when applying schema
…n applying schema (elastic#14202) * [Metricbeat] Add FailOnRequired when applying schema (elastic#14143) * Add FailOnRequired when applying schema (cherry picked from commit 972900f)
Original problem reported is when running metricbeat mongodb module,
metrics
metricset failled with:I was able to reproduce it and turned out this error message changes with different keys not found when I reproduced it.
This error should be fixed by adding
schema.FailOnRequired
when applying the schema.Second problem is after adding
schema.FailOnRequired
, a warning showed up when trying to runmetrics
metricset:I believe some metrics in
metrics
metricset are missing.count
suffix, likemongodb.metrics.query_executor.scanned_indexes
.How to test this:
Please start mongodb locally and start Metricbeat with mongodb module enabled. Make sure metrics are flowing with no error above. You might see some error/warning message at INFO level indicating permission error but that's not related to this.