Skip to content

build(infrastructure): coordinate AzureRM 5.5 constraints - #1607

Open
Marcel Bindseil (bindsi) wants to merge 2 commits into
mainfrom
build/coordinated-azurerm-5.5
Open

Marcel Bindseil (bindsi) wants to merge 2 commits into
mainfrom
build/coordinated-azurerm-5.5

Conversation

@bindsi

Copy link
Copy Markdown
Member

Pull Request

Description

Coordinate the AzureRM 5.5 provider update across every Terraform deployment and
reusable module. This supersedes the fragmented Dependabot PRs #1569 through #1572,
which left child module constraints capped below AzureRM 5.5.

The change raises all AzureRM upper bounds to < 5.5.1, resolves AzureRM 5.5.0
through each Terraform root and module, and regenerates the provider documentation.

Type of Change

  • 🐛 Bug fix (non-breaking change fixing an issue)
  • ✨ New feature (non-breaking change adding functionality)
  • 💥 Breaking change (fix or feature causing existing functionality to change)
  • 📚 Documentation update
  • 🏗️ Infrastructure change (Terraform/IaC)
  • ♻️ Refactoring (no functional changes)

Component(s) Affected

  • infrastructure/terraform/prerequisites/ - Azure subscription setup
  • infrastructure/terraform/ - Terraform infrastructure
  • infrastructure/setup/ - OSMO control plane / Helm
  • workflows/ - Training and evaluation workflows
  • training/ - Training pipelines and scripts
  • docs/ - Documentation

Testing Performed

  • Terraform plan reviewed (no unexpected changes)
  • Terraform apply tested in dev environment
  • Training scripts tested locally with Isaac Sim
  • OSMO workflow submitted successfully
  • Smoke tests passed (smoke_test_azure.py)

Verified locally:

  • TFLint reported zero issues
  • Terraform formatting and validation passed for the root, VPN, DNS, and automation deployments
  • Terraform tests passed: 211 passed, 0 failed, 0 errors
  • All ten local Terraform lockfiles resolved AzureRM 5.5.0
  • Terraform documentation consistency check passed for all 11 generated references
  • Markdown spelling and patch hygiene checks passed

Terraform plan and apply remain pending because they require an authenticated Azure environment.

Documentation Impact

  • No documentation changes needed
  • Documentation updated in this PR
  • Documentation issue filed

Bug Fix Checklist

Complete this section for bug fix PRs. Skip for other contribution types.

  • Linked to issue being fixed
  • Regression test included, OR
  • Justification for no regression test:

Checklist

- synchronize deployment and module provider bounds
- regenerate Terraform provider documentation

🏗️ - Generated by Copilot
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA c83fc7e.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Sep 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.10%. Comparing base (b04a986) to head (c83fc7e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1607   +/-   ##
=======================================
  Coverage   88.10%   88.10%           
=======================================
  Files         279      279           
  Lines       23203    23203           
  Branches     3199     3199           
=======================================
  Hits        20444    20444           
  Misses       2071     2071           
  Partials      688      688           
Flag Coverage Δ *Carryforward flag
go 100.00% <ø> (ø)
pester 86.64% <ø> (ø) Carriedforward from b04a986
pytest-data-pipeline 100.00% <ø> (ø) Carriedforward from b04a986
pytest-dataviewer 89.66% <ø> (ø) Carriedforward from b04a986
pytest-dm-tools 100.00% <ø> (ø) Carriedforward from b04a986
pytest-evaluation 95.40% <ø> (ø)
pytest-fuzz 3.99% <ø> (ø) Carriedforward from b04a986
pytest-inference 100.00% <ø> (ø) Carriedforward from b04a986
pytest-shared-ci 100.00% <ø> (ø) Carriedforward from b04a986
pytest-training 92.60% <ø> (ø) Carriedforward from b04a986
vitest 85.66% <ø> (ø) Carriedforward from b04a986
vitest-app 85.66% <ø> (ø) Carriedforward from b04a986
vitest-components 85.66% <ø> (ø) Carriedforward from b04a986
vitest-features 85.66% <ø> (ø) Carriedforward from b04a986
vitest-lib 85.66% <ø> (ø) Carriedforward from b04a986
vitest-state 85.66% <ø> (ø) Carriedforward from b04a986

*This pull request uses carry forward flags. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for this one, I do wonder if we need to change the approach with Dependabot for the terraform versions. This is going to keep happening, and since dependabot cannot regenerate the docs, it's always going to be an incomplete PR.
At minimum I guess we should ensure one only dependabot PR is opened across all folders, to prevent multiple PRs. Wdyt?

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants