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

ADX dashboard starter #1234

Merged
merged 3 commits into from
Jan 22, 2025
Merged

ADX dashboard starter #1234

merged 3 commits into from
Jan 22, 2025

Conversation

flanakin
Copy link
Collaborator

@flanakin flanakin commented Jan 7, 2025

πŸ› οΈ Description

Create ADX dashboard that covers the same scenarios as Cost summary, Rate optimization, and Data ingestion reports.

πŸ“· Screenshots

Cost summary page
Rate optimization page
Data ingestion page

πŸ“‹ Checklist

πŸ”¬ How did you test this change?

  • 🀏 Lint tests
  • 🀞 PS -WhatIf / az validate
  • πŸ‘ Manually deployed + verified
  • πŸ’ͺ Unit tests
  • πŸ™Œ Integration tests

πŸ™‹β€β™€οΈ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🀏 The change is less than 20 lines of code.

πŸ“‘ Did you update docs/changelog.md?

  • βœ… Updated changelog (required for dev PRs)
  • ➑️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

πŸ“– Did you update documentation?

  • βœ… Public docs in docs (required for dev)
  • βœ… Internal dev docs in src (required for dev)
  • ➑️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

@flanakin flanakin requested a review from bwatts64 January 8, 2025 19:59
@MSBrett
Copy link
Contributor

MSBrett commented Jan 13, 2025

The dashboards look great, but take a long time to load because of the number of controls on each page. Could we split those pages into multiple smaller ones to improve performance?

@MSBrett MSBrett added Needs: Review πŸ‘€ PR that is ready to be reviewed and removed Needs: Review πŸ‘€ PR that is ready to be reviewed labels Jan 13, 2025
@MSBrett MSBrett merged commit 7913c70 into dev Jan 22, 2025
1 check passed
@MSBrett MSBrett deleted the flanakin/dev/hubs-adxdash branch January 22, 2025 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Review πŸ‘€ PR that is ready to be reviewed Tool: FinOps hubs Data pipeline solution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants