You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
azapi provider: Support preserve_resource_id_casing feature flag (default false). When enabled, if the resource ID the provider would write back to state differs from the existing state value only by casing, the existing casing is preserved. This avoids spurious Unexpected Identity Change errors and diffs when consumers (or upstream modules) rely on a specific casing the Azure API may not preserve. Only the id and resource_id attributes are affected. Can also be sourced from the ARM_PRESERVE_RESOURCE_ID_CASING environment variable (GH-1122).
azapi provider: Publish official windows_arm64 provider binaries (GH-1148).
azapi_resource resource: Re-introduce the ignore_body_changes property to ignore changes to specified fields in the request body (GH-1192).
azapi_data_plane_resource resource: Allow name to be optional based on the URL format (GH-1178).
azapi_data_plane_resource resource: Support Key Vault certificates (GH-1174).
BUG FIXES:
Strip UTF-8 BOM prefix from JSON responses (GH-1190).
Preserve azapi_resource state for an in-flight Create interrupted by SIGINT (GH-1183).
Fix MoveState target identity for azurerm resource moves (GH-1175).
What this PR does.What this PR does: Bumps the azapi Terraform provider version from 2.11.0 to 2.12.0 in the test infrastructure for the azure-adf-health codebundle. This is an automated minor dependency update generated by Renovate Bot.
This pass. ## What this PR does
This PR, generated by Renovate Bot, is titled as a Terraform azapi provider version bump (2.11.0 → 2.12.0), but it also includes changes to the LiteLLM spend governance codebundle templates:
Bumps the azapi provider version in a test Terraform config.
Changes the default LITELLM_SPEND_THRESHOLD_USD from "0" to "100" in the SLI template.
Refactors the taskset template's runtimeVarsProvided from inline Jinja value expressions to a structured format with default, description, and validation (regex patterns).
What the review found
One major finding: The default value for LITELLM_SPEND_THRESHOLD_USD is inconsistent between the two templates. The SLI template was intentionally changed to default to "100", but the taskset template sets it to "0". Previously both agreed on "0". This mismatch could cause confusing behavior where the SLI alerts at $100 but the taskset has alerting disabled.
The Terraform provider bump and the taskset template restructuring (descriptions, validation patterns) are clean. The regex validation patterns are all well-formed and appropriate for their respective fields.
This review is diff-only (no repository checkout available). No review skills were loaded (none were available); the review applied baseline correctness, security, and obvious-defect lenses.
ID
Status
Finding
Where
F1
new · major
Default for LITELLM_SPEND_THRESHOLD_USD is inconsistent between SLI and taskset templates
k8s-litellm-spend-governance-taskset.yaml:64
How this was reviewed
Incremental diff of 3 files at 0bc6413 · shallow · 0 skills · summary only.
Took 45s: preparation 5s, AI review 40s.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
2.11.0->2.12.0Release Notes
Azure/terraform-provider-azapi (azapi)
v2.12.0Compare Source
ENHANCEMENTS:
azapiprovider: Supportpreserve_resource_id_casingfeature flag (defaultfalse). When enabled, if the resource ID the provider would write back to state differs from the existing state value only by casing, the existing casing is preserved. This avoids spuriousUnexpected Identity Changeerrors and diffs when consumers (or upstream modules) rely on a specific casing the Azure API may not preserve. Only theidandresource_idattributes are affected. Can also be sourced from theARM_PRESERVE_RESOURCE_ID_CASINGenvironment variable (GH-1122).azapiprovider: Publish official windows_arm64 provider binaries (GH-1148).azapi_resourceresource: Re-introduce theignore_body_changesproperty to ignore changes to specified fields in the request body (GH-1192).azapi_data_plane_resourceresource: Allownameto be optional based on the URL format (GH-1178).azapi_data_plane_resourceresource: Support Key Vault certificates (GH-1174).BUG FIXES:
azapi_resourcestate for an in-flight Create interrupted by SIGINT (GH-1183).MoveStatetarget identity forazurermresource moves (GH-1175).Configuration
📅 Schedule: Branch creation - "before 10am on monday" in timezone America/New_York, Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Renovate Bot.