-
Notifications
You must be signed in to change notification settings - Fork 191
Fix config_InitializeHub failure caused by HTTP redirects in Azure Data Explorer externaldata commands #1908
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
base: dev
Are you sure you want to change the base?
Conversation
…RLs for open data CSV files Co-authored-by: RolandKrummenacher <[email protected]>
RolandKrummenacher
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.
@copilot Why have you made changes in docs/deploy? The issue is not related to remoteHub configuration. Keep the change log and the changes in app.bicep.
…og fixes Co-authored-by: RolandKrummenacher <[email protected]>
You're right - the docs/deploy changes included unrelated remoteHub configuration that was already in the source but hadn't been compiled yet. I've reverted those files in commit 6aa9627, keeping only the app.bicep and changelog changes. The docs/deploy JSON files will be properly updated during the official release build process. |
| ? 'https://github.com/microsoft/finops-toolkit/releases/latest/download' | ||
| : 'https://github.com/microsoft/finops-toolkit/releases/download/v${finOpsToolkitVersion}' | ||
| ? 'https://raw.githubusercontent.com/microsoft/finops-toolkit/refs/heads/dev/src/open-data' | ||
| : 'https://raw.githubusercontent.com/microsoft/finops-toolkit/refs/tags/v${finOpsToolkitVersion}/src/open-data' |
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.
🛠️ Description
GitHub releases download URLs now return HTTP 302 redirects. Azure Data Explorer's
externaldata()command rejects redirects, causingconfig_InitializeHubpipeline failures when loading open data CSV files (PricingUnits, Regions, ResourceTypes, Services).Changed URL pattern:
Files modified:
ftkReleaseUrivariable insrc/templates/finops-hub/modules/Microsoft.FinOpsHubs/Analytics/app.bicepdocs-mslearn/toolkit/changelog.mdNote: The compiled deployment templates in
docs/deploy/will be updated during the official release build process.Affects all FinOps Hub deployments using Azure Data Explorer or Microsoft Fabric.
Related to #1885
📷 Screenshots
📋 Checklist
🔬 How did you test this change?
🙋♀️ Do any of the following that apply?
📑 Did you update
docs/changelog.md?📖 Did you update documentation?
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.