Skip to content

Fix incorrect default values and parameter name in hubs template docs#2086

Merged
flanakin merged 3 commits intodevfrom
fix/enableRecommendations-default-doc
Apr 14, 2026
Merged

Fix incorrect default values and parameter name in hubs template docs#2086
flanakin merged 3 commits intodevfrom
fix/enableRecommendations-default-doc

Conversation

@RolandKrummenacher
Copy link
Copy Markdown
Collaborator

@RolandKrummenacher RolandKrummenacher commented Apr 13, 2026

Summary

Fix three documentation errors in the FinOps hubs template parameter table (docs-mslearn/toolkit/hubs/template.md):

  • enableRecommendations: Default listed as True, but Bicep template defines false
  • dataExplorerSkuName: Default listed as Dev(No SLA)_Standard_E2a_v4, but Bicep template uses Dev(No SLA)_Standard_D11_v2
  • dataExplorerLogRetentionInDays: Parameter name and description referenced *_log tables, but the actual Bicep parameter is dataExplorerRawRetentionInDays referencing *_raw tables

Verified that createUiDefinition.json and Deploy-Hub.ps1 are consistent with the Bicep defaults — the errors were only in the docs.

Test plan

  • Verified defaults match main.bicep and modules/hub.bicep
  • Verified createUiDefinition.json defaults are consistent
  • Verify docs render correctly on Microsoft Learn

🤖 Generated with Claude Code

The documentation incorrectly listed the default value for enableRecommendations
as True, but the Bicep template defines it as false.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
github-actions bot and others added 2 commits April 13, 2026 13:57
- dataExplorerSkuName default was E2a_v4, but Bicep uses D11_v2
- dataExplorerLogRetentionInDays referenced *_log tables, but the actual
  parameter is dataExplorerRawRetentionInDays referencing *_raw tables

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@RolandKrummenacher RolandKrummenacher changed the title Fix enableRecommendations default value in docs Fix incorrect default values and parameter name in hubs template docs Apr 13, 2026
@flanakin flanakin merged commit 281527d into dev Apr 14, 2026
5 checks passed
@flanakin flanakin deleted the fix/enableRecommendations-default-doc branch April 14, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants