Skip to content

Fix Japan time zone name for managed exports - #2219

Closed
TakumiSeo wants to merge 2 commits into
devfrom
main
Closed

Fix Japan time zone name for managed exports#2219
TakumiSeo wants to merge 2 commits into
devfrom
main

Conversation

@TakumiSeo

@TakumiSeo TakumiSeo commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

🛠️ Description

Change the managed exports time zone value for Japan from Japan to Tokyo.

Azure Cost Management exports recognize Tokyo as the supported time zone name. Using Japan causes managed export deployment validation to fail.

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

📦 Deploy to test?

  • Hubs + ADX (managed)
  • Hubs + Fabric (manual)
  • Hubs (manual)
  • Hubs (no data)
  • Workbooks
  • Alerts

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ✅ Updated changelog (required for dev PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in docs (required for dev)
  • ✅ Public docs in docs-mslearn (required for dev)
  • ✅ Internal dev docs in docs-wiki (required for dev)
  • ✅ Internal dev docs in src (required for dev)
  • ➡️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

Copilot AI review requested due to automatic review settings July 29, 2026 08:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Micro PR 🔬 Very small PR that should be especially easy for newcomers label Jul 29, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Skill: DevOps GitHub setup and automation label Jul 29, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added Skill: Documentation Documentation updates Tool: FinOps guide Implementing FinOps guide Needs: Review 👀 PR that is ready to be reviewed labels Jul 29, 2026
@MSBrett

MSBrett commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

We're missing the actual bicep changes for the TZ.

@MSBrett MSBrett added Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity Needs: Information Untriaged issue needs more information to be triaged and removed Needs: Review 👀 PR that is ready to be reviewed Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity labels Jul 29, 2026
@flanakin

flanakin commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

@TakumiSeo Can you include the commits needed for this PR? Let us know if you need help!

@TakumiSeo

Copy link
Copy Markdown
Contributor Author

I opened a corrected PR from my fork with the intended Bicep changes: #2236

This PR was accidentally opened from microsoft:main and does not contain the file changes, so the corrected PR supersedes it. Sorry for the confusion.

@microsoft-github-policy-service microsoft-github-policy-service Bot added Needs: Triage 🔍 Untriaged issue needs to be reviewed Needs: Review 👀 PR that is ready to be reviewed and removed Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity Needs: Information Untriaged issue needs more information to be triaged labels Aug 4, 2026
@flanakin flanakin closed this Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Micro PR 🔬 Very small PR that should be especially easy for newcomers Needs: Review 👀 PR that is ready to be reviewed Needs: Triage 🔍 Untriaged issue needs to be reviewed Skill: DevOps GitHub setup and automation Skill: Documentation Documentation updates Tool: FinOps guide Implementing FinOps guide

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants