Skip to content

Fix USGov cloud FinOps hub deployment time zone error (#2087)#2097

Open
RolandKrummenacher wants to merge 2 commits intodevfrom
krummrol/fix-2087-govcloud-timezone
Open

Fix USGov cloud FinOps hub deployment time zone error (#2087)#2097
RolandKrummenacher wants to merge 2 commits intodevfrom
krummrol/fix-2087-govcloud-timezone

Conversation

@RolandKrummenacher
Copy link
Copy Markdown
Collaborator

Summary

Fixes #2087 — FinOps hub deployment to USGov Virginia fails with ErrorCode=InvalidWorkflowTriggerRecurrence, ErrorMessage=The recurrence of trigger has an invalid time zone 'Universal Coordinated Time'.

Two root causes in timeZones.bicep:

  1. No USGov/DoD regions in the region-to-time-zone map, so gov deployments fell through to the default.
  2. The default value 'Universal Coordinated Time' is the display name, not a Windows time zone ID. Per the ADF schedule trigger docs the valid ID is UTC.

Changes

  • Added mappings for usgovvirginia, usgovtexas, usgovarizona (US Mountain Standard Time — no DST), usgoviowa, usdodcentral, usdodeast.
  • Changed the fallback from 'Universal Coordinated Time' to 'UTC'.
  • Added doc-link comments so future region additions use correct Windows time zone IDs.
  • Added changelog entry under v14 hubs Fixed.

Test plan

  • 🤏 Lint / bicep build passes
  • 👍 Manually deploy to USGov Virginia and confirm Data Factory triggers activate
  • 👍 Redeploy to a commercial region (e.g. eastus) and confirm no regression
  • 💪 Pester unit tests
  • 🙌 Integration tests

Notes for reviewer

The ADF docs also state that when timeZone is UTC, startTime/endTime must be suffixed with Z; for named zones they must not be. The startTime values in the consuming bicep files (e.g. '2023-01-01T01:01:00') have no Z, which matches the named-zone format. With USGov regions now mapping to named zones, the existing strings remain valid. A future region that falls through to the UTC fallback could trip this rule on trigger activation — out of scope for this fix, but worth tracking if we ever want triggers to work without an explicit mapping.

🤖 Generated with Claude Code

Fixes #2087

- Add USGov (Virginia, Texas, Arizona, Iowa) and DoD (Central, East) region-to-timezone mappings so Data Factory schedule triggers activate successfully on Gov Cloud deployments.
- Change the fallback time zone from the invalid display name 'Universal Coordinated Time' to the Windows time zone ID 'UTC'.
- Add doc-link comments explaining the constraint so future region additions use the correct Windows time zone IDs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@RolandKrummenacher RolandKrummenacher added this to the v14 milestone Apr 16, 2026
@RolandKrummenacher RolandKrummenacher self-assigned this Apr 16, 2026
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.

FinOps Hubs for Gov Cloud Deployment Error: "Invalid Time Zone"

3 participants