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

adding refined app developer dashboard and telemetry piece #526

Closed
wants to merge 3 commits into from
Closed

adding refined app developer dashboard and telemetry piece #526

wants to merge 3 commits into from

Conversation

R-Lawton
Copy link
Contributor

@R-Lawton R-Lawton commented Apr 9, 2024 •
edited
Loading

Refined the pre-existing app developer dashboard
What's new:

  • Split the requests from one singular table and panels for both error requests and successful request into there own sections
  • Added in tooltips for all panels
  • Moved the location of panels so errors show at the top followed by success followed by API summary
  • Added variables for API name and API endpoint path selection

Verification

  1. To populate the panels run the single cluster getting started guide https://docs.kuadrant.io/getting-started-single-cluster/

  2. Then run the commands from the observability README https://github.com/Kuadrant/kuadrant-operator/tree/main/config/observability#readme
    You will need to add the label kubectl label httproute toystore -n default deployment=toystore

Screenshot 2024-04-09 at 17 24 25
Screenshot 2024-04-09 at 17 24 34

@R-Lawton R-Lawton requested a review from a team as a code owner April 9, 2024 16:14
@david-martin david-martin self-requested a review April 9, 2024 16:40
@R-Lawton
Copy link
Contributor Author

R-Lawton commented Apr 11, 2024 •
edited
Loading

Closing in favour of @david-martin api repeating row version

@R-Lawton R-Lawton closed this Apr 11, 2024
@david-martin david-martin mentioned this pull request Apr 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@david-martin david-martin Awaiting requested review from david-martin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@R-Lawton @david-martin