Skip to content

[Monitoring/ML] add number of forecasts to monitoring overview#20758

Merged
hendrikmuhs merged 1 commit intoelastic:masterfrom
hendrikmuhs:forecast-monitoring
Jul 17, 2018
Merged

[Monitoring/ML] add number of forecasts to monitoring overview#20758
hendrikmuhs merged 1 commit intoelastic:masterfrom
hendrikmuhs:forecast-monitoring

Conversation

@hendrikmuhs
Copy link
Copy Markdown

Add forecasts (Number of forecasts for an ml jobs) as column to the ML Jobs monitoring page. Follow up of elastic/elasticsearch#31647 which added the necessary backend parts.

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@hendrikmuhs
Copy link
Copy Markdown
Author

@tsullivan @peteharverson Would you be able to review?

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@tsullivan
Copy link
Copy Markdown
Member

Hi, can you look into the CI failure?

jenkins test this

@tsullivan tsullivan self-requested a review July 16, 2018 18:28
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hendrikmuhs hendrikmuhs merged commit a0a22cf into elastic:master Jul 17, 2018
hendrikmuhs pushed a commit to hendrikmuhs/kibana that referenced this pull request Jul 17, 2018
…ic#20758)

Add forecasts (Number of forecasts for an ml jobs) as column to the ML Jobs monitoring page. Related to: elastic/elasticsearch#31647
hendrikmuhs pushed a commit that referenced this pull request Jul 18, 2018
… (#20889)

Add forecasts (Number of forecasts for an ml jobs) as column to the ML Jobs monitoring page. Related to: elastic/elasticsearch#31647
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.

4 participants