Skip to content

Improve azure error logging#2025

Draft
bec-callow-oct wants to merge 4 commits into
mainfrom
bec/hpy-1405-improve-azure-error-log
Draft

Improve azure error logging#2025
bec-callow-oct wants to merge 4 commits into
mainfrom
bec/hpy-1405-improve-azure-error-log

Conversation

@bec-callow-oct

@bec-callow-oct bec-callow-oct commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

⚠️ Does this change require a corresponding Server Change?
⚠️ If so - please add a "Requires Server Change" label to this PR!

When deploying to Azure via Bicep/ARM templates Octopus task logs display Azure failures with minimal context. This PR retrieves additional information from Azure so that we can provide a more useful error message to the user.

bec-callow-oct and others added 4 commits June 18, 2026 10:01
This change belongs in a separate PR. Reverts commit 7c70d3a.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Extract repeated base error message string into a local variable
- Rename operations list to failedOperations for clarity
- Remove redundant failureCount variable (operations.Count is equivalent)
- Remove JObject.FromObject fallback in FormatStatusMessage which silently
  dropped ARM JSON fields not mapped to StatusMessage's public properties

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant