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

DTSAM-288: Updated new case ids. #2223

Merged
merged 6 commits into from
May 20, 2024
Merged

Conversation

SivaKanukolluQA
Copy link
Contributor

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/DTSAM-288

Change description

Updated new case IDs.

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

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

Plan Result (aat)

⚠️ Resource Deletion will happen ⚠️

This plan contains resource delete operation. Please check the plan result very carefully!

Plan: 2 to add, 0 to change, 2 to destroy.
  • Replace
    • module.role-assignment-database-v15.null_resource.set-schema-ownership["role_assignment"]
    • module.role-assignment-database-v15.null_resource.set-user-permissions-additionaldbs["role_assignment"]
Change Result (Click me)
  # module.role-assignment-database-v15.null_resource.set-schema-ownership["role_assignment"] must be replaced
-/+ resource "null_resource" "set-schema-ownership" {
      ~ id       = "5806576767294518690" -> (known after apply)
      ~ triggers = { # forces replacement
          ~ "force_trigger" = "5" -> "4"
            # (2 unchanged elements hidden)
        }
    }

  # module.role-assignment-database-v15.null_resource.set-user-permissions-additionaldbs["role_assignment"] must be replaced
-/+ resource "null_resource" "set-user-permissions-additionaldbs" {
      ~ id       = "248906929758881127" -> (known after apply)
      ~ triggers = { # forces replacement
          ~ "force_trigger"  = "5" -> "4"
            # (3 unchanged elements hidden)
        }
    }

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

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

Plan Result (prod)

⚠️ Resource Deletion will happen ⚠️

This plan contains resource delete operation. Please check the plan result very carefully!

Plan: 1 to add, 0 to change, 1 to destroy.
  • Replace
    • module.role-assignment-database-v15.null_resource.set-user-permissions-additionaldbs["role_assignment"]
Change Result (Click me)
  # module.role-assignment-database-v15.null_resource.set-user-permissions-additionaldbs["role_assignment"] must be replaced
-/+ resource "null_resource" "set-user-permissions-additionaldbs" {
      ~ id       = "819940353534217651" -> (known after apply)
      ~ triggers = { # forces replacement
          ~ "force_trigger"  = "5" -> "4"
            # (3 unchanged elements hidden)
        }
    }

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

mikebrownccd
mikebrownccd previously approved these changes May 3, 2024
Copy link
Contributor

@mikebrownccd mikebrownccd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

abul-syed
abul-syed previously approved these changes May 3, 2024
Copy link
Contributor

@abul-syed abul-syed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

mikebrownccd
mikebrownccd previously approved these changes May 9, 2024
Copy link
Contributor

@mikebrownccd mikebrownccd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

abul-syed
abul-syed previously approved these changes May 9, 2024
Copy link
Contributor

@abul-syed abul-syed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

mattnayler
mattnayler previously approved these changes May 13, 2024
Copy link
Contributor

@mikebrownccd mikebrownccd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@mattnayler mattnayler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-apporved

@mikebrownccd mikebrownccd merged commit e93bb2d into master May 20, 2024
5 checks passed
@mikebrownccd mikebrownccd deleted the DTSAM-288_FTA_new_case_ids branch May 20, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aat/destroy enable_keep_helm Do not uninstall helm release after pipeline completion ns:am prd:am prod/destroy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants