Skip to content

Introduce --operation switch and inspect mode#8067

Merged
mitchdenny merged 7 commits into
mainfrom
mitchdenny/inspect-mode-take-2
Mar 14, 2025
Merged

Introduce --operation switch and inspect mode#8067
mitchdenny merged 7 commits into
mainfrom
mitchdenny/inspect-mode-take-2

Conversation

@mitchdenny
Copy link
Copy Markdown
Member

@mitchdenny mitchdenny commented Mar 13, 2025

Replaces this PR: #8033

Introducing a --operation switch which takes the options run, publish, and inspect. Introduced tests to insure backwards compatability. In inspect mode the apphost sits idle so that the launcher can make RPC requests to interrogate the application model after it has been (partially) evaluated.

Copilot AI review requested due to automatic review settings March 13, 2025 23:42
@mitchdenny mitchdenny self-assigned this Mar 13, 2025
@github-actions github-actions Bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Mar 13, 2025
@mitchdenny mitchdenny requested review from davidfowl and karolz-ms and removed request for Copilot March 13, 2025 23:42
@mitchdenny mitchdenny added area-cli NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) labels Mar 13, 2025
@mitchdenny mitchdenny added this to the 9.2 milestone Mar 13, 2025
@mitchdenny mitchdenny marked this pull request as draft March 13, 2025 23:43
Comment thread src/Aspire.Hosting/DistributedApplicationBuilder.cs Outdated
Comment thread tests/Aspire.Hosting.Tests/WithHttpCommandTests.cs
@mitchdenny mitchdenny marked this pull request as ready for review March 14, 2025 02:26
@mitchdenny mitchdenny requested a review from karolz-ms March 14, 2025 02:26
@mitchdenny mitchdenny merged commit 543a7da into main Mar 14, 2025
@mitchdenny mitchdenny deleted the mitchdenny/inspect-mode-take-2 branch March 14, 2025 03:44
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication area-cli NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants