-
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
Cherry-pick #8106 to 6.x: Adding xpack code for ES index recovery metricset #8137
Conversation
// specific language governing permissions and limitations | ||
// under the License. | ||
|
||
package index_recovery |
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.
don't use an underscore in package name
Just saw there is no changelog either in this PR or the original one, is that intended? |
Yeah, it's intentional because we don't (yet) want to document the process of using Metricbeat to perform X-Pack monitoring. We'll do that once we have X-Pack monitoring via Metricbeat working for all Elastic products that are monitored today. Otherwise users can end up with a messy setup with only some products allowed to be monitored via Metricbeat and others still needing to do internal monitoring. |
jenkins test this |
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. Failing windows test is not related. @kvch is looking into it.
@ruflin Thanks for noticing that. Just to double check: my plan is to wait till the 6.x branch CI is green again, then rebase this PR against 6.x so (hopefully) it goes green too. Correct? |
@ycombinator Both options are ok for me. Either rebase when it's green or also now as the failing tests are not related. |
9e7ca1d
to
74e6a9a
Compare
* Adding index_recovery xpack monitoring documents * Get all recoveries if xpack.enabled is true * Removing newline (which also reordered imports) * Don't reuse value so much (cherry picked from commit f6b3e78)
74e6a9a
to
8852789
Compare
The only failure I'm seeing in CI for this PR is in the windows libbeat build:
This is the same failure that is currently occurring on CI for the target branch of this PR, 6.x (e.g. https://beats-ci.elastic.co/job/elastic+beats+6.x+multijob-windows/25/beat=libbeat,label=windows/console). Also, this failure is unrelated to the code in this PR. So I'm going to merge this PR now... |
Cherry-pick of PR #8106 to 6.x branch. Original message:
This PR teaches the
elasticsearch/index_recovery
metricset to indexindex_recovery
documents into.monitoring-es-6-mb-*
indices. These documents should be compatible in structure toindex_recovery
documents in the current.monitoring-es-6-*
indices indexed via the internal monitoring method.