elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.62k stars 8.22k forks source link

[Remote Clusters] Error Callout Has Wrong Icon and Color #110155

Closed cuff-links closed 2 weeks ago

cuff-links commented 3 years ago

Summary

Environment

Kibana: 7.15.0 BC1 Elasticsearch 7.15.0 BC1 Browsers: All Browsers

Steps to reproduce:

1.) Navigate to Remote Clusters 2.) Create a new Remote Cluster 3.) Add information for a cluster except set the max number of connections to 999999999999999999 4.) Try to save.

Expected Result

The callout that shows the parsing error should be red with an alert icon.

Actual Result

The callout is yellow with an x icon. (See Image)

Screen Shot 2021-08-25 at 3 47 23 PM
elasticmachine commented 3 years ago

Pinging @elastic/kibana-stack-management (Team:Stack Management)

elasticmachine commented 1 month ago

Pinging @elastic/kibana-management (Team:Kibana Management)

alisonelizabeth commented 1 month ago

This is reported on an older version, so we will need to triage if still valid. In this case, we should be using the danger callout. See: https://eui.elastic.co/#/display/callout#danger

SoniaSanzV commented 1 month ago

I checked and this still happens. I'll fix it.

I'm wondering if we should also add a Max node connections validation when the number is introduced (not sure if we have that info or that complete depends on the backend). Probable will be better to open another issue if we decide to add the validation

mattkime commented 4 weeks ago

@SoniaSanzV I agree that validation should be added as the error message in the screen shot isn't very helpful.