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

[CES-109] Switch Messages Cosmos secondary location to ItalyNorth #1188

Merged

Conversation

mamu0
Copy link
Contributor

@mamu0 mamu0 commented Sep 25, 2024

List of changes

Replace NorthEurope with ItalyNorth region for data redundancy in read-only mode for messages namespace
Added Private endpoint for italy north.

Motivation and context

ItalyNorth migration. This is a first step to have data in ItalyNorth without affecting operations on the main region, WestEurope

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Env to apply

  • DEV
  • UAT
  • PROD

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

Other information


If PR is partially applied, why? (reserved to mantainers)

How to apply

After PR is approved

  1. run deploy pipeline from Azure DevOps io-platform-iac-projects
  2. select PR branch
  3. wait for approval

Copy link
Contributor

@Krusty93 Krusty93 left a comment

Choose a reason for hiding this comment

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

module.lollipop_assertions_storage.azurerm_advanced_threat_protection.this is going to be deleted

EDIT: it does make sense if these variables have the following values:
advanced_threat_protection: true
use_legacy_defender_version: false

src/domains/citizen-auth-common/01_network.tf Outdated Show resolved Hide resolved
src/domains/citizen-auth-common/01_network.tf Outdated Show resolved Hide resolved
Copy link
Contributor

@BurnedMarshal BurnedMarshal 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

sonarcloud bot commented Sep 27, 2024

@mamu0 mamu0 merged commit a56097a into main Sep 27, 2024
6 checks passed
@mamu0 mamu0 deleted the CES-109-cosmos-db-replica-read-only-italy-north-autenticazione branch September 27, 2024 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants