Skip to content

Conversation

@MattKotsenas
Copy link
Member

Description

Kusto has two "Kusto Explorer" apps for viewing and querying data:

  • One is a Windows-only ClickOnce app
  • One is a web-based client

Both are popular with different audiences. This change adds both to the actions list for the cluster resource. I can also add the commands to database resources, but I thought that might be cluttering for most users.

Contributes to #8233

Checklist

  • Is this feature complete?
    • Yes. Ready to ship.
    • No. Follow-up changes expected.
  • Are you including unit tests for the changes and scenario tests if relevant?
    • Yes
    • No
  • Did you add public API?
    • Yes
      • If yes, did you have an API Review for it?
        • Yes
        • No
      • Did you add <remarks /> and <code /> elements on your triple slash comments?
        • Yes
        • No
    • No
  • Does the change make any security assumptions or guarantees?
    • Yes
      • If yes, have you done a threat model and had a security review?
        • Yes
        • No
    • No
  • Does the change require an update in our Aspire docs?

Copilot AI review requested due to automatic review settings November 2, 2025 07:32
@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2025

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12609

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 12609"

@MattKotsenas
Copy link
Member Author

I'm not sure how best to test this scenario (if required) so feedback welcome in that area.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Nov 2, 2025
@MattKotsenas
Copy link
Member Author

Here's a snapshot from the Playground app

image

@MattKotsenas MattKotsenas changed the title Add custom commands to open Kusto servers in Azure Data Explorer Add custom commands to open Kusto servers in Kusto Explorer Nov 2, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds custom commands to Azure Kusto (Azure Data Explorer) resources that allow users to open the Kusto Explorer tools directly from the Aspire dashboard. The feature provides two commands: one to launch the desktop Kusto Explorer (Windows-only) and another to launch the web-based Kusto Explorer.

  • Adds two new resource commands for opening Kusto Explorer (Desktop and Web versions)
  • Implements platform-specific command visibility (Desktop command hidden on non-Windows platforms)
  • Adds fallback interaction dialog when web explorer fails to launch automatically

@davidfowl
Copy link
Member

Why is it a command vs being an additional URL?

@MattKotsenas
Copy link
Member Author

I went this route for two reasons:

  1. It uses the "official" tool launcher that Kusto provides, and was suggested by the Kusto team
  2. It bypasses any potential port shenanigans; if you do any kind of port forwarding for the local scenario, this way will definitely see the client-side port and use it correctly. (I've not personally run into this myself)

Let me know what you think or what your concerns are and I'm happy to adjust.

@MattKotsenas
Copy link
Member Author

Also to provide more context:

For deployed clusters, you currently can click on the URL and you'll get redirected to the web explorer. So I could just add the desktop app as a command. Adding both was more for discovery and parity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants