[WIP] Extend the Aspire Dashboard to reflect AppHost topology and publish an Azure Monitor Health Model - #20149
Open
James Gould (james-gould) wants to merge 30 commits into
Open
James Gould (james-gould) wants to merge 30 commits into
James Gould (james-gould) wants to merge 30 commits into
Conversation
…re still kinda wonky though
merge conflicts
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 20149Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 20149" |
Contributor
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
Contributor
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
Contributor
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Currently WIP, please dogfood with the Aspire CLI and provide feedback if possible
Enhances the
Graphview of the dashboard to reflect the AppHost's topology, reflecting the health state which propagates through the notes up to theAppHostroot node.Experimental, unstable changes
When
aspire publishis executed, the health checks against the provisioned resource types are converted intoSignal Definitionswithin an Azure Monitor health model. The intent is to allow some degree of customisation around the signal definitions created by default, along with disabling the provisioning of a health model altogether.This PR is opened purely to expose the changes via the
aspireCLI for dogfooding, feedback and easier testing. Some notes and feedback around how the topology is derived from the Aspire Discord may be integrated here or in a precursor PR:Screenshots
Checklist
<remarks />and<code />elements on your triple slash comments?