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

DTSCCI-1136 upgrade terraform version inline with min version #4990

Merged
merged 5 commits into from
Feb 3, 2025

Conversation

rishikrsharma
Copy link
Contributor

JIRA link

DTSCCI-1136

Change description

upgrade terraform version inline with min version

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No

@rishikrsharma rishikrsharma requested a review from a team as a code owner November 7, 2024 18:41
@rishikrsharma rishikrsharma requested review from nigeldunne and sherlynkhaw and removed request for a team November 7, 2024 18:41
@rishikrsharma rishikrsharma reopened this Feb 3, 2025
@hmcts-jenkins-a-to-c
Copy link
Contributor

Plan Result (aat)

Plan: 0 to add, 1 to change, 0 to destroy.
  • Update
    • module.citizen-ui-draft-store.azurerm_redis_cache.redis
Change Result (Click me)
  # module.citizen-ui-draft-store.azurerm_redis_cache.redis will be updated in-place
  ~ resource "azurerm_redis_cache" "redis" {
        id                                 = "/subscriptions/1c4f0704-a29e-403d-b719-b90c34ef14c9/resourceGroups/civil-citizen-ui-draft-store-cache-aat/providers/Microsoft.Cache/redis/civil-citizen-ui-draft-store-aat"
        name                               = "civil-citizen-ui-draft-store-aat"
        tags                               = {
            "application"         = "civil"
            "autoShutdown"        = "true"
            "builtFrom"           = "https://github.com/HMCTS/civil-citizen-ui.git"
            "businessArea"        = "CFT"
            "contactSlackChannel" = "#civil_contact"
            "environment"         = "staging"
            "managedBy"           = "Civil"
        }
        # (24 unchanged attributes hidden)

      ~ redis_configuration {
          - data_persistence_authentication_method  = "SAS" -> null
          ~ maxfragmentationmemory_reserved         = 125 -> 642
          ~ maxmemory_delta                         = 125 -> 642
          ~ maxmemory_reserved                      = 125 -> 642
            # (13 unchanged attributes hidden)
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

@hmcts-jenkins-a-to-c
Copy link
Contributor

Plan Result (prod)

Plan: 0 to add, 1 to change, 0 to destroy.
  • Update
    • module.citizen-ui-draft-store.azurerm_redis_cache.redis
Change Result (Click me)
  # module.citizen-ui-draft-store.azurerm_redis_cache.redis will be updated in-place
  ~ resource "azurerm_redis_cache" "redis" {
        id                                 = "/subscriptions/8999dec3-0104-4a27-94ee-6588559729d1/resourceGroups/civil-citizen-ui-draft-store-cache-prod/providers/Microsoft.Cache/redis/civil-citizen-ui-draft-store-prod"
        name                               = "civil-citizen-ui-draft-store-prod"
        tags                               = {
            "application"         = "civil"
            "builtFrom"           = "https://github.com/HMCTS/civil-citizen-ui.git"
            "businessArea"        = "CFT"
            "contactSlackChannel" = "#civil_contact"
            "environment"         = "production"
            "managedBy"           = "Civil"
        }
        # (24 unchanged attributes hidden)

      ~ redis_configuration {
          - data_persistence_authentication_method  = "SAS" -> null
            # (16 unchanged attributes hidden)
        }
    }

Plan: 0 to add, 1 to change, 0 to destroy.

gmikerlemos
gmikerlemos previously approved these changes Feb 3, 2025
velimd
velimd previously approved these changes Feb 3, 2025
@rishikrsharma rishikrsharma merged commit 09f17f6 into master Feb 3, 2025
4 of 5 checks passed
@rishikrsharma rishikrsharma deleted the DTSCCI-1136-terraform-version-upgrade branch February 3, 2025 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants