Skip to content

Enhance Azure Logs Setup Detection and Configuration#122

Closed
jongio wants to merge 2 commits intomainfrom
fixazlogs
Closed

Enhance Azure Logs Setup Detection and Configuration#122
jongio wants to merge 2 commits intomainfrom
fixazlogs

Conversation

@jongio
Copy link
Copy Markdown
Owner

@jongio jongio commented Jan 30, 2026

Improve Azure Logs setup detection by adding logic to differentiate between deployed but not configured and not deployed states. Introduce an auto-fix feature for missing Bicep outputs and enhance documentation to clarify optional configurations. Expand test coverage for new detection states and auto-fix functionality.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 30, 2026

🚀 Website Preview

Your PR preview is ready!

📎 Preview URL: https://jongio.github.io/azd-app/pr/122/

This preview will be automatically cleaned up when the PR is closed.

github-actions Bot added a commit that referenced this pull request Jan 30, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026

Codecov Report

❌ Patch coverage is 61.97183% with 81 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.82%. Comparing base (13ccec3) to head (df39444).
⚠️ Report is 30 commits behind head on main.

Files with missing lines Patch % Lines
cli/src/internal/azure/discovery.go 24.65% 55 Missing ⚠️
cli/src/internal/dashboard/azure_setup_autofix.go 84.54% 15 Missing and 2 partials ⚠️
cli/src/internal/dashboard/azure_setup.go 68.96% 7 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   54.72%   54.82%   +0.09%     
==========================================
  Files         148      149       +1     
  Lines       22318    22525     +207     
==========================================
+ Hits        12214    12349     +135     
- Misses       9340     9408      +68     
- Partials      764      768       +4     
Flag Coverage Δ
unittests 54.82% <61.97%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
cli/src/internal/dashboard/server_routes.go 58.62% <100.00%> (+0.72%) ⬆️
cli/src/internal/dashboard/azure_setup.go 38.75% <68.96%> (+1.80%) ⬆️
cli/src/internal/dashboard/azure_setup_autofix.go 84.54% <84.54%> (ø)
cli/src/internal/azure/discovery.go 32.40% <24.65%> (-2.22%) ⬇️

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- Add missing outputs for Log Analytics workspace in temp.bicep to expose workspace details.
- Improve setup detection logic to differentiate between "deployed but not configured" and "not deployed" states.
- Implement a collapsible section in the setup guide for quick fixes related to missing Bicep outputs.
- Update documentation to clarify that azure.yaml configuration is optional and primarily for overrides.
- Enhance auto-discovery reliability with caching, better error reporting, and support for multiple resource groups.
- Introduce an "Auto-Fix" button in the setup guide for one-click resolution of missing outputs.
- Expand test coverage for new detection states and auto-fix functionality.
github-actions Bot added a commit that referenced this pull request Feb 27, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions Bot added a commit that referenced this pull request Feb 27, 2026
@jongio
Copy link
Copy Markdown
Owner Author

jongio commented Mar 16, 2026

Closing: This PR is 5 months old with merge conflicts. Please re-open from a fresh branch if still needed.

@jongio jongio closed this Mar 16, 2026
@jongio jongio deleted the fixazlogs branch March 16, 2026 16:11
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