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

[Monitoring] Skip elasticsearch/node_detail.js api integration tests for cloud #30708

Closed
chrisronline opened this issue Feb 11, 2019 · 1 comment · Fixed by #30879
Closed
Labels
Team:Monitoring Stack Monitoring team

Comments

@chrisronline
Copy link
Contributor

chrisronline commented Feb 11, 2019

Relates to #29966

Tests

This test needs to be skipped for cloud.

Error

The error we get when running on cloud:

           │       + expected - actual
           │ 
           │                "bucket_size": "10 seconds"
           │                "data": []
           │                "metric": {
           │                  "app": "elasticsearch"
           │       -          "description": "CPU Usage time compared to the CPU quota shown in percentage. If CPU quotas are not set, then no data will be shown."
           │       +          "description": "Percentage of CPU usage for the Elasticsearch process."
           │                  "field": "node_stats.process.cpu.percent"
           │                  "format": "0,0.[00]"
           │       -          "hasCalculation": true
           │       -          "isDerivative": true
           │       +          "hasCalculation": false
           │       +          "isDerivative": false

Reason

Cloud runs in docker containers and we have logic that forks based on a specific config setting (xpack.monitoring.ui.container.elasticsearch.enabled). The test response is hard-coded to assume this isn't set so the tests will never pass running against a Kibana server with this config set.

@chrisronline chrisronline added the Team:Monitoring Stack Monitoring team label Feb 11, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Monitoring Stack Monitoring team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants