Azure / terraform-azurerm-avm-res-cdn-profile

AVM Terraform Module for Azure CDN Profiles
MIT License
3 stars 3 forks source link

[AVM Module Issue]: support for later azapi versions #46

Open MikeDembek opened 1 month ago

MikeDembek commented 1 month ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

(Optional) Module Version

0.1.0

(Optional) Correlation Id

No response

Description

This module pins azapi provider version 1.9, but at least some other AVMs, such as avm-res-network-virtualnetwork, require later versions (~> 1.13 in that example), which prevents concurrent use of those modules. Could this module be updated to permit later versions of the azapi provider?

Here's the issue I see on execution where the azapi provider version isn't constrained in my config, just in the referenced modules:

Could not retrieve the list of available versions for provider azure/azapi: no available releases match the given constraints 1.9.0, ~> 1.13

didayal-msft commented 1 month ago

Thank you @MikeDembek for submitting this feedback. We will review and work to update the version cc @Poven795909

microsoft-github-policy-service[bot] commented 1 month ago

[!WARNING] Tagging the AVM Core Team (@Azure/avm-core-team-technical-terraform) due to a module owner or contributor having not responded to this issue within 3 business days. The AVM Core Team will attempt to contact the module owners/contributors directly.

[!TIP]

  • To prevent further actions to take effect, the "Status: Response Overdue 🚩" label must be removed, once this issue has been responded to.
  • To avoid this rule being (re)triggered, the ""Needs: Triage :mag:" label must be removed as part of the triage process (when the issue is first responded to)!

[!NOTE] This message was posted as per ITA01TF.

Poven795909 commented 1 month ago

Will test with new AzApi version and will push the changes. Thanks for the feedback

microsoft-github-policy-service[bot] commented 4 weeks ago

[!WARNING] Tagging the AVM Core Team (@Azure/avm-core-team-technical-terraform) due to a module owner or contributor having not responded to this issue within 3 business days. The AVM Core Team will attempt to contact the module owners/contributors directly.

[!TIP]

  • To prevent further actions to take effect, the "Status: Response Overdue 🚩" label must be removed, once this issue has been responded to.
  • To avoid this rule being (re)triggered, the ""Needs: Triage :mag:" label must be removed as part of the triage process (when the issue is first responded to)!

[!NOTE] This message was posted as per ITA01TF.

microsoft-github-policy-service[bot] commented 2 weeks ago

[!WARNING] Tagging the AVM Core Team (@Azure/avm-core-team-technical-terraform) due to a module owner or contributor having not responded to this issue within 3 business days. The AVM Core Team will attempt to contact the module owners/contributors directly.

[!TIP]

  • To prevent further actions to take effect, the "Status: Response Overdue 🚩" label must be removed, once this issue has been responded to.
  • To avoid this rule being (re)triggered, the ""Needs: Triage :mag:" label must be removed as part of the triage process (when the issue is first responded to)!

[!NOTE] This message was posted as per ITA01TF.

microsoft-github-policy-service[bot] commented 2 weeks ago

[!CAUTION] This issue requires the AVM Core Team's (@Azure/avm-core-team-technical-terraform) immediate attention as it hasn't been responded to within 6 business days.

[!TIP]

  • To avoid this rule being (re)triggered, the "Needs: Triage :mag:" and "Status: Response Overdue :triangular_flag_on_post:" labels must be removed when the issue is first responded to!
  • Remove the "Needs: Immediate Attention :bangbang:" label once the issue has been responded to.

[!NOTE] This message was posted as per ITA02TF.

Poven795909 commented 1 week ago

@MikeDembek, apologies for the delay ! through internal testing and discussion, we have decided to wait for v2 for azapi to be released (tentatively by September end or Oct first week) by Hashicorp this to ensure migration to latest version of azapi doesnt cause any breaking change.