Add DR deployment - #138
Open
danzhu54 wants to merge 17 commits into
Open
Conversation
WeihanLi
reviewed
Mar 4, 2026
Removes the explicit Release configuration from the web application publish command so the project uses its default publish settings.
Adds a disaster recovery stage that uploads the Velopack release to the DR storage account before production, with production approval now depending on DR completion.
Skip downloads when the Azure Storage container does not exist and create it before uploading the first Velopack release.
Run catalog generation in DR after Dev and before Production using DR-specific storage, webhook secret, and Azure subscription settings.
Check for the catalog container before generation and create it when missing, failing clearly if Azure Storage operations fail.
Runs GenDesignNotes in DR before Production using DR-specific Azure credentials and configuration.
temporary remove depends on dev for testing
Adds a DR deployment stage before Production and parses NuGet references using the final slash to support nested paths.
Re-enables the catalog generation Dev stage and makes DR depend on it, while disabling the usage generation Dev stage and removing its DR dependency.
DownloadNuGetUsageDatabaseAsync now returns a bool indicating whether the database blob existed, catching a 404 BlobNotFound error instead of throwing. CrawlMain logs a message and creates a new database when none was previously indexed, avoiding a crash on first run.
DownloadPlannerUsageDatabaseAsync now catches BlobNotFound (404) errors and returns (false, null) instead of throwing, allowing GenUsagePlanner to create a new database when no previously indexed usages exist.
Run Dev before DR, and let DR use the template’s default build type.
danzhu54
commented
Aug 17, 2026
| ThrowIfNullOrEmpty(referenceIdentifier); | ||
|
|
||
| var indexOfSlash = referenceIdentifier.IndexOf('/'); | ||
| var indexOfSlash = referenceIdentifier.LastIndexOf('/'); |
Collaborator
Author
There was a problem hiding this comment.
What is the purpose of this change? @syrle-foronda
Member
There was a problem hiding this comment.
some package has multi deep / that cause error on getting actual version
package-name/OS/version <- this cause issue unlike normally written package name `package-name-OS/version'
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
3 participants
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.
Changes: