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.

Sign up for GitHub

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

Jump to bottom

[Infra] Processes tab shows toast with a 500 error when performing a full page refresh #187385

Closed
crespocarlos opened this issue Jul 2, 2024 · 1 comment · Fixed by #187459
Closed

[Infra] Processes tab shows toast with a 500 error when performing a full page refresh #187385

crespocarlos opened this issue Jul 2, 2024 · 1 comment · Fixed by #187459
Assignees
@jennypavlova
Labels
bug Fixes for quality problems that affect the customer experience Feature:ObsHosts Hosts feature within Observability Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@crespocarlos
Copy link
Contributor

Summary

/process_list endpoint returns a 500 when performing a full page refresh.

500_processes.mov

When the page is loading, 2 calls happen. The first returns a 500

The error is caused by a missing index_pattern attribute, which is missing at the time of the first call because the data view is still loading and can't provide the index_pattern information.

Suggestion

Instead of passing the index pattern in the request, the endpoint should be capable of getting in the server, similar to what metadata endpoint does https://github.com/elastic/kibana/blob/main/x-pack/plugins/observability_solution/infra/server/routes/metadata/lib/get_metric_metadata.ts

Acceptance criteria

  • /process_list should return a 400 in case mandatory parameters are missing
  • /process_list should be able to determine the index pattern to be used.
@crespocarlos crespocarlos added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Jul 2, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@crespocarlos crespocarlos added bug Fixes for quality problems that affect the customer experience Feature:ObsHosts Hosts feature within Observability labels Jul 2, 2024
@crespocarlos crespocarlos changed the title [Infra] Processes tab crashes when performing a full page refresh [Infra] Processes tab shows toast with a 500 error when performing a full page refresh Jul 2, 2024
@jennypavlova jennypavlova self-assigned this Jul 2, 2024
@jennypavlova jennypavlova mentioned this issue Jul 3, 2024
Merged
jennypavlova added a commit that referenced this issue Jul 4, 2024
@jennypavlova
[Infra] Fix: Processes tab shows toast with a 500 error when performi…
f0edaa1 
…ng a full page refresh (#187459)

Closes #187385 
## Summary

This PR fixes the issue with the processes tab showing a 500 error in a
toast when performing a full page refresh.

## Testing
Go to asset details and open the processes tab. Refresh the page: there
should not be any error visible.


https://github.com/elastic/kibana/assets/14139027/8998d141-8841-4138-8c76-7f555f6c44f0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jennypavlova jennypavlova

Labels
bug Fixes for quality problems that affect the customer experience Feature:ObsHosts Hosts feature within Observability Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
3 participants
@crespocarlos @jennypavlova @elasticmachine