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 azurerm provider #425

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Conversation

RMcVelia
Copy link
Contributor

@RMcVelia RMcVelia commented Jun 5, 2024

Description

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

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

Also update github actions in the deploy action to latest versions

Changes proposed in this pull request

azurerm 3.43.0/3.53.0 -> 3.104.2
statuscake 2.1.0 -> 2.2.0
kubernetes 2.20.0 -> 2.30.0

Guidance to review

make env terraform-plan
make dns domains-infra-plan
make qa 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)
        }

@RMcVelia RMcVelia added deploy Deploy review app DevOps Infra team PR labels Jun 5, 2024
@RMcVelia RMcVelia changed the title 1840 update azurerm provider to 31042 Update azurerm provider Jun 5, 2024
@github-actions github-actions bot temporarily deployed to review-425 June 5, 2024 09:10 Destroyed
@github-actions github-actions bot temporarily deployed to review-425 June 5, 2024 09:13 Destroyed
@RMcVelia RMcVelia merged commit 6e9597f into main Jun 6, 2024
14 checks passed
@RMcVelia RMcVelia deleted the 1840-update-azurerm-provider-to-31042 branch June 6, 2024 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Deploy review app DevOps Infra team PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants