Skip to content

Conversation

@henryde
Copy link
Contributor

@henryde henryde commented Nov 19, 2025

No description provided.

@henryde henryde requested review from Copilot and sttomm November 19, 2025 11:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds quota support to landing zone resources and data sources in the Terraform provider. It allows users to define and manage quota key-value pairs for landing zones.

Key Changes:

  • Added quotas field as a list of key-value pairs to landing zone resource and data source schemas
  • Updated client types to support the new quota structure
  • Upgraded Terraform plugin framework dependencies to support the new schema features

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
internal/provider/landingzone_resource.go Added quotas schema with list default value and nested key-value attributes
internal/provider/landingzone_data_source.go Added computed quotas schema with nested key-value attributes
client/landingzone.go Added MeshLandingZoneQuota struct and renamed types for consistency; added Quotas field to spec
go.mod Updated Terraform plugin framework dependencies to versions supporting listdefault
docs/resources/landingzone.md Added documentation for the new quotas attribute
docs/data-sources/landingzone.md Added documentation for the new computed quotas attribute
CHANGELOG.md Added feature entry for quota support
Comments suppressed due to low confidence (1)

go.mod:3

  • Go 1.24.0 does not exist. The latest stable Go version as of November 2025 would be in the 1.23.x series. This should be corrected to a valid Go version like 1.23 or the actual version being used.
go 1.24.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@henryde henryde merged commit 5ce99b5 into main Nov 19, 2025
16 checks passed
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.

3 participants