Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update terraform providers #343

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

RMcVelia
Copy link
Contributor

@RMcVelia RMcVelia commented Jun 3, 2024

Context

Update terraform providers as azurerm is using a version of the preview api that will become eol shortly

Changes proposed in this pull request

azurerm -> 3.104.2
statuscake -> 2.2.0
kubernetes -> 2.30.0

Guidance to review

make env terraform-plan
make dns domains-infra-plan
make env dns domains-plan

redis will show below. data_persistence_authentication_method was added in a later module, SAS is the default.

  # module.redis.azurerm_redis_cache.main[0] will be updated in-place
  ~ resource "azurerm_redis_cache" "main" {

      ~ redis_configuration {
          + data_persistence_authentication_method  = "SAS"
            # (11 unchanged attributes hidden)
        }

      ~ timeouts {
          ~ create = "30m" -> "1h"
          ~ update = "30m" -> "1h"
        }

Link to Trello card

https://trello.com/c/ST9hu2Bq/1840-update-azurerm-provider-to-31042

Checklist

  • Attach to Trello card
  • Rebased main
  • Cleaned commit history
  • Tested by running locally

@RMcVelia RMcVelia added devops deploy Deploys a review app labels Jun 4, 2024
@github-actions github-actions bot temporarily deployed to review-343 June 4, 2024 10:43 Destroyed
@RMcVelia RMcVelia merged commit 22017f2 into main Jun 4, 2024
24 checks passed
@RMcVelia RMcVelia deleted the 1840-update-azurerm-provider-to-31042 branch June 4, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Deploys a review app devops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants