Skip to content

Add 'finrem' to additional managed identities access#218

Open
kiran-yenigala-hmcts wants to merge 1 commit into
masterfrom
kiran-yenigala-hmcts-patch-1
Open

Add 'finrem' to additional managed identities access#218
kiran-yenigala-hmcts wants to merge 1 commit into
masterfrom
kiran-yenigala-hmcts-patch-1

Conversation

@kiran-yenigala-hmcts

Copy link
Copy Markdown
Contributor

Before creating a pull request make sure that:

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:

JIRA link (if applicable)

Change description

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

  • Yes
  • No

@hmcts-jenkins-a-to-c

Copy link
Copy Markdown
Contributor

Plan Result (aat)

Plan: 1 to add, 0 to change, 0 to destroy.
  • Create
    • module.vault.azurerm_key_vault_access_policy.managed_identity_names_access_policy["finrem"]
Change Result (Click me)
  # module.vault.azurerm_key_vault_access_policy.managed_identity_names_access_policy["finrem"] will be created
  + resource "azurerm_key_vault_access_policy" "managed_identity_names_access_policy" {
      + id                 = (known after apply)
      + key_vault_id       = "/subscriptions/1c4f0704-a29e-403d-b719-b90c34ef14c9/resourceGroups/ccd-shared-aat/providers/Microsoft.KeyVault/vaults/ccd-aat"
      + object_id          = "d4ead5ad-2d69-40d1-946e-3bcdf7626dc8"
      + secret_permissions = [
          + "Get",
          + "List",
        ]
      + tenant_id          = "531ff96d-0ae9-462a-8d2d-bec7c0b42082"
    }

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

@hmcts-jenkins-a-to-c

Copy link
Copy Markdown
Contributor

Plan Result (prod)

No changes. Your infrastructure matches the configuration.

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.

2 participants