-
Notifications
You must be signed in to change notification settings - Fork 168
Document deployment state caching #5301
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
Document deployment state caching #5301
Conversation
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
IEvangelist
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great start, @copilot please address all feedback.
| The `aspire deploy` command manages deployment state through cached configuration files stored locally on your machine. This caching mechanism streamlines repeated deployments by preserving provisioning settings and parameters, making subsequent deployments faster and more efficient. | ||
|
|
||
| ## Default behavior | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add an introductory sentence, or short paragraph to describe this section.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added introductory paragraph to the "Default behavior" section in commit 4ed162e.
|
|
||
| ### First deployment | ||
|
|
||
| When you run `aspire deploy` for the first time, the command: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the first time, or for the first time in a target --environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to clarify "for the first time, or for the first time in a target --environment" in commit 4ed162e.
|
|
||
| On subsequent `aspire deploy` executions, the command: | ||
|
|
||
| 1. Detects the existing deployment state file at `~/.aspire/deployments/{AppHostSha}/production.json`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
References to files should be italicized, not code blocks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed file references to italics throughout the document in commit 4ed162e.
| 1. Proceeds with deployment using the cached values (no re-prompting). | ||
|
|
||
| ## Environment-specific deployments | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, please add an intro paragraph here. Perhaps use this opportunity to explain why environment-specific deploys are important.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added introductory paragraph explaining the importance of environment-specific deployments in commit 4ed162e.
| This behaves identically to using the `--environment` flag, loading the appropriate cached configuration file. | ||
|
|
||
| ## Cache management | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a brief introduction here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added brief introduction to the "Cache management" section in commit 4ed162e.
| - **Path pattern:** `~/.aspire/deployments/{AppHostSha}/{environment}.json` | ||
| - **Default environment:** `production` | ||
| - **AppHostSha:** A hash value representing the application host configuration, ensuring deployment states are specific to each application configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Italic for files and paths. Also, need proper punctuation on all bullet lists.
| - **Path pattern:** `~/.aspire/deployments/{AppHostSha}/{environment}.json` | |
| - **Default environment:** `production` | |
| - **AppHostSha:** A hash value representing the application host configuration, ensuring deployment states are specific to each application configuration. | |
| - **Path pattern:** _~/.aspire/deployments/{AppHostSha}/{environment}.json_. | |
| - **Default environment:** `production`. | |
| - **AppHostSha:** A hash value representing the application host configuration, ensuring deployment states are specific to each application configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Applied formatting changes: italicized file paths and added proper punctuation to all bullet lists in commit 4ed162e.
| - **Path pattern:** `~/.aspire/deployments/{AppHostSha}/{environment}.json` | ||
| - **Default environment:** `production` | ||
| - **AppHostSha:** A hash value representing the application host configuration, ensuring deployment states are specific to each application configuration. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be really nice to include a simple GitHub example, that conceptually describes and shows how to persist the local state using @actions/cache or something like that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a complete "Use deployment state in CI/CD pipelines" section with a GitHub Actions example showing how to cache deployment state using actions/cache in commit 4ed162e. The example includes cache key strategies and a security caution about managing sensitive values.
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
|
Thoughts on this @captainsafia |
|
Hey @copilot please move this out of the Azure TOC section, as it's not specific to Azure. Also, please rename the doc, the H1 and the TOC entry to Deployment state caching. |
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Moved the documentation out of the Azure TOC section to sit directly under "Publish and Deploy" in commit 5ab8d29. Also renamed:
Updated all references and added a redirect for the old URL. |
| run: aspire deploy --environment production | ||
| env: | ||
| AZURE_CREDENTIALS: ${{ secrets.AZURE_CREDENTIALS }} | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @copilot do we need to also cache the deployment state after the deploy here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The actions/cache@v4 action automatically saves the cache after the job completes successfully - no additional step needed. Updated the explanation in commit 2d0bdce to explicitly mention that the cache is automatically saved after the deployment step runs.
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
* Document ClaimActions configuration for dashboard OpenID Connect authentication (#5192) * Initial plan * Document ClaimActions configuration for dashboard OIDC authentication Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com> * Apply suggestion from @JamesNK * Apply suggestion from @JamesNK * Apply suggestion from @JamesNK * Update configuration.md --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com> Co-authored-by: James Newton-King <james@newtonking.com> * Document breaking change: DefaultAzureCredential defaults to ManagedIdentityCredential on ACA and App Service (#5303) * Initial plan * Add breaking change documentation for DefaultAzureCredential in 13.0 Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Fix markdown linting error - add blank line before list Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update docs/compatibility/13.0/defaultazurecredential-managedidentity-default.md --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: David Pine <david.pine@microsoft.com> * Update Azure AI Foundry and GitHub Models documentation to use strongly-typed model constants (#5302) * Initial plan * Update documentation to use AIFoundryModel and GitHubModel constants Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update compatibility matrix to use GitHubModel constant Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Apply suggestions from code review --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: David Pine <david.pine@microsoft.com> * Update environment variable format to polyglot-friendly pattern (#5320) * Initial plan * Update environment variable format to polyglot-friendly pattern Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update AppHost project documentation for Aspire 13.0 SDK format (#5319) * Initial plan * Update AppHost project section to show new SDK format for Aspire 13.0 Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Add AddCSharpApp and CSharpAppResource documentation to app-host-overview.md (#5318) * Initial plan * Add AddCSharpApp and CSharpAppResource documentation to app-host-overview.md Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Apply suggestions from code review * Update docs/fundamentals/app-host-overview.md --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: David Pine <david.pine@microsoft.com> * Add breaking changes documentation for pipeline activity reporter API rename (PR #12137) (#5310) * Initial plan * Add breaking changes article for PR #12137 and update documentation Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Fix parameter naming consistency in breaking change examples Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Fix cross reference warnings by using inline code formatting Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: David Pine <david.pine@microsoft.com> * Add documentation for ASPIREPIPELINES001, ASPIREPIPELINES002, and ASPIREPIPELINES003 diagnostics (#5359) * Initial plan * Add documentation for three new pipeline-related diagnostics Co-authored-by: captainsafia <1857993+captainsafia@users.noreply.github.com> * Fix ms.date to reflect current date in diagnostic documentation Co-authored-by: captainsafia <1857993+captainsafia@users.noreply.github.com> * Update diagnostic codes to use ASPIREPIPELINES001-003 instead of mixed naming scheme Co-authored-by: captainsafia <1857993+captainsafia@users.noreply.github.com> * Add ASPIREPIPELINES001-003 diagnostics to TOC Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: captainsafia <1857993+captainsafia@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Add Aspire MCP docs (#5446) * Update configuration * Document deployment state caching (#5301) * Initial plan * Add local deployment state documentation Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Address feedback: Add intros, italicize paths, add CI/CD example Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Rename and relocate deployment state doc outside Azure section Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Clarify cache save behavior in GitHub Actions example Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Add certificate trust customization article for Aspire 13 (#5311) * Initial plan * Add certificate trust customization article Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Fix cross-reference title to match actual article title Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Address PR feedback: update API examples, add runtime callout, document default scopes Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update callback APIs to use async Task and simplified context objects Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Replace deprecated callback APIs with new simplified APIs Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update documentation for new polyglot environment variables (issue #5308) (#5313) * Initial plan * Update documentation with new polyglot environment variable format Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Add clarification about resource names in environment variables Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Improve clarity of note about connection name parameter Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> * Update docs/fundamentals/app-host-overview.md --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: David Pine <david.pine@microsoft.com> * Update documentation for Aspire 13 release * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix: build --------- Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com> Co-authored-by: James Newton-King <james@newtonking.com> Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com> Co-authored-by: captainsafia <1857993+captainsafia@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
deployment/aspire-deploy/local-deployment-state.mdOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Internal previews