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

[Synthetics UI] Location Selector uses custom-hearbeat-id for navigation #145758

Closed
awahab07 opened this issue Nov 18, 2022 · 1 comment · Fixed by #145825
Closed

[Synthetics UI] Location Selector uses custom-hearbeat-id for navigation #145758

awahab07 opened this issue Nov 18, 2022 · 1 comment · Fixed by #145825
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.6.0

Comments

@awahab07
Copy link
Contributor

awahab07 commented Nov 18, 2022

Kibana version: v8.6.0

Elasticsearch version: v8.6.0

Describe the bug:
On Synthetics -> Monitors -> Monitor Details -> Overview | History pages, the Location Selector uses custom_heartbea_id for project monitors for navigation which breaks the page.

Steps to reproduce:

  1. Setup Kibana and Synthetics so that you are able to push synthetics project monitors.
  2. Enable the new Synthetics App in Kibana -> Advanced Settings and push a monitor with two or more locations. (The dev locations with dev synthetics service, "North America - US Central" and "US Central QA" also work)
  3. Go to Synthetics -> Monitors -> Monitor Details -> Overview page and change the location from the Location Selector in the page header.
  4. Notice that the page will break as the navigated url contains custom_heartbeat_id instead of config_id.

Expected behavior:
config_id should be used instead of custom_heartbeat_id for navigation to Monitor Details -> Overview | History pages.

Screenshots (if relevant):

Location-switcher-uses-heartbeat-id-instead-of-config-id.mov

Errors in browser console (if relevant):

{
  error: "Not Found"
  message: "Monitor id mostly-fail-push-project-08-default not found!"
  statusCode: 404
}

Provide logs and/or server output (if relevant):

Any additional context:
Need to check if it is as a result of this #143309 and is a valid case, if so, the behavior need to be retested with newly pushed/created monitors i.e. Saved Objects may need migration.

@awahab07 awahab07 added bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v8.6.0 labels Nov 18, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@afgomez afgomez self-assigned this Nov 21, 2022
afgomez pushed a commit that referenced this issue Nov 21, 2022
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 21, 2022
kibanamachine added a commit that referenced this issue Nov 21, 2022
…145850)

# Backport

This will backport the following commits from `main` to `8.6`:
- [[Synthethics] Use `monitor.config_id` for the URLs
(#145825)](#145825)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Alejandro Fernández
Gómez","email":"alejandro.fernandez@elastic.co"},"sourceCommit":{"committedDate":"2022-11-21T12:15:50Z","message":"[Synthethics]
Use `monitor.config_id` for the URLs (#145825)\n\n##
Summary\r\n\r\nCloses
#145758","sha":"cc50e51fedd666ee065008be1ced33defa375582","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Team:uptime","release_note:skip","v8.6.0","v8.7.0"],"number":145825,"url":"https://github.com/elastic/kibana/pull/145825","mergeCommit":{"message":"[Synthethics]
Use `monitor.config_id` for the URLs (#145825)\n\n##
Summary\r\n\r\nCloses
#145758","sha":"cc50e51fedd666ee065008be1ced33defa375582"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"8.6","label":"v8.6.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/145825","number":145825,"mergeCommit":{"message":"[Synthethics]
Use `monitor.config_id` for the URLs (#145825)\n\n##
Summary\r\n\r\nCloses
#145758","sha":"cc50e51fedd666ee065008be1ced33defa375582"}}]}]
BACKPORT-->

Co-authored-by: Alejandro Fernández Gómez <alejandro.fernandez@elastic.co>
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 v8.6.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants