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

Adding xpack code for ES ML job metricset #8129

Merged

Conversation

ycombinator
Copy link
Contributor

This PR teaches the elasticsearch/ml_job metricset to index job_stats documents into .monitoring-es-6-mb-* indices. These documents should be compatible in structure to job_stats documents in the current .monitoring-es-6-* indices indexed via the internal monitoring method.

@ycombinator ycombinator added review Metricbeat Metricbeat needs_backport PR is waiting to be backported to other branches. v7.0.0-alpha1 monitoring v6.5.0 labels Aug 28, 2018
// specific language governing permissions and limitations
// under the License.

package ml_job

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

@ycombinator ycombinator merged commit bfd2843 into elastic:master Aug 29, 2018
@ycombinator ycombinator added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Aug 29, 2018
@ycombinator ycombinator deleted the metricbeat/elasticsearch/ml-job/x-pack branch August 29, 2018 15:02
ycombinator added a commit to ycombinator/beats that referenced this pull request Sep 7, 2018
* Adding job_stats xpack monitoring documents

* Fixing type assertions

(cherry picked from commit bfd2843)
ycombinator added a commit that referenced this pull request Sep 11, 2018
* Adding job_stats xpack monitoring documents

* Fixing type assertions

(cherry picked from commit bfd2843)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants