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

[Uptime] Transition link to Logs, Infra UI contains host.ip:null if monitor.status:down #48776

Closed
v01t opened this issue Oct 21, 2019 · 9 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.6.0

Comments

@v01t
Copy link

v01t commented Oct 21, 2019

Kibana version: 7.4.0

Elasticsearch version: 7.4.0

Server OS version: Centos 7

Browser version: Firefox 69.0.3

Browser OS version: Debian 10.1

Original install method (e.g. download page, yum, from source, etc.): Docker

Describe the bug:
Transition links from Uptime UI to Logs/Infra UI's contains host.ip:null instead of host.ip:<ip_value> and looks like:

/app/infra#/infrastructure/inventory?waffleFilter=(expression:'host.ip : null',kind:kuery)
/app/infra#/logs?logFilter=(expression:'host.ip : null',kind:kuery)

when last monitor.status: down

Steps to reproduce:

  1. Find monitor with current status:down (monitor should be shown as "Down", also specific time frame can be used to get the same from recovered monitors)
  2. check the links for "Show host metrics", "Show host logs" (probably others too)

Expected behavior:
Transition links suppose to contain value of host.ip in the url as its happen when monitor.status:up

@Bargs Bargs added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Oct 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@weltenwort weltenwort added Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Oct 22, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@shahzad31
Copy link
Contributor

i tried reproducing it, but i am unable to in 7.4 as well in master. For me it always contains ip even if monitor is down.

@andrewvc andrewvc added the bug Fixes for quality problems that affect the customer experience label Oct 22, 2019
@andrewvc andrewvc changed the title [UI Uptime] Transition link to Logs, Infra UI contains host.ip:null if monitor.status:down [Uptime] Transition link to Logs, Infra UI contains host.ip:null if monitor.status:down Oct 22, 2019
@v01t
Copy link
Author

v01t commented Oct 22, 2019

additionally I verified monitor records to ensure that host.ip are present in source object, field type in heartbeat and filebeat/metricbeat indices for host.ip = ip.

here is screenshot (see hover-link destination on the bottom)
Screenshot_20191022_225630

Please let me know if you will need any other additional information that may help resolve the issue I would be happy to provide it.

@andrewvc
Copy link
Contributor

@v01t when you expand the rows on the page do you see any IPs listed? That would be a key clue here.

@v01t
Copy link
Author

v01t commented Oct 30, 2019

@andrewvc no I do not see IP listed in expanded row for check with current status "down":
Screenshot_20191030_133117
if I select time frame with end-timestamp that equals to last registered "down" status for the check 1 from the screenshot (to have both of them with status "down") IP disappears for both of them.

Thanks for your reply and help.

@andrewvc
Copy link
Contributor

andrewvc commented Dec 3, 2019

I believe this is fixed in #50138 .

@andrewvc
Copy link
Contributor

andrewvc commented Dec 3, 2019

Closing this issue, if it persists post 7.6.0 please let us know.

Thanks for the detailed writeup!

@andrewvc
Copy link
Contributor

andrewvc commented Dec 3, 2019

Just to close the loop here, the issue seems to be that there was some issue where no IP was present. This can happen if the DNS lookup fails for instance.

@andrewvc andrewvc removed their assignment Jan 16, 2020
@andrewvc andrewvc self-assigned this Jan 24, 2020
@andrewvc andrewvc added the test-plan-ok issue has passed test plan label Jan 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.6.0
Projects
None yet
Development

No branches or pull requests

8 participants